Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 181384 - mplayer failed to compile (can't find a register in class `GENERAL_REGS'...)
Summary: mplayer failed to compile (can't find a register in class `GENERAL_REGS'...)
Status: RESOLVED DUPLICATE of bug 175627
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: x86 Linux
: High normal (vote)
Assignee: Gentoo Linux bug wranglers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-09 08:07 UTC by Guy Cirino
Modified: 2007-06-09 08:18 UTC (History)
0 users

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


Attachments
patch for libavcodec/Makefile (mplayer-h264-c-register-fix.patch,1.27 KB, patch)
2007-06-09 08:09 UTC, Guy Cirino
Details | Diff
ebuild update, patches the above attachement (mplayer-1.0.20070427.ebuild.diff,269 bytes, patch)
2007-06-09 08:10 UTC, Guy Cirino
Details | Diff
added update description to ChangeLog (ChangeLog.diff,671 bytes, patch)
2007-06-09 08:15 UTC, Guy Cirino
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Guy Cirino 2007-06-09 08:07:58 UTC
Hi guys,

This sounds like a variant of bug #175627, so I cloned it.  mplayer-1.0.20070427 (and possibly older ones) fails to build libavcodec/h264.c because it runs out of registers.  A Mennucc1 from the debian project spotted this error, see http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2007-March/050608.html and applied a fix, which I adapted and included in the attachements.  The fix basically flips to a different set of optimizations on x86 architecture which provides enough registers to compile h264.c.  I commented out A's fix to only apply when not building in debug mode, it just doesn't work if this isn't enabled.

Attached are the patch file and a diffed ebuild from mplayer-1.0.20070427.ebuild - the only change to the ebuild is to apply the patch.

Cheers,
_guy
Comment 1 Guy Cirino 2007-06-09 08:09:16 UTC
Created attachment 121545 [details, diff]
patch for libavcodec/Makefile
Comment 2 Guy Cirino 2007-06-09 08:10:26 UTC
Created attachment 121546 [details, diff]
ebuild update, patches the above attachement
Comment 3 Jakub Moc (RETIRED) gentoo-dev 2007-06-09 08:11:30 UTC
Why don't you just attach the patch to the relevant bug?

*** This bug has been marked as a duplicate of bug 175627 ***

*** This bug has been marked as a duplicate of bug 175627 ***
Comment 4 Guy Cirino 2007-06-09 08:15:57 UTC
Created attachment 121547 [details, diff]
added update description to ChangeLog
Comment 5 Guy Cirino 2007-06-09 08:18:38 UTC
(In reply to comment #3)
> Why don't you just attach the patch to the relevant bug?

Sorry, didn't think it was really the same thing..  Didn't update the source to solve the need of the extra registers, and the failure seemed different enough that I didn't find the fix until I'd been searching for a while.  If you still disagree, let me know and I'll put it there instead.

_guy