[64959] trunk/dports/devel/hs-NewBinary/Portfile

[Ryan Schmidt]

On Mar 19, 2010, at 08:22, Gregory Wright wrote:
> On 3/19/10 9:00 AM, Ryan Schmidt wrote:
>> 
>> The lint message in question reads:
>> 
>> Error: Portfile directory hs-NewBinary does not match port name hs-newbinary
>> 
>> The haskell portgroup deliberately lowercases the supplied haskell module name:
>> 
>>     name                hs-[string tolower ${package}]
>> 
>> So why are you defeating that in the ports you've been adding? If we want lowercase portnames, as this convenience in the portgroup suggests, you should not be defeating it in your ports. OTOH if we want CamelCase portnames, then the lowercasing convenience should be removed from the portgroup.
> 
> Here's the problem: the hackage package distribution site lowercases distfiles,
> so the distfile of the HTTP port is http-<version>.tar.gz.  However, they allow
> mixed case package names.  So for the port name to agree with the common name, we
> need mixed case.  The fix would seem to be to lowercase only the distfile name.

When Brian designed the haskell portgroup, he appears to have intended for the portname to be lowercased, presumably to match how we do perl module ports. (Perl module names are mixed-case, but the corresponding portnames are lowercased.)

You are now adding ports with mixed-case names.

We can do it either way, I'd just like us to be consistent, and for the portgroup to encode that behavior.