[PATCH] zlib: substitute sharedlibdir

[PATCH] zlib: substitute sharedlibdir

[Enrico Scholz]

From: Enrico Scholz <enrico.scholz@informatik.tu-chemnitz.de>

@sharedlibdir@ is not substituted so that linkerflags in zlib.pc expand to
"... -L@sharedlibdir@" which breaks e.g. glib builds.

Signed-off-by: Enrico Scholz <enrico.scholz@sigma-chemnitz.de>
---
 recipes/zlib/zlib-1.2.5/0001-autotools.patch |    4 +++-
 recipes/zlib/zlib_1.2.5.bb                   |    2 +-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/recipes/zlib/zlib-1.2.5/0001-autotools.patch b/recipes/zlib/zlib-1.2.5/0001-autotools.patch
index 6642860..f5c50f1 100644
--- a/recipes/zlib/zlib-1.2.5/0001-autotools.patch
+++ b/recipes/zlib/zlib-1.2.5/0001-autotools.patch
@@ -33,7 +33,7 @@ new file mode 100644
 index 0000000..177c89a
 --- /dev/null
 +++ b/configure.ac
-@@ -0,0 +1,17 @@
+@@ -0,0 +1,19 @@
 +AC_INIT(zlib,1.2.5)
 +AC_CONFIG_SRCDIR(adler32.c)
 +AM_INIT_AUTOMAKE(zlibs,1.2.5)
@@ -45,6 +45,8 @@ index 0000000..177c89a
 +
 +AC_HEADER_STDC
 +
++AC_SUBST(sharedlibdir,$libdir)
++
 +AC_CONFIG_FILES([
 +Makefile
 +zlib.pc
diff --git a/recipes/zlib/zlib_1.2.5.bb b/recipes/zlib/zlib_1.2.5.bb
index 34419f0..1f51404 100644
--- a/recipes/zlib/zlib_1.2.5.bb
+++ b/recipes/zlib/zlib_1.2.5.bb
@@ -1,6 +1,6 @@
 include zlib.inc
 
-PR = "${INC_PR}.0"
+PR = "${INC_PR}.1"
 
 SRC_URI += "file://0001-autotools.patch "
 
-- 
1.7.3.4

Re: [PATCH] zlib: substitute sharedlibdir

[Khem Raj]

On 1/13/2011 6:29 AM, Enrico Scholz wrote:
> From: Enrico Scholz<enrico.scholz@informatik.tu-chemnitz.de>
>
> @sharedlibdir@ is not substituted so that linkerflags in zlib.pc expand to
> "... -L@sharedlibdir@" which breaks e.g. glib builds.
>


I think zlib has --sharedlibdir configure option since 1.2.4.4+
so you could try to use it instead of patching

> Signed-off-by: Enrico Scholz<enrico.scholz@sigma-chemnitz.de>
> ---
>   recipes/zlib/zlib-1.2.5/0001-autotools.patch |    4 +++-
>   recipes/zlib/zlib_1.2.5.bb                   |    2 +-
>   2 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/recipes/zlib/zlib-1.2.5/0001-autotools.patch b/recipes/zlib/zlib-1.2.5/0001-autotools.patch
> index 6642860..f5c50f1 100644
> --- a/recipes/zlib/zlib-1.2.5/0001-autotools.patch
> +++ b/recipes/zlib/zlib-1.2.5/0001-autotools.patch
> @@ -33,7 +33,7 @@ new file mode 100644
>   index 0000000..177c89a
>   --- /dev/null
>   +++ b/configure.ac
> -@@ -0,0 +1,17 @@
> +@@ -0,0 +1,19 @@
>   +AC_INIT(zlib,1.2.5)
>   +AC_CONFIG_SRCDIR(adler32.c)
>   +AM_INIT_AUTOMAKE(zlibs,1.2.5)
> @@ -45,6 +45,8 @@ index 0000000..177c89a
>   +
>   +AC_HEADER_STDC
>   +
> ++AC_SUBST(sharedlibdir,$libdir)
> ++
>   +AC_CONFIG_FILES([
>   +Makefile
>   +zlib.pc
> diff --git a/recipes/zlib/zlib_1.2.5.bb b/recipes/zlib/zlib_1.2.5.bb
> index 34419f0..1f51404 100644
> --- a/recipes/zlib/zlib_1.2.5.bb
> +++ b/recipes/zlib/zlib_1.2.5.bb
> @@ -1,6 +1,6 @@
>   include zlib.inc
>
> -PR = "${INC_PR}.0"
> +PR = "${INC_PR}.1"
>
>   SRC_URI += "file://0001-autotools.patch "
>

Re: [PATCH] zlib: substitute sharedlibdir

[Enrico Scholz]

Khem Raj <raj.khem@gmail.com> writes:

>> @sharedlibdir@ is not substituted so that linkerflags in zlib.pc expand to
>> "... -L@sharedlibdir@" which breaks e.g. glib builds.
>
> I think zlib has --sharedlibdir configure option since 1.2.4.4+
> so you could try to use it instead of patching

afais, the ./configure will be completely replaced due to the
0001-autotools.patch (added by a2393dbf941554098).  So there exists no
such option anymore.


>> --- a/recipes/zlib/zlib-1.2.5/0001-autotools.patch
>> +++ b/recipes/zlib/zlib-1.2.5/0001-autotools.patch
>> @@ -33,7 +33,7 @@ new file mode 100644
>>   index 0000000..177c89a
>>   --- /dev/null
>>   +++ b/configure.ac


Enrico

Re: [PATCH] zlib: substitute sharedlibdir

[Khem Raj]

On Fri, Jan 14, 2011 at 1:55 AM, Enrico Scholz
<enrico.scholz@sigma-chemnitz.de> wrote:
> Khem Raj <raj.khem@gmail.com> writes:
>
>>> @sharedlibdir@ is not substituted so that linkerflags in zlib.pc expand to
>>> "... -L@sharedlibdir@" which breaks e.g. glib builds.
>>
>> I think zlib has --sharedlibdir configure option since 1.2.4.4+
>> so you could try to use it instead of patching
>
> afais, the ./configure will be completely replaced due to the
> 0001-autotools.patch (added by a2393dbf941554098).  So there exists no
> such option anymore.

now that I look into OE we have autoconf'ed it ourselves. Originally
it has a configure
but not all configure scripts are generated via autoconf so We kind of
swerved away
from zlib's build system has anyone proposed it to zlib upstream that
they autconf
their build system ? otherwise I think it would be better to use the
configure script as is

>
>
>>> --- a/recipes/zlib/zlib-1.2.5/0001-autotools.patch
>>> +++ b/recipes/zlib/zlib-1.2.5/0001-autotools.patch
>>> @@ -33,7 +33,7 @@ new file mode 100644
>>>   index 0000000..177c89a
>>>   --- /dev/null
>>>   +++ b/configure.ac
>
>
> Enrico
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>