#8 cygwin drive prefix processing

closed-accepted
nobody
None
5
2012-04-08
2009-01-06
No

added 'mount -p' call to support platform cygwin drive prefix resolution (cygwin default's to '/cygdrive').

duplicated the code from ‘CygwinPathMapper.java’ to create CygwinMount.java to run required 'mount command' – might be better to integrate with path mapper or other service entity… Changed PerlExecutableUtilities.java to use platform mount prefix…

Discussion

  • Jan Ploski

    Jan Ploski - 2009-02-07

    I committed your code to CVS with slight modifications. In particular, I put the class CygwinMount in the same package as PerlExecutableUtilities to avoid creating a dependency from epic.perleditor on epic.debug (no references exist so far in that direction).

    This is going to be released in 0.6.31. It would be appreciated if you could test the CVS version beforehand.

     
  • Nobody/Anonymous

    I recommend not to require adding cygwin's executables to the Windows PATH. The reason is that then cygwin1.dll is in the path which has caused problems with other open source release in the past (DLL hell).
    I'd rather recommend to extract the path to mount from the path to the cygwin perl executable. This solution does not require an additional preference field.
    Alternative to this solution would be to add another preference field or require setting a different environment variable CYGWIN_HOME which points to the cygwin bin directory.

    I hope this can be fixed in a future release.

     
  • yarrt

    yarrt - 2010-01-26

    last comment was mine.

     
  • Jan Ploski

    Jan Ploski - 2012-04-08
    • status: open --> closed-accepted
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks