[darcs-devel] [patch716] Replace System.FilePath.Posix with Syste... (and 1 more)

Eric Kow bugs at darcs.net
Fri Feb 3 19:12:31 UTC 2012


Eric Kow <kowey at darcs.net> added the comment:

Careful, there's a history to this one.  The choice of 
System.FilePath.Posix was to ensure a bit of bug-avoidance.  Before, we 
were using darcs-grown functions.  I wanted to replace with 
System.FilePath but we got a bit nervous about the implications.

The problem is that darcs uses posix-style paths internally regardless 
of the platform.  So the right thing to do is to make sure we use an 
abstract paths representation internally that is based on path 
components (Petr had been working on this, I forgot how far we got), and 
then we don't have to worry.  But so long as we're throwing strings 
around, we're best off going posix-everywhere (to avoid the potential 
bug that arises when a backslash creeps in there somewhere)

__________________________________
Darcs bug tracker <bugs at darcs.net>
<http://bugs.darcs.net/patch716>
__________________________________


More information about the darcs-devel mailing list