Clean up top level configure part 1
PATCH 1/1 Clean up top level configure part 1
Hi guys,
the patch in attachment is purely cosmetic. I switched the top level configure to use perl -w rather than just perl and spotted a bunch of warnings.
This patch:
- switches permanently to perl -w
- fixes all the warnings in the script.
Part1 because I plan to re-rewrite the top level configure from scratch by using the only known thing in every system (that being /bin/sh) and make path to perl configurable (not all system use /usr/bin/perl).
I also plan to kill the need of make/defines.mk.input and create make/define.mk directly.
use Usual::ACK::Apply::disclaimer
Cheers
Fabio
Signature · 5 lines
I'm going to make him an offer he can't refuse. -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: configure.diff URL: <http://listman.redhat.com/archives/cluster-devel/attachments/20070922/3ecdbb81/attachment.ksh>
PATCH 1/1 Clean up top level configure part 1
Quoted 7 lines · Fabio Massimo Di Nitto
Hi guys,
the patch in attachment is purely cosmetic. I switched the top level configure to use perl -w rather than just perl and spotted a bunch of warnings.
This patch:
Quoted 9 lines
- switches permanently to perl -w
- fixes all the warnings in the script.Part1 because I plan to re-rewrite the top level configure from scratch by using the only known thing in every system (that being
/bin/sh) and make path to perl configurable (not all system use/usr/bin/perl).
I also plan to kill the need of make/defines.mk.input and createmake/define.mkdirectly.
You missed one:
Signature · 8 lines
Patrick -------------- next part -------------- A non-text attachment was scrubbed... Name: configure1.diff Type: text/x-patch Size: 2269 bytes Desc: not available URL: <http://listman.redhat.com/archives/cluster-devel/attachments/20070924/9d1269d9/attachment.bin>