You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm-commits@maven.apache.org by ev...@apache.org on 2006/04/06 15:09:30 UTC
svn commit: r391976 -
/maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/changelog/SvnChangeLogCommand.java
Author: evenisse
Date: Thu Apr 6 06:09:28 2006
New Revision: 391976
URL: http://svn.apache.org/viewcvs?rev=391976&view=rev
Log:
o Remove possible NPE in command line execution
o Remove unnecessary timezone
Modified:
maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/changelog/SvnChangeLogCommand.java
Modified: maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/changelog/SvnChangeLogCommand.java
URL: http://svn.apache.org/viewcvs/maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/changelog/SvnChangeLogCommand.java?rev=391976&r1=391975&r2=391976&view=diff
==============================================================================
--- maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/changelog/SvnChangeLogCommand.java (original)
+++ maven/scm/trunk/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/changelog/SvnChangeLogCommand.java Thu Apr 6 06:09:28 2006
@@ -26,6 +26,7 @@
import org.apache.maven.scm.provider.svn.command.SvnCommand;
import org.apache.maven.scm.provider.svn.repository.SvnScmProviderRepository;
import org.apache.maven.scm.provider.svn.svnexe.command.SvnCommandLineUtils;
+import org.codehaus.plexus.util.StringUtils;
import org.codehaus.plexus.util.cli.CommandLineException;
import org.codehaus.plexus.util.cli.CommandLineUtils;
import org.codehaus.plexus.util.cli.Commandline;
@@ -89,8 +90,6 @@
{
SimpleDateFormat dateFormat = new SimpleDateFormat( DATE_FORMAT );
- dateFormat.setTimeZone( TimeZone.getTimeZone( "GMT" ) );
-
Commandline cl = SvnCommandLineUtils.getBaseSvnCommandLine( workingDirectory, repository );
cl.createArgument().setValue( "log" );
@@ -114,7 +113,7 @@
}
}
- if ( branch != null )
+ if ( StringUtils.isNotEmpty( branch ) )
{
// By specifying a branch and this repository url below, subversion should show
// the changelog of that branch, but limit it to paths that also occur in this repository.