
15 Jun
2009
15 Jun
'09
11:44 p.m.
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.darc... This is a one line fix for the mentioned problem. Separated from the test cases. Regards, Laszlo