Bug 483725 - rawhide rpm can't be build on f10 or epel
Summary: rawhide rpm can't be build on f10 or epel
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: gstreamer
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Adam Jackson
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 486008 (view as bug list)
Depends On: 486196
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-02-03 11:24 UTC by Levente Farkas
Modified: 2009-11-29 17:14 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-04-27 19:27:07 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
idealistic patch (1.48 KB, patch)
2009-04-15 11:40 UTC, Julian Sikorski
no flags Details | Diff

Description Levente Farkas 2009-02-03 11:24:39 UTC
hi,
the latest rawhide rpm gstreamer-0.10.22-1.fc11 can't be build on fedora-10 or epel-5 since it's required a newer libtool/ imho it'd be useful to fix somehow:
------------------------------------------------------
make[2]: Entering directory `/tmp/gstreamer-0.10.22/tools'
test -z "/usr/bin" || /bin/mkdir -p "/home/lfarkas/rpm/BUILDROOT/gstreamer-0.10.22-1.fc11.i386/usr/bin"
  /bin/sh ../libtool   --mode=install /usr/bin/install -c 'gst-xmllaunch' '/home/lfarkas/rpm/BUILDROOT/gstreamer-0.10.22-1.fc11.i386/usr/bin/gst-xmllaunch'
libtool: install: /usr/bin/install -c gst-xmllaunch /home/lfarkas/rpm/BUILDROOT/gstreamer-0.10.22-1.fc11.i386/usr/bin/gst-xmllaunch
  /bin/sh ../libtool   --mode=install /usr/bin/install -c 'gst-xmllaunch-0.10' '/home/lfarkas/rpm/BUILDROOT/gstreamer-0.10.22-1.fc11.i386/usr/bin/gst-xmllaunch-0.10'
libtool: install: invalid libtool wrapper script `gst-xmllaunch-0.10'
make[2]: *** [install-binPROGRAMS] Error 1
make[2]: Leaving directory `/tmp/gstreamer-0.10.22/tools'
make[1]: *** [install-am] Error 2
make[1]: Leaving directory `/tmp/gstreamer-0.10.22/tools'
make: *** [install-recursive] Error 1
------------------------------------------------------

Comment 1 Levente Farkas 2009-02-18 21:22:15 UTC
*** Bug 486008 has been marked as a duplicate of this bug. ***

Comment 2 Levente Farkas 2009-02-18 21:23:41 UTC
the solution at #486196

Comment 3 Julian Sikorski 2009-03-08 22:43:20 UTC
0.10.22-3.fc11 fails to rebuild as well.

Comment 4 Bastien Nocera 2009-03-09 10:56:00 UTC
The "fix" is to revert the changes done in bug 486196. It's not a bug in gstreamer on rawhide anyway, as it builds just fine there.

Please test with the instructions above.

Comment 5 Levente Farkas 2009-03-09 11:19:22 UTC
it's still wrong!
as i wrote i the other bug report do not build with:
make %{?_smp_mflags} ERROR_CFLAGS="" LIBTOOL="%{_bindir}/libtool"
but with:
make %{?_smp_mflags}
it's the fedora's recommended way and it's working on _both_ fedora and epel!
so it's not just the parallel build can be enabled but use the system libtool.
yours.

Comment 6 Julian Sikorski 2009-03-09 13:49:48 UTC
According to changelog, LIBTOOL voodoo is required to get rid of rpaths. I'll try finding another way.

Comment 7 Levente Farkas 2009-03-09 14:02:11 UTC
may be that changelog it outdated! it's no longer needed neither on fedora nor on epel.

Comment 8 Julian Sikorski 2009-03-09 14:09:53 UTC
Actually, the rpaths will prevent rpmbuild from working if that libtool kludge is removed. However, using the sed trick outlined in Fedora Packaging Guidelines [1]:

sed -i 's|^hardcode_libdir_flag_spec=.*|hardcode_libdir_flag_spec=""|g' libtool
sed -i 's|^runpath_var=LD_RUN_PATH|runpath_var=DIE_RPATH_DIE|g' libtool

does not break the build on Fedora 10, while at the same time taking care of the rpath problem.
Bastien, Adam, do you want me to commit this fix?

[1] https://fedoraproject.org/wiki/Packaging:Guidelines#Removing_Rpath

