[Cluster-devel] [PATCH] Clean up top level configure part 1

[Cluster-devel] [PATCH] Clean up top level configure part 1

[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:

- 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

-- 
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>

[Cluster-devel] [PATCH] Clean up top level configure part 1

[Patrick Caulfield]

Fabio Massimo Di Nitto wrote:
> 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.
>

You missed one:

-- 
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>