* [Buildroot] [PATCH] e2fsprogs: fix static build against external libblkid
@ 2014-06-11 10:15 Baruch Siach
2014-06-11 10:52 ` Peter Korsgaard
0 siblings, 1 reply; 4+ messages in thread
From: Baruch Siach @ 2014-06-11 10:15 UTC (permalink / raw)
To: buildroot
Fixes:
http://autobuild.buildroot.net/results/0159246b52659508a4bbba4126ab27b28a2b2e2c/
Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
...n-fix-external-libblkid-test-for-static-l.patch | 34 ++++++++++++++++++++++
1 file changed, 34 insertions(+)
create mode 100644 package/e2fsprogs/e2fsprogs-0004-configure.in-fix-external-libblkid-test-for-static-l.patch
diff --git a/package/e2fsprogs/e2fsprogs-0004-configure.in-fix-external-libblkid-test-for-static-l.patch b/package/e2fsprogs/e2fsprogs-0004-configure.in-fix-external-libblkid-test-for-static-l.patch
new file mode 100644
index 000000000000..65d6c4c024bc
--- /dev/null
+++ b/package/e2fsprogs/e2fsprogs-0004-configure.in-fix-external-libblkid-test-for-static-l.patch
@@ -0,0 +1,34 @@
+From b52e76432fe23490b891884f3a66504101c2a5bd Mon Sep 17 00:00:00 2001
+Message-Id: <b52e76432fe23490b891884f3a66504101c2a5bd.1402480818.git.baruch@tkos.co.il>
+From: Baruch Siach <baruch@tkos.co.il>
+Date: Wed, 11 Jun 2014 12:56:15 +0300
+Subject: [PATCH] configure.in: fix external libblkid test for static link
+
+External libblkid needs -luuid when linking statically. Use pkg-config to
+determine it correctly. Adding --static doesn't hurt even for dynamic
+linking.
+
+Signed-off-by: Baruch Siach <baruch@tkos.co.il>
+---
+
+Patch status: sent upstream: http://patchwork.ozlabs.org/patch/358584/
+
+ configure.in | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/configure.in b/configure.in
+index 448a2926776a..ec6068320878 100644
+--- a/configure.in
++++ b/configure.in
+@@ -551,7 +551,7 @@ then
+ [LIBBLKID=`$PKG_CONFIG --libs blkid`;
+ STATIC_LIBBLKID=`$PKG_CONFIG --static --libs blkid`],
+ [AC_MSG_ERROR([external blkid library not found])],
+- [$LIBBLKID])
++ [`$PKG_CONFIG --static --libs blkid`])
+ BLKID_CMT=#
+ AC_MSG_RESULT([Disabling private blkid library])
+ else
+--
+2.0.0
+
--
2.0.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] e2fsprogs: fix static build against external libblkid
2014-06-11 10:15 [Buildroot] [PATCH] e2fsprogs: fix static build against external libblkid Baruch Siach
@ 2014-06-11 10:52 ` Peter Korsgaard
2014-06-11 12:31 ` Baruch Siach
0 siblings, 1 reply; 4+ messages in thread
From: Peter Korsgaard @ 2014-06-11 10:52 UTC (permalink / raw)
To: buildroot
>>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes:
> Fixes:
> http://autobuild.buildroot.net/results/0159246b52659508a4bbba4126ab27b28a2b2e2c/
> Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> ---
> ...n-fix-external-libblkid-test-for-static-l.patch | 34 ++++++++++++++++++++++
> 1 file changed, 34 insertions(+)
> create mode 100644 package/e2fsprogs/e2fsprogs-0004-configure.in-fix-external-libblkid-test-for-static-l.patch
> diff --git a/package/e2fsprogs/e2fsprogs-0004-configure.in-fix-external-libblkid-test-for-static-l.patch b/package/e2fsprogs/e2fsprogs-0004-configure.in-fix-external-libblkid-test-for-static-l.patch
> new file mode 100644
> index 000000000000..65d6c4c024bc
> --- /dev/null
> +++ b/package/e2fsprogs/e2fsprogs-0004-configure.in-fix-external-libblkid-test-for-static-l.patch
> @@ -0,0 +1,34 @@
> +From b52e76432fe23490b891884f3a66504101c2a5bd Mon Sep 17 00:00:00 2001
> +Message-Id: <b52e76432fe23490b891884f3a66504101c2a5bd.1402480818.git.baruch@tkos.co.il>
> +From: Baruch Siach <baruch@tkos.co.il>
> +Date: Wed, 11 Jun 2014 12:56:15 +0300
> +Subject: [PATCH] configure.in: fix external libblkid test for static link
> +
> +External libblkid needs -luuid when linking statically. Use pkg-config to
> +determine it correctly. Adding --static doesn't hurt even for dynamic
> +linking.
> +
> +Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> +---
> +
> +Patch status: sent upstream: http://patchwork.ozlabs.org/patch/358584/
> +
> + configure.in | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/configure.in b/configure.in
> +index 448a2926776a..ec6068320878 100644
> +--- a/configure.in
> ++++ b/configure.in
> +@@ -551,7 +551,7 @@ then
> + [LIBBLKID=`$PKG_CONFIG --libs blkid`;
> + STATIC_LIBBLKID=`$PKG_CONFIG --static --libs blkid`],
> + [AC_MSG_ERROR([external blkid library not found])],
> +- [$LIBBLKID])
> ++ [`$PKG_CONFIG --static --libs blkid`])
Why not '[$STATIC_LIBBLKID])' instead of rerunning pkg-config?
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] e2fsprogs: fix static build against external libblkid
2014-06-11 10:52 ` Peter Korsgaard
@ 2014-06-11 12:31 ` Baruch Siach
2014-06-11 12:53 ` Peter Korsgaard
0 siblings, 1 reply; 4+ messages in thread
From: Baruch Siach @ 2014-06-11 12:31 UTC (permalink / raw)
To: buildroot
Hi Peter,
On Wed, Jun 11, 2014 at 12:52:00PM +0200, Peter Korsgaard wrote:
> >>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes:
>
> > Fixes:
> > http://autobuild.buildroot.net/results/0159246b52659508a4bbba4126ab27b28a2b2e2c/
>
> > Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> > ---
> > ...n-fix-external-libblkid-test-for-static-l.patch | 34 ++++++++++++++++++++++
> > 1 file changed, 34 insertions(+)
> > create mode 100644 package/e2fsprogs/e2fsprogs-0004-configure.in-fix-external-libblkid-test-for-static-l.patch
>
> > diff --git a/package/e2fsprogs/e2fsprogs-0004-configure.in-fix-external-libblkid-test-for-static-l.patch b/package/e2fsprogs/e2fsprogs-0004-configure.in-fix-external-libblkid-test-for-static-l.patch
> > new file mode 100644
> > index 000000000000..65d6c4c024bc
> > --- /dev/null
> > +++ b/package/e2fsprogs/e2fsprogs-0004-configure.in-fix-external-libblkid-test-for-static-l.patch
> > @@ -0,0 +1,34 @@
> > +From b52e76432fe23490b891884f3a66504101c2a5bd Mon Sep 17 00:00:00 2001
> > +Message-Id: <b52e76432fe23490b891884f3a66504101c2a5bd.1402480818.git.baruch@tkos.co.il>
> > +From: Baruch Siach <baruch@tkos.co.il>
> > +Date: Wed, 11 Jun 2014 12:56:15 +0300
> > +Subject: [PATCH] configure.in: fix external libblkid test for static link
> > +
> > +External libblkid needs -luuid when linking statically. Use pkg-config to
> > +determine it correctly. Adding --static doesn't hurt even for dynamic
> > +linking.
> > +
> > +Signed-off-by: Baruch Siach <baruch@tkos.co.il>
> > +---
> > +
> > +Patch status: sent upstream: http://patchwork.ozlabs.org/patch/358584/
> > +
> > + configure.in | 2 +-
> > + 1 file changed, 1 insertion(+), 1 deletion(-)
> > +
> > +diff --git a/configure.in b/configure.in
> > +index 448a2926776a..ec6068320878 100644
> > +--- a/configure.in
> > ++++ b/configure.in
> > +@@ -551,7 +551,7 @@ then
> > + [LIBBLKID=`$PKG_CONFIG --libs blkid`;
> > + STATIC_LIBBLKID=`$PKG_CONFIG --static --libs blkid`],
> > + [AC_MSG_ERROR([external blkid library not found])],
> > +- [$LIBBLKID])
> > ++ [`$PKG_CONFIG --static --libs blkid`])
>
> Why not '[$STATIC_LIBBLKID])' instead of rerunning pkg-config?
Quoting the autoconf manual:
AC_CHECK_LIB (LIBRARY, FUNCTION, [ACTION-IF-FOUND],
[ACTION-IF-NOT-FOUND], [OTHER-LIBRARIES])
So, if I understand correctly, $STATIC_LIBBLKID in [ACTION-IF-FOUND] gets
defined only after the test has run successfully.
baruch
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] e2fsprogs: fix static build against external libblkid
2014-06-11 12:31 ` Baruch Siach
@ 2014-06-11 12:53 ` Peter Korsgaard
0 siblings, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2014-06-11 12:53 UTC (permalink / raw)
To: buildroot
>>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes:
Hi,
>> > +Patch status: sent upstream: http://patchwork.ozlabs.org/patch/358584/
>> > +
>> > + configure.in | 2 +-
>> > + 1 file changed, 1 insertion(+), 1 deletion(-)
>> > +
>> > +diff --git a/configure.in b/configure.in
>> > +index 448a2926776a..ec6068320878 100644
>> > +--- a/configure.in
>> > ++++ b/configure.in
>> > +@@ -551,7 +551,7 @@ then
>> > + [LIBBLKID=`$PKG_CONFIG --libs blkid`;
>> > + STATIC_LIBBLKID=`$PKG_CONFIG --static --libs blkid`],
>> > + [AC_MSG_ERROR([external blkid library not found])],
>> > +- [$LIBBLKID])
>> > ++ [`$PKG_CONFIG --static --libs blkid`])
>>
>> Why not '[$STATIC_LIBBLKID])' instead of rerunning pkg-config?
> Quoting the autoconf manual:
> AC_CHECK_LIB (LIBRARY, FUNCTION, [ACTION-IF-FOUND],
> [ACTION-IF-NOT-FOUND], [OTHER-LIBRARIES])
> So, if I understand correctly, $STATIC_LIBBLKID in [ACTION-IF-FOUND] gets
> defined only after the test has run successfully.
Sorry, you're right - I misread the patch. It should arguably still pass
$LIBBLKID so the user can pass extra libraries, but looking at the
configure script I see it clears it just above, so it cannot be used
anyway.
Committed, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-06-11 12:53 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-06-11 10:15 [Buildroot] [PATCH] e2fsprogs: fix static build against external libblkid Baruch Siach
2014-06-11 10:52 ` Peter Korsgaard
2014-06-11 12:31 ` Baruch Siach
2014-06-11 12:53 ` Peter Korsgaard
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox