Opened 18 years ago

Closed 18 years ago

Last modified 20 months ago

#403 closed defect (fixed)

extract does not work with recent lame

Reported by: SF/weine Owned by: fingolfin
Priority: normal Component: Documentation
Keywords: Cc:
Game:

Description

At least recent versions of lame have no parameter called --bitwidth, hence the "--bitwidth 16" has to be removed for the mp3-encoding to work properly when extracting. With this change, extraction works for me.

Ticket imported from: #594568. Ticket imported from: bugs/403.

Change History (6)

comment:1 by eriktorbjorn, 18 years ago

Out of curiosity, which version of lame is that? From a quick peek at their CVS, it looks like --bitwidth was first added over a year ago in which case 3.90 and later should have it.

Of course, I could be completely wrong about this. It would hardly be the first time.

comment:2 by SF/weine, 18 years ago

Hmmm. Seems that the version of Lame I had installed wasn't so recent after all; it was v3.88. So I'd have to change my statement; extract does not work with somewhat older versions of Lame. I still consider this a bug, however, especially since the option does not seem to be necessary.

comment:3 by fingolfin, 18 years ago

Does your lame version support the "-b" option (which does exactly the same?

If yes, we can switch to using that. If no, I'll close this as WONTFIX, as I think it's not to hard to require users to update to a current lame version (like 3.92).

comment:4 by SF/weine, 18 years ago

As far as I can tell:

-b <bitrate> set the bitrate in kbps, default 128 kbps --bitwidth w input bit width is w (default 16)

The latter parameter is missing in earlier versions of Lame. Since the bitwidth used by extract.c is 16, and this is the default (both for older versions that does not support the bitwidth parameter, and the newer versions that does), --bitwidth 16 is superfluous for new versions of Lame, and a showstopper with older versions.

-b does something completely different.

comment:5 by fingolfin, 18 years ago

Owner: set to fingolfin
Resolution: fixed
Status: newclosed

comment:6 by digitall, 20 months ago

Component: Documentation
Note: See TracTickets for help on using tickets.