Opened 3 months ago

Last modified 7 weeks ago

#23587 new enhancement

lame-4.0 (Wait for 4.1)

Reported by: Bruce Dubbs Owned by: blfs-book
Priority: elevated Milestone: 99-Waiting
Component: BOOK Version: git
Severity: medium Keywords:
Cc:

Description (last modified by Xi Ruoyao)

New major version.

Attachments (1)

lame-4.0-upstream_fixes-1.patch​ (7.6 KB ) - added by zeckma 3 months ago.
Here is a patch I created to fix LAME-4.0 without the need for CFLAGS and it properly fixes the code.

Download all attachments as: .zip

Change History (14)

comment:1 by Bruce Dubbs, 3 months ago

Note: this is is first update since 2017.

comment:2 by Joe Locash, 3 months ago

This FTBFS. Upstream ticket: ​https://sourceforge.net/p/lame/bugs/523/

comment:3 by Bruce Dubbs, 3 months ago

Evidently the person who did this update just checked that it built on windows.

I got this to compile with:

sed -i -e 's/^\(\s*hardcode_libdir_flag_spec\s*=\).*/\1/' configure &&
sed -i '/hip_decode1_headersB/a hip_finish_pinfo' include/libmp3lame.sym &&
export CFLAGS+=" -Wno-implicit-function-declaration -Wno-incompatible-pointer-types" &&
./configure --prefix=/usr --enable-mp3rtp --disable-static &&
make

But the link phase of the frontend/ fails with

'undefined reference to `id3tag_set_textinfo_utf8'

That function is in libmp3lame/.libs/id3tag.o, but adding that creates cascading undefined references.

I'm inclined to just make this wontfix and use the current version in the book that seems to have been working for the last 10 years.

comment:4 by Xi Ruoyao, 3 months ago

Description: modified (diff)
Summary: lame-3.101 → lame-4.0

4.0 still FTBFS.

in reply to:  4 ; comment:5 by Bruce Dubbs, 3 months ago

Replying to Xi Ruoyao:

4.0 still FTBFS.

Upstream claims it will be fixed in 4.1.

in reply to:  5 comment:6 by Joe Locash, 3 months ago

Replying to Bruce Dubbs:

Upstream claims it will be fixed in 4.1.

The latest svn builds fine.

comment:7 by Bruce Dubbs, 3 months ago

Summary: lame-4.0 → lame-4.0 (Wait for 4,1)

comment:8 by zeckma, 3 months ago

Summary: lame-4.0 (Wait for 4,1) → lame-4.0 (Wait for 4.1)

comment:9 by zeckma, 3 months ago

mpg123 should be added as a recommended dependency in 4.x for use with decoding, but can be disabled via --disable-decoder.

comment:10 by zeckma, 3 months ago

This update is a security release, enabling security hardening by default, rejecting interleaving encodings on single channels, and fixed an integer underflow. There is another security fix that is Windows-only as it involves DLLs.

See ​https://svn.code.sf.net/p/lame/svn/trunk/lame/doc/html/history.html for more information.

comment:11 by zeckma, 3 months ago

Priority: normal → elevated

by zeckma, 3 months ago

Here is a patch I created to fix LAME-4.0 without the need for CFLAGS and it properly fixes the code.

comment:12 by Bruce Dubbs, 2 months ago

Milestone: 13.1 → 13.2

comment:13 by Bruce Dubbs, 7 weeks ago

Milestone: 13.2 → 99-Waiting
Note: See TracTickets for help on using tickets.