Ryan,

>> Modified: trunk/dports/science/ncarg/Portfile
> 
>> +    reinplace "s|int_p_NULL|(int*)NULL|g" 
>> ${worksrcpath}/ncarg2d/src/libncarg/ezmap/mapngb.c
> 
> This is exactly the kind of change that would be more appropriately made as a 
> patchfile, not a reinplace.

Thank you for a good suggestion. I had thought that the fewer patches the 
better.
It is difficult for the upstream developer to read Portfile but diffs.

> It is better to write commit messages that say what you did, not just what 
> ticket to look at for details.

Yes, you are right. It is easier to read the commit comment  in the 
command-line than finding corresponding Trac tickets.

Takeshi
_______________________________________________
macports-dev mailing list
[email protected]
http://lists.macosforge.org/mailman/listinfo.cgi/macports-dev

Reply via email to