Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 145377 - media-plugins/xmms-speex-0.9.1: work with latest speex ebuild
Summary: media-plugins/xmms-speex-0.9.1: work with latest speex ebuild
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Luis Medinas (RETIRED)
URL:
Whiteboard:
Keywords: PMASKED
Depends on: xmms-must-die
Blocks:
  Show dependency tree
 
Reported: 2006-08-28 10:40 UTC by Martin von Gagern
Modified: 2006-10-23 00:58 UTC (History)
0 users

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


Attachments
xmms-speex-0.9.1-r1.ebuild (xmms-speex-0.9.1-r1.ebuild,1.01 KB, text/plain)
2006-08-28 10:43 UTC, Martin von Gagern
Details
xmms-speex-0.9.1-r1-Makefile.patch (xmms-speex-0.9.1-r1-Makefile.patch,1.95 KB, patch)
2006-08-28 10:48 UTC, Martin von Gagern
Details | Diff
xmms-speex-0.9.1-intcrement.patch (xmms-speex-0.9.1-intcrement.patch,730 bytes, patch)
2006-08-28 10:50 UTC, Martin von Gagern
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin von Gagern 2006-08-28 10:40:29 UTC
bug 60542 comment 9 states that the current xmms-speex plugin only works with up to speex-1.1.5, and the dependency settings reflect this fact. However as the reason is only some change in include dir location, it is easily remedied.
I modified the ebuild to work with my media-libs/speex-1.1.12 and probably all speex versions starting from 1.1.6. I also commented all settings of CFLAGS in the Makefiles so that the CFLAGS settings from portage are used, plus -fPIC.
Comment 1 Martin von Gagern 2006-08-28 10:43:29 UTC
Created attachment 95293 [details]
xmms-speex-0.9.1-r1.ebuild

The new speex ebuild.
Patches are now named in full, for easier update to new versions that still need the old patches.
The HOMEPAGE is still unavailable (bug 123818), and I know nothing new here.
src_compile now calls make explicitly, so that passing args is possible.
Comment 2 Martin von Gagern 2006-08-28 10:48:04 UTC
Created attachment 95294 [details, diff]
xmms-speex-0.9.1-r1-Makefile.patch

This flag combines the intention of xmms-speex-0.9.1-Makefile.patch and xmms-speex-0.9.1-fPIC.patch. A closer look:
* INCDIRS now includes SPEEX_INCDIR which is set in ebuild.
* CFLAGS setting disabled to take the one from the environment
* utf8 enabled as in bug 60542 comment 12
* install target made DESTDIR aware, though current ebuild doesn't use this
* VERSION of gui subdir changed to match root dir
* output from xmms-config assigned to make variable so that build logs will
  display the expanded form for easier problem handling with this
Comment 3 Martin von Gagern 2006-08-28 10:50:55 UTC
Created attachment 95297 [details, diff]
xmms-speex-0.9.1-intcrement.patch

Without this patch I get the following errors with my gcc-4.1.1:

speexutil.c:240: error: invalid lvalue in increment
speexutil.c:261: error: invalid lvalue in increment
speexutil.c:271: error: invalid lvalue in increment

Reason each time is "((int*) packet)++;" where packet is of type char*.
Comment 4 Jakub Moc (RETIRED) gentoo-dev 2006-10-23 00:58:13 UTC
xmms and all plugins are now package.masked and will be removed from the tree in one month.

WONTFIX.