Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 124771 - media-sound/mppenc not available for ppc
Summary: media-sound/mppenc not available for ppc
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: PPC Linux
: High enhancement
Assignee: PPC Porters
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-03-03 04:44 UTC by Dirk Silkenbäumer
Modified: 2007-02-10 22:48 UTC (History)
1 user (show)

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


Attachments
musepack-tools ebuild for ppc (mppdec only) (musepack-tools-1.15v-r1.ebuild,1.80 KB, text/plain)
2006-03-03 04:47 UTC, Dirk Silkenbäumer
Details
Makefile patch (musepack-tools-1.15v-Makefile-ppc.patch,837 bytes, patch)
2006-03-03 04:49 UTC, Dirk Silkenbäumer
Details | Diff
Patch for mppenc-1.16.ebuild (mppenc-1.16.patch,587 bytes, patch)
2007-02-10 12:46 UTC, Matti Bickel (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Silkenbäumer 2006-03-03 04:44:59 UTC
The musepack-tools are not available for ppc. Anyhow the mppdec utility compiles and runs fine on my Macmini. mppenc and replaygain are using x86 asm-code.
Comment 1 Dirk Silkenbäumer 2006-03-03 04:47:04 UTC
Created attachment 81192 [details]
musepack-tools ebuild for ppc (mppdec only)
Comment 2 Dirk Silkenbäumer 2006-03-03 04:49:00 UTC
Created attachment 81193 [details, diff]
Makefile patch
Comment 3 Wormo (RETIRED) gentoo-dev 2006-03-23 22:30:47 UTC
Thanks for the patches -- I'll see what I can do about getting the other tools to build.

According to the musepack forum, the encoder can be built on OS X as well (it just doesn't have the optimized assembly, as you noticed) so it must also be big-endian safe
http://www.musepack.net/forum/viewtopic.php?t=180
Comment 4 Matti Bickel (RETIRED) gentoo-dev 2007-02-08 14:57:44 UTC
What you basically want to set is
#define ENDIAN HAVE_BIG_ENDIAN
at least, that's what i get from the source code (and what's defined w/ __APPLE__, other big endian arches are screwed, too)

I'll have a shoot at it once i return home. :)

Oh and musepack-tools were repackaged to mppenc, Diego made a new package for that. We should be getting our fixes there.
Comment 5 Matti Bickel (RETIRED) gentoo-dev 2007-02-10 12:46:13 UTC
Yeah, compiles fine without the -D__APPLE__ thing, but produces crap output (mplayer can't play it). With that set, mplayer recognizes and plays the file just fine.

Sound: can you take a look at my patch and include it in a -r bump that ppc can mark ~arch?
Comment 6 Matti Bickel (RETIRED) gentoo-dev 2007-02-10 12:46:32 UTC
Created attachment 109714 [details, diff]
Patch for mppenc-1.16.ebuild

That's all that seems to be required to get it working on my machine.
Comment 7 Diego Elio Pettenò (RETIRED) gentoo-dev 2007-02-10 16:33:20 UTC
I don't see why a new revision, as you didn't have the keyword there, please don't get people to rebuild a package just because.

Anyway please note that musepack-tools would probably be deprecated as soon as upstream decides to release the sources for mppenc.
Comment 8 Matti Bickel (RETIRED) gentoo-dev 2007-02-10 22:37:50 UTC
Not bumping is fine with me. However, we need the patch in mppenc first...

I don't get your second paragraph, though. I thought mppenc sources were what we're talking about here?
Comment 9 Matti Bickel (RETIRED) gentoo-dev 2007-02-10 22:48:57 UTC
Fix in CVS and keyworded ~ppc.