Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 247748 - media-libs/gmyth: fails with forced --as-needed
Summary: media-libs/gmyth: fails with forced --as-needed
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: GStreamer package maintainers
URL: http://www.gentoo.org/proj/en/qa/asne...
Whiteboard:
Keywords:
: 234801 (view as bug list)
Depends on:
Blocks: as-needed
  Show dependency tree
 
Reported: 2008-11-20 14:28 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2010-06-24 10:02 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Build log (media-libs:gmyth-0.7:20081120-125208.log,66.29 KB, text/plain)
2008-11-20 14:28 UTC, Diego Elio Pettenò (RETIRED)
Details
gmyth-as-needed.patch (gmyth-as-needed.patch,1.01 KB, patch)
2009-08-27 21:51 UTC, James Le Cuirot
Details | Diff
gmyth-as-needed-0.7.1.patch (gmyth-as-needed-0.7.1.patch,1.02 KB, patch)
2009-08-27 22:12 UTC, James Le Cuirot
Details | Diff
adding patch to ebuild and running autotools (gmyth-0.7.diff,641 bytes, patch)
2010-01-09 16:14 UTC, Kacper Kowalik (Xarthisius) (RETIRED)
Details | Diff
fixing as-needed issues (an aclocal) (gmyth-0.7-as-needed.patch,1.38 KB, patch)
2010-01-09 16:20 UTC, Kacper Kowalik (Xarthisius) (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Elio Pettenò (RETIRED) gentoo-dev 2008-11-20 14:28:01 UTC
I'm reporting this bug because the package in summary fails to build when forcing --as-needed on through spec files (check out http://blog.flameeyes.eu/2008/11/14/problems-and-mitigation-strategies-for-as-needed for details).

Check the attached build log.

Thanks,
Diego
Comment 1 Diego Elio Pettenò (RETIRED) gentoo-dev 2008-11-20 14:28:55 UTC
Created attachment 172507 [details]
Build log
Comment 2 James Le Cuirot gentoo-dev 2009-08-27 21:51:08 UTC
Created attachment 202434 [details, diff]
gmyth-as-needed.patch

Patch to fix this. Only Makefile.in needs patching but I also included Makefile.am for completeness.
Comment 3 James Le Cuirot gentoo-dev 2009-08-27 22:12:24 UTC
Created attachment 202436 [details, diff]
gmyth-as-needed-0.7.1.patch

Here's a patch for 0.7.1, which isn't in Portage yet.
Comment 4 Diego Elio Pettenò (RETIRED) gentoo-dev 2009-12-01 10:47:27 UTC
*** Bug 234801 has been marked as a duplicate of this bug. ***
Comment 5 Samuli Suominen (RETIRED) gentoo-dev 2009-12-21 14:56:50 UTC
Well, the patch should only be for Makefile.am and the ebuild needs autotools.eclass and AT_M4DIR="m4" eautoreconf, but

**** aclocal *****
***** PWD: /var/tmp/portage/media-libs/gmyth-0.7/work/gmyth
***** aclocal -I m4

configure.ac:38: error: AC_SUBST: `DX_FLAG_[]DX_CURRENT_FEATURE' is not a valid shell variable name
m4/ac_doxygen.m4:77: DX_REQUIRE_PROG is expanded from...
m4/ac_doxygen.m4:117: DX_ARG_ABLE is expanded from...
m4/ac_doxygen.m4:178: DX_INIT_DOXYGEN is expanded from...
configure.ac:38: the top level
autom4te-2.65: /usr/bin/m4 failed with exit status: 1
aclocal-1.10: autom4te failed with exit status: 1

it fails drasticly here, so not applying at this time
Comment 6 Kacper Kowalik (Xarthisius) (RETIRED) gentoo-dev 2010-01-09 16:14:54 UTC
Created attachment 215827 [details, diff]
adding patch to ebuild and running autotools
Comment 7 Kacper Kowalik (Xarthisius) (RETIRED) gentoo-dev 2010-01-09 16:20:13 UTC
Created attachment 215830 [details, diff]
fixing as-needed issues (an aclocal)

Autotools have to be called due to infinite mktemp check. 

Package now compiles, however tests fail for me:
1. they fail to compile due to wrong includes <gmyth/*> It can be worked around by cd "${S}"; ln -s src gmyth; make check
2. after successful compilation they run and hang indefinitely
Comment 8 Markos Chandras (RETIRED) gentoo-dev 2010-06-24 10:02:49 UTC
This bug is only related to --as-needed issue

Bug 315565 is for the broken test suite

Patch applied and now it compiles fine with --as-needed

Thanks Kacper