Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(335)

Issue 4317045: Patch to fix maven dependencies (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 9 months ago by Jasvir
Modified:
13 years, 9 months ago
Reviewers:
johnfargo, thorsten
CC:
shindig.remailer_gmail.com
Base URL:
https://svn.apache.org/repos/asf/shindig/trunk/
Visibility:
Public.

Description

* removes duplicate run dependencies

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -17 lines) Patch
M pom.xml View 1 chunk +0 lines, -17 lines 0 comments Download

Messages

Total messages: 3
Jasvir
14 years, 9 months ago (2011-03-25 23:40:59 UTC) #1
johnfargo
LGTM On 2011/03/25 23:40:59, jasvir wrote:
14 years, 9 months ago (2011-03-25 23:41:18 UTC) #2
thorsten_apache.org
14 years, 9 months ago (2011-03-28 08:20:40 UTC) #3
Can you please stop cross-posting?

salu2

On Fri, 2011-03-25 at 23:40 +0000, jasvir@gmail.com wrote:
> Reviewers: johnfargo,
> 
> Description:
> * removes duplicate run dependencies
> 
> 
> Please review this at http://codereview.appspot.com/4317045/
> 
> Affected files:
>    M     pom.xml
> 
> 
> Index: pom.xml
> ===================================================================
> --- pom.xml	(revision 1085592)
> +++ pom.xml	(working copy)
> @@ -505,23 +505,6 @@
>       <profile>
>         <id>run</id>
>         <dependencies>
> -        <dependency>
> -          <groupId>org.apache.shindig</groupId>
> -          <artifactId>shindig-gadgets</artifactId>
> -          <version>${project.version}</version>
> -        </dependency>
> -
> -        <dependency>
> -          <groupId>org.apache.shindig</groupId>
> -          <artifactId>shindig-social-api</artifactId>
> -          <version>${project.version}</version>
> -        </dependency>
> -
> -        <dependency>
> -          <groupId>org.apache.shindig</groupId>
> -          <artifactId>shindig-extras</artifactId>
> -          <version>${project.version}</version>
> -        </dependency>
>         </dependencies>
> 
>         <build>
> 
> 

-- 
Thorsten Scherler <thorsten.at.apache.org>
codeBusters S.L. - web based systems
<consulting, training and solutions>
http://www.codebusters.es/
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b