All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/gobject-introspection: fix datarootdir
@ 2021-08-08 10:16 Fabrice Fontaine
  2021-08-08 13:16 ` Yegor Yefremov via buildroot
  2021-08-08 20:25 ` Thomas Petazzoni
  0 siblings, 2 replies; 3+ messages in thread
From: Fabrice Fontaine @ 2021-08-08 10:16 UTC (permalink / raw)
  To: buildroot; +Cc: Anisse Astier, Fabrice Fontaine

datarootdir has been removed since version 1.66 and
https://gitlab.gnome.org/GNOME/gobject-introspection/-/commit/a88b1ac548516ab25140a2e9dea8cac894bbac93
resulting in the following build failures with autotools-based
applications using introspection (e.g. libmediaart or libsecret) if
gobject-introspection is not installed on host since commit
abc110e3624aa4e8a9bd8595b3fdc0aec60845e0:

make[3]: *** No rule to make target 'MediaArt-2.0.typelib', needed by 'all'.  Stop.

or

Makefile:2917: /usr/share/gobject-introspection-1.0/Makefile.introspection: No such file or directory

Fixes:
 - http://autobuild.buildroot.org/results/70585b81fd6aa78e2aa6e5173f35c0f26a1408a9
 - http://autobuild.buildroot.org/results/95aa8e189f3326ecae6133e298c56ffcbbc3198f

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/gobject-introspection/gobject-introspection.mk | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/package/gobject-introspection/gobject-introspection.mk b/package/gobject-introspection/gobject-introspection.mk
index d8138b2e96..fbc545b3c7 100644
--- a/package/gobject-introspection/gobject-introspection.mk
+++ b/package/gobject-introspection/gobject-introspection.mk
@@ -116,12 +116,12 @@ define GOBJECT_INTROSPECTION_INSTALL_WRAPPERS
 
 	# Gobject-introspection installs Makefile.introspection in
 	# $(STAGING_DIR)/usr/share which is needed for autotools-based programs to
-	# build .gir and .typelib files. Unfortuantly, gobject-introspection-1.0.pc
+	# build .gir and .typelib files. Unfortunately, gobject-introspection-1.0.pc
 	# uses $(prefix)/share as the directory, which
 	# causes the host /usr/share being used instead of $(STAGING_DIR)/usr/share.
-	# Change datarootdir to $(libdir)/../share which will prefix $(STAGING_DIR)
+	# Change datadir to $(libdir)/../share which will prefix $(STAGING_DIR)
 	# to the correct location.
-	$(SED) "s%datarootdir=.*%datarootdir=\$${libdir}/../share%g" \
+	$(SED) "s%datadir=.*%datadir=\$${libdir}/../share%g" \
 		$(STAGING_DIR)/usr/lib/pkgconfig/gobject-introspection-1.0.pc
 
 	# By default, girdir and typelibdir use datadir and libdir as their prefix,
-- 
2.30.2

_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH 1/1] package/gobject-introspection: fix datarootdir
  2021-08-08 10:16 [Buildroot] [PATCH 1/1] package/gobject-introspection: fix datarootdir Fabrice Fontaine
@ 2021-08-08 13:16 ` Yegor Yefremov via buildroot
  2021-08-08 20:25 ` Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Yegor Yefremov via buildroot @ 2021-08-08 13:16 UTC (permalink / raw)
  To: Fabrice Fontaine; +Cc: Anisse Astier, Aleksander Morgado, buildroot

Hi Fabrice,

On Sun, Aug 8, 2021 at 12:19 PM Fabrice Fontaine
<fontaine.fabrice@gmail.com> wrote:
>
> datarootdir has been removed since version 1.66 and
> https://gitlab.gnome.org/GNOME/gobject-introspection/-/commit/a88b1ac548516ab25140a2e9dea8cac894bbac93
> resulting in the following build failures with autotools-based
> applications using introspection (e.g. libmediaart or libsecret) if
> gobject-introspection is not installed on host since commit
> abc110e3624aa4e8a9bd8595b3fdc0aec60845e0:
>
> make[3]: *** No rule to make target 'MediaArt-2.0.typelib', needed by 'all'.  Stop.
>
> or
>
> Makefile:2917: /usr/share/gobject-introspection-1.0/Makefile.introspection: No such file or directory
>
> Fixes:
>  - http://autobuild.buildroot.org/results/70585b81fd6aa78e2aa6e5173f35c0f26a1408a9
>  - http://autobuild.buildroot.org/results/95aa8e189f3326ecae6133e298c56ffcbbc3198f
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Tested against libqmi and ModemManager.

