grab probably shouldn't be case sensitive

Question

grab probably shouldn't be case sensitive

Closed this issue 8 years ago · 5 comments

grab is case sensitive and probably shouldn't be

https://github.com/ndmitchell/neil/blob/master/src/Cabal.hs#L225

Answer 1 · 2016-05-28T08:39:21.000Z

The fact cabal isn't case sensitive is a massive misfeature. Think of a good language. It should have a canonical form. Given it uses kebab casing, lower case is the only thing that makes sense. Part of the idea of grab is to enforce that.

Answer 2 · 2016-05-28T08:43:39.000Z

That's reasonable.

Answer 3 · 2016-05-28T08:45:45.000Z

Do I not have a check somewhere that all cabal fields are lowercase? If I don't I should. If I do, perhaps it wants moving before the first grab.

Answer 4 · 2016-05-28T08:48:59.000Z

That check is "Incorrect declaration style" and happens at https://github.com/ndmitchell/neil/blob/master/src/Cabal.hs#L245, after grab.

Answer 5 · 2016-05-29T08:54:11.000Z

Fixed as part of #27.