After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 726571 - should use Makefile.introspection
should use Makefile.introspection
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: common
git master
Other OpenBSD
: Normal normal
: 1.15.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
: 750296 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2014-03-17 18:49 UTC by Antoine Jacoutot
Modified: 2018-05-22 05:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
do not hardcode libtool path (3.20 KB, patch)
2014-04-09 14:45 UTC, Antoine Jacoutot
committed Details | Review
plugins-base: do not hardcode libtool path (7.41 KB, patch)
2014-04-18 08:40 UTC, Antoine Jacoutot
committed Details | Review
plugins-bad: do not hardcode libtool path (1.72 KB, patch)
2014-04-18 08:40 UTC, Antoine Jacoutot
committed Details | Review

Description Antoine Jacoutot 2014-03-17 18:49:18 UTC
Hi.

The hand-rolled rules in the Makefile are missing some things that the m4
macros have -- particularly support for a non-standard libtool path.
See also: https://fedorahosted.org/libosinfo/ticket/11

On OpenBSD this allows using the OS provided libtool(1)
(https://bugzilla.gnome.org/show_bug.cgi?id=726518) and fix the build.

This is needed for at least gstreamer, gst-plugins-base and gst-plugins-bad.
No patch sorry, I am not familiar enough with this... :-/
Comment 1 Antoine Jacoutot 2014-04-09 14:45:18 UTC
Created attachment 273901 [details] [review]
do not hardcode libtool path

FWIW this patch does the trick.
Comment 2 Antoine Jacoutot 2014-04-17 07:50:40 UTC
Hi. Any opinion on this?
Thanks.
Comment 3 Sebastian Dröge (slomo) 2014-04-17 08:24:06 UTC
I think we should use Makefile.introspection if that's possible somehow. Can you provide a patch for that instead?
Comment 4 Tim-Philipp Müller 2014-04-17 08:30:17 UTC
Should be fine I guess:

commit 3e24bbefba9889c621d35234143271d6f0453326
Author: Antoine Jacoutot <ajacoutot@gnome.org>
Date:   Wed Apr 9 16:44:07 2014 +0200

    libs: g-ir-scanner: do not hardcode libtool path
    
    https://bugzilla.gnome.org/show_bug.cgi?id=726571


Will see what the build bots say and leave it open for -base.
Comment 5 Tim-Philipp Müller 2014-04-17 08:31:01 UTC
There's another bug to use Makefile.introspection, but I think there was some stuff missing there, and it feels like a change I'd rather push at the beginning of a cycle than towards the end.
Comment 6 Antoine Jacoutot 2014-04-17 10:08:02 UTC
(In reply to comment #4)
> Should be fine I guess:
> 
> commit 3e24bbefba9889c621d35234143271d6f0453326
> Author: Antoine Jacoutot <ajacoutot@gnome.org>
> Date:   Wed Apr 9 16:44:07 2014 +0200
> 
>     libs: g-ir-scanner: do not hardcode libtool path
> 
>     https://bugzilla.gnome.org/show_bug.cgi?id=726571
> 
> 
> Will see what the build bots say and leave it open for -base.

Awesome, thanks Tim.
Note that as mentioned in my initial post, the same should be done in plugins-bad and plugins-base. Do you want a patch or I suppose it'd go faster if you take care of it yourself?
Thanks.
Comment 7 Antoine Jacoutot 2014-04-18 08:40:04 UTC
Created attachment 274655 [details] [review]
plugins-base: do not hardcode libtool path

Patch for plugins-base.
Comment 8 Antoine Jacoutot 2014-04-18 08:40:29 UTC
Created attachment 274656 [details] [review]
plugins-bad: do not hardcode libtool path

Patch for plugins-bad.
Comment 9 Antoine Jacoutot 2014-04-27 16:53:27 UTC
Hi Tim.

Any other input regarding the plugins-base and plugins-bad patches?
Thanks.
Comment 10 Shawn Walker 2015-06-03 20:41:48 UTC
*** Bug 750296 has been marked as a duplicate of this bug. ***
Comment 11 Tim-Philipp Müller 2017-07-15 11:08:15 UTC
Thanks for the bug report and the patches.

We're moving away from autotools towards Meson, so I'm going to close this as I don't think it makes sense to change this at this point, sorry.

Please give the Meson port a spin on OpenBSD and let us know if there are any issues with that, thanks!
Comment 12 Randy Li (ayaka) 2018-05-22 02:55:46 UTC
The gst-libs/gst/gl/Makefile.am in -base is skipped in the previous patch.
Comment 13 Edward Hervey 2018-05-22 05:03:53 UTC
Pushed a fix for that. Thanks !