Reviewed-by: Yegor Yefremov <yegorslists@googlemail.com>

Best regards,
Yegor

> ---
>  package/gobject-introspection/gobject-introspection.mk | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/package/gobject-introspection/gobject-introspection.mk b/package/gobject-introspection/gobject-introspection.mk
> index d8138b2e96..fbc545b3c7 100644
> --- a/package/gobject-introspection/gobject-introspection.mk
> +++ b/package/gobject-introspection/gobject-introspection.mk
> @@ -116,12 +116,12 @@ define GOBJECT_INTROSPECTION_INSTALL_WRAPPERS
>
>         # Gobject-introspection installs Makefile.introspection in
>         # $(STAGING_DIR)/usr/share which is needed for autotools-based programs to
> -       # build .gir and .typelib files. Unfortuantly, gobject-introspection-1.0.pc
> +       # build .gir and .typelib files. Unfortunately, gobject-introspection-1.0.pc
>         # uses $(prefix)/share as the directory, which
>         # causes the host /usr/share being used instead of $(STAGING_DIR)/usr/share.
> -       # Change datarootdir to $(libdir)/../share which will prefix $(STAGING_DIR)
> +       # Change datadir to $(libdir)/../share which will prefix $(STAGING_DIR)
>         # to the correct location.
> -       $(SED) "s%datarootdir=.*%datarootdir=\$${libdir}/../share%g" \
> +       $(SED) "s%datadir=.*%datadir=\$${libdir}/../share%g" \
>                 $(STAGING_DIR)/usr/lib/pkgconfig/gobject-introspection-1.0.pc
>
>         # By default, girdir and typelibdir use datadir and libdir as their prefix,
> --
> 2.30.2
>
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH 1/1] package/gobject-introspection: fix datarootdir
  2021-08-08 10:16 [Buildroot] [PATCH 1/1] package/gobject-introspection: fix datarootdir Fabrice Fontaine
  2021-08-08 13:16 ` Yegor Yefremov via buildroot
@ 2021-08-08 20:25 ` Thomas Petazzoni
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2021-08-08 20:25 UTC (permalink / raw)
  To: Fabrice Fontaine; +Cc: Anisse Astier, buildroot

On Sun,  8 Aug 2021 12:16:50 +0200
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:

> datarootdir has been removed since version 1.66 and
> https://gitlab.gnome.org/GNOME/gobject-introspection/-/commit/a88b1ac548516ab25140a2e9dea8cac894bbac93
> resulting in the following build failures with autotools-based
> applications using introspection (e.g. libmediaart or libsecret) if
> gobject-introspection is not installed on host since commit
> abc110e3624aa4e8a9bd8595b3fdc0aec60845e0:
> 
> make[3]: *** No rule to make target 'MediaArt-2.0.typelib', needed by 'all'.  Stop.
> 
> or
> 
> Makefile:2917: /usr/share/gobject-introspection-1.0/Makefile.introspection: No such file or directory
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/70585b81fd6aa78e2aa6e5173f35c0f26a1408a9
>  - http://autobuild.buildroot.org/results/95aa8e189f3326ecae6133e298c56ffcbbc3198f
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
>  package/gobject-introspection/gobject-introspection.mk | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-08-08 20:25 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-08 10:16 [Buildroot] [PATCH 1/1] package/gobject-introspection: fix datarootdir Fabrice Fontaine
2021-08-08 13:16 ` Yegor Yefremov via buildroot
2021-08-08 20:25 ` Thomas Petazzoni

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.