Bug 247748

Summary:	media-libs/gmyth: fails with forced --as-needed
Product:	Gentoo Linux	Reporter:	Diego Elio Pettenò (RETIRED) <flameeyes>
Component:	New packages	Assignee:	GStreamer package maintainers <gstreamer>
Status:	RESOLVED FIXED
Severity:	normal	CC:	chewi, esigra, gentoo-bugs, mythtv+disabled
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
URL:	http://www.gentoo.org/proj/en/qa/asneeded.xml
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	129413
Attachments:	Build log gmyth-as-needed.patch gmyth-as-needed-0.7.1.patch adding patch to ebuild and running autotools fixing as-needed issues (an aclocal)

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