This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Joel Brobecker <brobecker at adacore dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Binutils <binutils at sourceware dot org>, GDB <gdb-patches at sourceware dot org>
- Date: Wed, 18 Feb 2015 04:56:26 -0800
- Subject: Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4
- Authentication-results: sourceware.org; auth=none
- References: <20150107144548 dot GX5432 at adacore dot com> <20150218120841 dot GD23529 at adacore dot com>
On Wed, Feb 18, 2015 at 4:08 AM, Joel Brobecker <brobecker@adacore.com> wrote:
> Yay? Nay?
>
> Thank you.
>
> On Wed, Jan 07, 2015 at 06:45:48PM +0400, Joel Brobecker wrote:
>> Hello,
>>
>> This patch enhances config/zlib.m4 to introduce an extra option
>> --with-libz-prefix which allows us to provide the location of
>> the zlib library we want to use during the build.
>>
>> config/ChangeLog:
>>
>> * zlib.m4 (AM_ZLIB): Add --with-libz-prefix option support.
>>
>> I didn't see any file in the GCC project that uses this macro,
>> so for the GCC repository, the change to zlib.m4 is it. But
>> I am also attaching to this email a copy of the patch that
>> will be applied to the binutils-gdb.git repository, with all
>> configury using this macro being re-generated - mostly for info,
>> also as a heads-up to both binutils and GDB.
>>
>> This was tested by regenerating all autoconf/automake files in
>> the binutils-gdb project, and rebuilding GDB, using the following
>> combinations:
>>
>> --with-zlib (system zlib used)
>> --with-libz-prefix=/zlib/prefix (specific zlib linked in)
>> --with-zlib --with-libz-prefix=/zlib/prefix (specific zlib linked in)
>>
>> --without-zlib (zlib support turned off)
>> --without-zlib --with-zlib-prefix (zlib support turned off)
>>
>> --with-zlib (no system zlib available, configure fails with expected error)
>> --with-zlib --with-libz-prefix=/invalid/zlib/prefix
>> (no system zlib, configure fails with same error)
>>
>> OK to commit?
Why do you want to turn off zlib? On Linux/x86, zlib is required
for assembler. At least, you should issue an error when --without-libz
is used in binutils for Linux/x86 target.
I guess someone has asked it before. Why can't zlib be made the
same as
--with-mpc=PATH specify prefix directory for installed MPC package.
Equivalent to --with-mpc-include=PATH/include plus
--with-mpc-lib=PATH/lib
--with-mpc-include=PATH specify directory for installed MPC include files
--with-mpc-lib=PATH specify directory for the installed MPC library
It is more flexible than your patch. If you have some existing packages
which use your scheme, you can translate the configure command line
options to this one.
--
H.J.