Hi,

Ovidiu Feodorov wrote at Dienstag, 30. Juni 2009 04:55:

[snip]

> It seems quite obvious that this could be handled by cygwin svn

No. Cygwin provides a POSIX layer for the file operations to allow unix
utilities to compile and work in the Cygwin environment. Although some
basic commands in Cygwin understand DOSish syntax, its up to the individual
maintainer to patch that into the code and maintain this patch. Obviously
this is no fun and normally not done.

> , but I 
> believe it could also be worked around from the Maven layer, with
> minimum of effort, especially that Maven SCM already has Cygwin-related
> configuration options. If Maven would generate maven-scm-xxxxx-targets
> to contain cygwin paths Windows paths, that would quell svn.
> 
> The file list in maven-scm-xxxxx-targets is generated by
>
org.apache.maven.scm.provider.svn.svnexe.command.SvnCommandLineUtils.addTargets()
>
(https://svn.apache.org/repos/asf/maven/scm/tags/maven-scm-1.2/maven-scm-providers/maven-scm-providers-svn/maven-scm-provider-svnexe/src/main/java/org/apache/maven/scm/provider/svn/svnexe/command/SvnCommandLineUtils.java).
> 
> SvnCommandLineUtils.addTargets() could take into consideration
> Cygwin-related  <svn-settings> (<useCygwinPath> and <cygwinMountPath>)
> and build the paths accordingly. <useCygwinPath> and <cygwinMountPath>
> are already there, and this was probably the hardest part  (see
> http://maven.apache.org/scm/subversion.html).
> 
> If you think this is a sensible solution that'll make the life of the
> developers easier, and are willing to apply it, I'll submit the patch.
>
> Or, it is also quite possible that I am totally missing something
> obvious here, and I will be happy to be guided to the right path.

IMHO the flag useCygwinPath should be enough. The implementation can
determin the paths itself by executing "cygpath -u path" for each. The
cygwinMountPath option is not a good choice, since it depends on the
developer's environment and might not work at all if the path is on the
network.

- Jörg


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to