* [Buildroot] [PATCH 1/1] package/libglib2: remove bindir prefix from gio-2.0.pc
@ 2019-06-11 17:30 Fabrice Fontaine
2019-06-11 18:19 ` Adam Duskett
0 siblings, 1 reply; 4+ messages in thread
From: Fabrice Fontaine @ 2019-06-11 17:30 UTC (permalink / raw)
To: buildroot
Apply LIBGLIB2_REMOVE_BINDIR_PREFIX_FROM_PC_FILE to gio-2.0.pc so
${bindir} prefix is also removed for gbus-codegen otherwise build of
packages using this binary will fail if gbus-codegen is not installed on
host
Fixes:
- http://autobuild.buildroot.org/results/5c0e4f3d33e01198688d58388aabe159bd005234
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
package/libglib2/libglib2.mk | 16 ++++++++++------
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/package/libglib2/libglib2.mk b/package/libglib2/libglib2.mk
index 78d6e5ab32..c8db706684 100644
--- a/package/libglib2/libglib2.mk
+++ b/package/libglib2/libglib2.mk
@@ -85,13 +85,17 @@ endef
LIBGLIB2_POST_INSTALL_TARGET_HOOKS += LIBGLIB2_REMOVE_DEV_FILES
-# Newer versions of libglib2 prefix glib-genmarshal, gobject-query, and
-# glib-mkenums with ${bindir}. Unfortunately, this will resolve to the host
-# systems /bin/ directory, which will cause compilation issues if the host does
-# not have these programs. By removing the ${bindir}/ prefix, these programs
-# are resolved in PATH instead.
+# Newer versions of libglib2 prefix glib-genmarshal, gobject-query,
+# glib-mkenums, glib_compile_schemas, glib_compile_resources and gdbus-codegen
+# with ${bindir}. Unfortunately, this will resolve to the host systems /bin/
+# directory, which will cause compilation issues if the host does not have these
+# programs. By removing the ${bindir}/ prefix, these programs are resolved in
+# PATH instead.
define LIBGLIB2_REMOVE_BINDIR_PREFIX_FROM_PC_FILE
- $(SED) 's%$${bindir}/%%g' $(STAGING_DIR)/usr/lib/pkgconfig/glib-2.0.pc
+ for file in glib-2.0.pc gio-2.0.pc ; do \
+ $(SED) 's%$${bindir}/%%g' \
+ $(STAGING_DIR)/usr/lib/pkgconfig/$$file ; \
+ done
endef
LIBGLIB2_POST_INSTALL_TARGET_HOOKS += LIBGLIB2_REMOVE_BINDIR_PREFIX_FROM_PC_FILE
--
2.20.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] package/libglib2: remove bindir prefix from gio-2.0.pc
2019-06-11 17:30 [Buildroot] [PATCH 1/1] package/libglib2: remove bindir prefix from gio-2.0.pc Fabrice Fontaine
@ 2019-06-11 18:19 ` Adam Duskett
2019-06-11 19:22 ` Yann E. MORIN
0 siblings, 1 reply; 4+ messages in thread
From: Adam Duskett @ 2019-06-11 18:19 UTC (permalink / raw)
To: buildroot
All;
Openembedded has a patch for this that might be even better for us to use:
https://github.com/openembedded/openembedded-core/blob/master/meta/recipes-core/glib-2.0/glib-2.0/0001-Do-not-write-bindir-into-pkg-config-files.patch
If not, then consider this an
Acked-by: Adam Duskett <aduskett@gmail.com>
On Tue, Jun 11, 2019 at 1:30 PM Fabrice Fontaine
<fontaine.fabrice@gmail.com> wrote:
>
> Apply LIBGLIB2_REMOVE_BINDIR_PREFIX_FROM_PC_FILE to gio-2.0.pc so
> ${bindir} prefix is also removed for gbus-codegen otherwise build of
> packages using this binary will fail if gbus-codegen is not installed on
> host
>
> Fixes:
> - http://autobuild.buildroot.org/results/5c0e4f3d33e01198688d58388aabe159bd005234
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
> package/libglib2/libglib2.mk | 16 ++++++++++------
> 1 file changed, 10 insertions(+), 6 deletions(-)
>
> diff --git a/package/libglib2/libglib2.mk b/package/libglib2/libglib2.mk
> index 78d6e5ab32..c8db706684 100644
> --- a/package/libglib2/libglib2.mk
> +++ b/package/libglib2/libglib2.mk
> @@ -85,13 +85,17 @@ endef
>
> LIBGLIB2_POST_INSTALL_TARGET_HOOKS += LIBGLIB2_REMOVE_DEV_FILES
>
> -# Newer versions of libglib2 prefix glib-genmarshal, gobject-query, and
> -# glib-mkenums with ${bindir}. Unfortunately, this will resolve to the host
> -# systems /bin/ directory, which will cause compilation issues if the host does
> -# not have these programs. By removing the ${bindir}/ prefix, these programs
> -# are resolved in PATH instead.
> +# Newer versions of libglib2 prefix glib-genmarshal, gobject-query,
> +# glib-mkenums, glib_compile_schemas, glib_compile_resources and gdbus-codegen
> +# with ${bindir}. Unfortunately, this will resolve to the host systems /bin/
> +# directory, which will cause compilation issues if the host does not have these
> +# programs. By removing the ${bindir}/ prefix, these programs are resolved in
> +# PATH instead.
> define LIBGLIB2_REMOVE_BINDIR_PREFIX_FROM_PC_FILE
> - $(SED) 's%$${bindir}/%%g' $(STAGING_DIR)/usr/lib/pkgconfig/glib-2.0.pc
> + for file in glib-2.0.pc gio-2.0.pc ; do \
> + $(SED) 's%$${bindir}/%%g' \
> + $(STAGING_DIR)/usr/lib/pkgconfig/$$file ; \
> + done
> endef
> LIBGLIB2_POST_INSTALL_TARGET_HOOKS += LIBGLIB2_REMOVE_BINDIR_PREFIX_FROM_PC_FILE
>
> --
> 2.20.1
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] package/libglib2: remove bindir prefix from gio-2.0.pc
2019-06-11 18:19 ` Adam Duskett
@ 2019-06-11 19:22 ` Yann E. MORIN
2019-06-13 2:50 ` Carlos Santos
0 siblings, 1 reply; 4+ messages in thread
From: Yann E. MORIN @ 2019-06-11 19:22 UTC (permalink / raw)
To: buildroot
Adam, All,
On 2019-06-11 14:19 -0400, Adam Duskett spake thusly:
> Openembedded has a patch for this that might be even better for us to use:
> https://github.com/openembedded/openembedded-core/blob/master/meta/recipes-core/glib-2.0/glib-2.0/0001-Do-not-write-bindir-into-pkg-config-files.patch
As discussed on IRC (Fabrice, you should *really* join):
Upstream glib refused that patch, because they do want to have the
paths.
So it has zero chance to be applied, so we'll have to carry it over and
over.
So, I prefer we go with the hook. Still, see comments below...
> If not, then consider this an
> Acked-by: Adam Duskett <aduskett@gmail.com>
>
> On Tue, Jun 11, 2019 at 1:30 PM Fabrice Fontaine
> <fontaine.fabrice@gmail.com> wrote:
> >
> > Apply LIBGLIB2_REMOVE_BINDIR_PREFIX_FROM_PC_FILE to gio-2.0.pc so
> > ${bindir} prefix is also removed for gbus-codegen otherwise build of
> > packages using this binary will fail if gbus-codegen is not installed on
> > host
> >
> > Fixes:
> > - http://autobuild.buildroot.org/results/5c0e4f3d33e01198688d58388aabe159bd005234
> >
> > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> > ---
> > package/libglib2/libglib2.mk | 16 ++++++++++------
> > 1 file changed, 10 insertions(+), 6 deletions(-)
> >
> > diff --git a/package/libglib2/libglib2.mk b/package/libglib2/libglib2.mk
> > index 78d6e5ab32..c8db706684 100644
> > --- a/package/libglib2/libglib2.mk
> > +++ b/package/libglib2/libglib2.mk
> > @@ -85,13 +85,17 @@ endef
> >
> > LIBGLIB2_POST_INSTALL_TARGET_HOOKS += LIBGLIB2_REMOVE_DEV_FILES
> >
> > -# Newer versions of libglib2 prefix glib-genmarshal, gobject-query, and
> > -# glib-mkenums with ${bindir}. Unfortunately, this will resolve to the host
> > -# systems /bin/ directory, which will cause compilation issues if the host does
> > -# not have these programs. By removing the ${bindir}/ prefix, these programs
> > -# are resolved in PATH instead.
> > +# Newer versions of libglib2 prefix glib-genmarshal, gobject-query,
> > +# glib-mkenums, glib_compile_schemas, glib_compile_resources and gdbus-codegen
> > +# with ${bindir}. Unfortunately, this will resolve to the host systems /bin/
> > +# directory, which will cause compilation issues if the host does not have these
> > +# programs. By removing the ${bindir}/ prefix, these programs are resolved in
> > +# PATH instead.
> > define LIBGLIB2_REMOVE_BINDIR_PREFIX_FROM_PC_FILE
> > - $(SED) 's%$${bindir}/%%g' $(STAGING_DIR)/usr/lib/pkgconfig/glib-2.0.pc
> > + for file in glib-2.0.pc gio-2.0.pc ; do \
> > + $(SED) 's%$${bindir}/%%g' \
> > + $(STAGING_DIR)/usr/lib/pkgconfig/$$file ; \
IF the sed fails (for whatever reason), you'll miss the error, so...
> > + done
... prefer using $(foreach ...) instead of a shell loop.
Regards,
Yann E. MORIN.
> > endef
> > LIBGLIB2_POST_INSTALL_TARGET_HOOKS += LIBGLIB2_REMOVE_BINDIR_PREFIX_FROM_PC_FILE
> >
> > --
> > 2.20.1
> >
> > _______________________________________________
> > buildroot mailing list
> > buildroot at busybox.net
> > http://lists.busybox.net/mailman/listinfo/buildroot
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH 1/1] package/libglib2: remove bindir prefix from gio-2.0.pc
2019-06-11 19:22 ` Yann E. MORIN
@ 2019-06-13 2:50 ` Carlos Santos
0 siblings, 0 replies; 4+ messages in thread
From: Carlos Santos @ 2019-06-13 2:50 UTC (permalink / raw)
To: buildroot
On Tue, Jun 11, 2019 at 4:22 PM Yann E. MORIN <yann.morin.1998@free.fr> wrote:
>
> Adam, All,
>
> On 2019-06-11 14:19 -0400, Adam Duskett spake thusly:
> > Openembedded has a patch for this that might be even better for us to use:
> > https://github.com/openembedded/openembedded-core/blob/master/meta/recipes-core/glib-2.0/glib-2.0/0001-Do-not-write-bindir-into-pkg-config-files.patch
>
> As discussed on IRC (Fabrice, you should *really* join):
>
> Upstream glib refused that patch, because they do want to have the
> paths.
>
> So it has zero chance to be applied, so we'll have to carry it over and
> over.
>
> So, I prefer we go with the hook. Still, see comments below...
>
> > If not, then consider this an
> > Acked-by: Adam Duskett <aduskett@gmail.com>
> >
> > On Tue, Jun 11, 2019 at 1:30 PM Fabrice Fontaine
> > <fontaine.fabrice@gmail.com> wrote:
> > >
> > > Apply LIBGLIB2_REMOVE_BINDIR_PREFIX_FROM_PC_FILE to gio-2.0.pc so
> > > ${bindir} prefix is also removed for gbus-codegen otherwise build of
> > > packages using this binary will fail if gbus-codegen is not installed on
> > > host
> > >
> > > Fixes:
> > > - http://autobuild.buildroot.org/results/5c0e4f3d33e01198688d58388aabe159bd005234
> > >
> > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> > > ---
> > > package/libglib2/libglib2.mk | 16 ++++++++++------
> > > 1 file changed, 10 insertions(+), 6 deletions(-)
> > >
> > > diff --git a/package/libglib2/libglib2.mk b/package/libglib2/libglib2.mk
> > > index 78d6e5ab32..c8db706684 100644
> > > --- a/package/libglib2/libglib2.mk
> > > +++ b/package/libglib2/libglib2.mk
> > > @@ -85,13 +85,17 @@ endef
> > >
> > > LIBGLIB2_POST_INSTALL_TARGET_HOOKS += LIBGLIB2_REMOVE_DEV_FILES
> > >
> > > -# Newer versions of libglib2 prefix glib-genmarshal, gobject-query, and
> > > -# glib-mkenums with ${bindir}. Unfortunately, this will resolve to the host
> > > -# systems /bin/ directory, which will cause compilation issues if the host does
> > > -# not have these programs. By removing the ${bindir}/ prefix, these programs
> > > -# are resolved in PATH instead.
> > > +# Newer versions of libglib2 prefix glib-genmarshal, gobject-query,
> > > +# glib-mkenums, glib_compile_schemas, glib_compile_resources and gdbus-codegen
> > > +# with ${bindir}. Unfortunately, this will resolve to the host systems /bin/
> > > +# directory, which will cause compilation issues if the host does not have these
> > > +# programs. By removing the ${bindir}/ prefix, these programs are resolved in
> > > +# PATH instead.
> > > define LIBGLIB2_REMOVE_BINDIR_PREFIX_FROM_PC_FILE
> > > - $(SED) 's%$${bindir}/%%g' $(STAGING_DIR)/usr/lib/pkgconfig/glib-2.0.pc
> > > + for file in glib-2.0.pc gio-2.0.pc ; do \
> > > + $(SED) 's%$${bindir}/%%g' \
> > > + $(STAGING_DIR)/usr/lib/pkgconfig/$$file ; \
>
> IF the sed fails (for whatever reason), you'll miss the error, so...
>
> > > + done
>
> ... prefer using $(foreach ...) instead of a shell loop.
Or simply
define LIBGLIB2_REMOVE_BINDIR_PREFIX_FROM_PC_FILE
$(SED) 's%$${bindir}/%%g' \
$(STAGING_DIR)/usr/lib/pkgconfig/glib-2.0.pc \
$(STAGING_DIR)/usr/lib/pkgconfig/gio-2.0.pc
endef
--
Carlos Santos <unixmania@gmail.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-06-13 2:50 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-11 17:30 [Buildroot] [PATCH 1/1] package/libglib2: remove bindir prefix from gio-2.0.pc Fabrice Fontaine
2019-06-11 18:19 ` Adam Duskett
2019-06-11 19:22 ` Yann E. MORIN
2019-06-13 2:50 ` Carlos Santos
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.