reduce-algebra portfile dev

[Mark Brethen]

On Aug 20, 2011, at 6:34 PM, Ryan Schmidt wrote:

> On Aug 20, 2011, at 18:30, Mark Brethen wrote:
> 
>> On Aug 20, 2011, at 3:08 PM, Ryan Schmidt wrote:
>> 
>>> A relevant line in the log was:
>>> 
>>> :info:destroot make: `install' is up to date.
>>> 
>>> This means the project's Makefile is not written properly. They assume they are running on a case-sensitive filesystem, which most Macs aren't. It's a conflict between the virtual rule "install" (as in "make install") and the real file "INSTALL" which provides installation instructions.
>>> 
>> I looked at the "INSTALL" file and all it contains is this one line:
>> 
>> See the file BUILDING for commentary about configuring and building
>> this code.
> 
> Right. They can put whatever they want in the INSTALL file but its mere presence is interfering with the virtual "install" rule in the Makefile, on case-insensitive filesystems.
> 
> 
>> There is another file in the directory named "install-sh"
> 
> Unrelated.
> 
> 
>>> The best solution is to fix the Makefile and provide the patch back to the developers. Locate the ".PHONY" line in the Makefile and add "install" to it. If there is no .PHONY line, add one:
>>> 
>>> .PHONY install
>>> 
>>> An alternate solution is to just delete the INSTALL file, for example in the post-extract phase.
>> 
>> The first line in Makefile:
>> 
>> .PHONY: dist all csl psl
>> 
>> Patch this line to add "install-sh"?
> 
> No, add "install".
> 
> 
I thought Makefile is created from Makefile.am and Makefile.in, but in the Makefile it says it is a hand-written top-level Makefile. Do I patch all 3 makefiles?


Mark