Comment 9 Levente Farkas 2009-03-09 14:16:36 UTC
or you can modify it to:
---------------------------
%if 0%{?fedora} >= 9
make ERROR_CFLAGS="" LIBTOOL="%{_bindir}/libtool"
%else
make %{?_smp_mflags}
%endif
---------------------------
and then everybod will happy.
and imho in case of fedora before configure
autoreconf
libtoolize -f
would also be useful since the included libtool is much older then the system one.

Comment 10 Julian Sikorski 2009-03-09 14:19:40 UTC
In my opinion changes to the autotools-generated files should be kept to minimum.

Comment 11 Julian Sikorski 2009-03-10 15:34:56 UTC
OK, scratch this. Without provenpackager, I won't be able to commit that. Please merge the fix mentioned in comment #8 (the make line should be changed to plain make %{?_smp_mflags}, and the BR on libtool can be dropped.

Comment 12 Levente Farkas 2009-04-12 21:43:00 UTC
any reason why #8 not applied?

Comment 13 Bastien Nocera 2009-04-12 23:19:12 UTC
I'm waiting for a patch against the F-10 branch.

Comment 14 Julian Sikorski 2009-04-13 17:32:03 UTC
The bug says that the rawhide rpm can't be rebuilt on F-10, so I believe it's the devel branch that should be patched. In case I'm misunderstanding, you want that change separated from gstreamer update, right?

Comment 15 Levente Farkas 2009-04-14 09:48:24 UTC
this patch is for rawhide. i would like to be able to rebuild rawhide's src.rpm on both f10 and epel. with this patch it can be compiled on both. that's why i like to apply it to rawhide's spec file.

Comment 16 Julian Sikorski 2009-04-15 11:40:49 UTC
Created attachment 339670 [details]
idealistic patch

Looks like it won't be that easy. I tested the rebuild using rpmbuild and not mock, and it turns out that gstreamer won't rebuild in mock with the attached patch applied. It fails with the following error:

gtk-doc: Running scanner gstreamer-scan
Use of uninitialized value $ENV{"RUN"} in concatenation (.) or string at /usr/bin/gtkdoc-scangobj line 1548, <TYPES> line 41.
/builddir/build/BUILD/gstreamer-0.10.22/docs/gst/.libs/lt-gstreamer-scan: error while loading shared libraries: libgstreamer-0.10.so.0: cannot open shared object file: No such file or directory
Scan failed: 
make[5]: *** [scan-build.stamp] Error 127
make[5]: Leaving directory `/builddir/build/BUILD/gstreamer-0.10.22/docs/gst'
make[4]: *** [all] Error 2
make[4]: Leaving directory `/builddir/build/BUILD/gstreamer-0.10.22/docs/gst'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory `/builddir/build/BUILD/gstreamer-0.10.22/docs'
make[2]: *** [all] Error 2
make[2]: Leaving directory `/builddir/build/BUILD/gstreamer-0.10.22/docs'
make[1]: make[1]: Leaving directory `/builddir/build/BUILD/gstreamer-0.10.22'
*** [all-recursive] Error 1
make: *** [all] Error 2

I think it works with local rpmbuild since I have gstreamer rpm installed. So, I suppose a different approach is needed to solve this problem. Sorry for spreading misinformation, I should have checked more thoroughly.

Comment 17 Bastien Nocera 2009-04-27 19:27:07 UTC
(In reply to comment #15)
> this patch is for rawhide. i would like to be able to rebuild rawhide's src.rpm
> on both f10 and epel. with this patch it can be compiled on both. that's why i
> like to apply it to rawhide's spec file.  

Just like I mentioned in bug 486882, we won't be making changes to the rawhide spec file to be able to build on older distributions.

If you want to update gstreamer in F-10, feel free to open a new bug with a patch against F-10, and ask for ACLs in pkgdb.

If you want to update gstreamer in RHEL, same thing, along with a list of reasons why we should be updating the package.

Comment 18 David Ward 2009-11-29 17:14:45 UTC
I've filed an upstream bug about '--disable-rpath' not working in the configure script as shipped.  When fixed, this should prevent the need for either the system libtool or the sed hack on Fedora or RHEL.

https://bugzilla.gnome.org/show_bug.cgi?id=603275


Note You need to log in before you can comment on or make changes to this bug.