[
https://issues.apache.org/jira/browse/SCM-191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17964370#comment-17964370
]
Olivier Lamy commented on SCM-191:
----------------------------------
This project has moved from Jira to GitHub Issues. This issue was migrated to
[apache/maven-scm#430|https://github.com/apache/maven-scm/issues/430].
> Difficult to disable cvs compress mode using system property
> -Dmaven.scm.cvs.use_compression=false and cvs-settings.xml property file
> -------------------------------------------------------------------------------------------------------------------------------------
>
> Key: SCM-191
> URL: https://issues.apache.org/jira/browse/SCM-191
> Project: Maven SCM (Moved to GitHub Issues)
> Issue Type: Bug
> Components: maven-scm-provider-cvs
> Affects Versions: 1.0-beta-3
> Reporter: Thibaut CAZENAVE
> Assignee: Emmanuel Venisse
> Priority: Minor
> Fix For: 1.0-beta-4
>
>
> 1st bug :
> The system property -Dmaven.scm.cvs.use_compression=false doesn't work if
> getSettings() method of CvsUtil class doesn't find the cvs-settings.xml file.
> In this case, this method return a new Settings() for which
> compressionLevel=3 (constructor of Settings class).
> In method getBaseCommand() of CvsCommandUtils class
> (org.apache.maven.scm.provider.cvslib.command), since
> settings.getCompressionLevel() = 3, if condition is true and then it adds
> -z3 to CommandLine cl.
> 2nd bug :
> If you set <compressionLevel>0</compressionLevel> in cvs-settings.xml without
> setting system property -Dmaven.scm.cvs.use_compression=false, -z3 cvs
> compression is used.
> In method getBaseCommand(), since settings.getCompressionLevel() > 0, if
> condition is false and then we go in else if condition (!System.getProperty(
> "maven.scm.cvs.use_compression", "true" ).equals( "false" )) which happens to
> be true so it adds -z3 to CommandLine cl.
> Patch proposed :
> Replace in CvsCommandUtils getBaseCommand() method :
> ---------------------
> if ( settings.getCompressionLevel() > 0 )
> {
> cl.createArgument().setValue( "-z" +
> settings.getCompressionLevel() );
> }
> else if ( !System.getProperty( "maven.scm.cvs.use_compression",
> "true" ).equals( "false" ) )
> {
> cl.createArgument().setValue( "-z3" );
> }
> ---------------------
> with :
> ---------------------
> if (settings.getCompressionLevel() > 0 &&
> System.getProperty("maven.scm.cvs.use_compression", "true").equals("true"))
> cl.createArgument().setValue("-z" + settings.getCompressionLevel());
> --------------------
> Tibo
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
