working on a fix for ticket #212

Laszlo Nagy rizsotto at gmail.com
Mon Jun 15 19:44:22 EDT 2009


hi Duncan,

>> My findings:
>>  - The System.Platform did not passed parse tests. The reason: both
>> System.OS and System.Arch allowed the dash character in the name. But
>> System.Platform text representation was concatenate the two strings
>> separated by a dash. So I just simple removed the dash from them.
>> Solved! ;)
>
> Great. Could that be made into a separate patch from the addition of the
> tests? The commit message can say exactly what you just said in
> explanation of the change.
>

  http://sites.google.com/site/rizsotto/patches/cabal-system.platform-fix.darcs-send

This is a one line fix for the mentioned problem. Separated from the test cases.

Regards,
Laszlo



More information about the cabal-devel mailing list