Bug 262874

Summary: audio/libmad: Update to 0.16.0
Product: Ports & Packages Reporter: Daniel Engberg <diizzy>
Component: Individual Port(s)Assignee: Po-Chuan Hsieh <sunpoet>
Status: Closed FIXED    
Severity: Affects Only Me CC: diizzy, pkubaj, vishwin
Priority: --- Flags: bugzilla: maintainer-feedback? (sunpoet)
Version: Latest   
Hardware: Any   
OS: Any   
URL: https://github.com/tenacityteam/libmad/blob/main/CHANGES
See Also: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=264442
Attachments:
Description Flags
Patch for libmad
none
Patch for libmad
none
Patch for libmad v3 none

Description Daniel Engberg freebsd_committer freebsd_triage 2022-03-28 01:37:23 UTC
Created attachment 232766 [details]
Patch for libmad

Switch to Tenacity's fork which contains patched code for all known CVEs and various build fixes. This also makes it easier to maintain as we're using an active fork. Upstream has also switched to CMake.

Compile tested on FreeBSD 13.0-STABLE #2 stable/13-n248607-93a95ebbf7c (amd64) (make, make check-plist)
Poudriere testport OK 12.2-RELEASE (amd64)
Poudriere testport OK 13.0-RELEASE (i386)
Comment 1 Daniel Engberg freebsd_committer freebsd_triage 2022-03-28 01:39:00 UTC
pkubaj, can you have a look on PPC?
Comment 2 Daniel Engberg freebsd_committer freebsd_triage 2022-03-30 19:31:09 UTC
There's one pending PR upstream for i386 optimization, I'll update patch once that's accepted upstream
Comment 3 Piotr Kubaj freebsd_committer freebsd_triage 2022-03-30 20:06:40 UTC
Seems ok on powerpc and powerpc64.
Comment 4 Daniel Engberg freebsd_committer freebsd_triage 2022-03-30 21:59:34 UTC
Created attachment 232831 [details]
Patch for libmad

Use upstream commits
Comment 5 Daniel Engberg freebsd_committer freebsd_triage 2022-03-30 22:02:49 UTC
Created attachment 232832 [details]
Patch for libmad v3

Attach correct patch
Comment 6 Daniel Engberg freebsd_committer freebsd_triage 2022-03-31 11:02:41 UTC
(In reply to Piotr Kubaj from comment #3)
Great, thanks!
Comment 7 Daniel Engberg freebsd_committer freebsd_triage 2022-03-31 19:46:58 UTC
I should also add that the backported patches fixes identification of x86 for both 32-bit and 64-bit on FreeBSD
Comment 8 commit-hook freebsd_committer freebsd_triage 2022-05-06 10:00:02 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=519c89efe3a9ddc8f3a57d1bb4940ff355086576

commit 519c89efe3a9ddc8f3a57d1bb4940ff355086576
Author:     Daniel Engberg <diizzy@FreeBSD.org>
AuthorDate: 2022-05-06 09:47:30 +0000
Commit:     Daniel Engberg <diizzy@FreeBSD.org>
CommitDate: 2022-05-06 09:59:33 +0000

    audio/libmad: Update to 0.16.0

    Switch to Tenacity's fork which contains patched code for all known CVEs
    and various build fixes. This also makes it easier to maintain as we're
    using an active fork. Upstream has also switched to CMake.

    PR:             262874
    Approved by:    sunpoet (maintainer timeout 4+ weeks)

 audio/libmad/Makefile                     |  30 ++---
 audio/libmad/distinfo                     |   9 +-
 audio/libmad/files/mad.pc.in (gone)       |  11 --
 audio/libmad/files/patch-configure (gone) |  40 -------
 audio/libmad/files/patch-layer12.c (gone) | 175 ------------------------------
 audio/libmad/files/patch-layer3.c (gone)  |  17 ---
 audio/libmad/pkg-plist                    |  10 +-
 7 files changed, 23 insertions(+), 269 deletions(-)
Comment 9 Charlie Li freebsd_committer freebsd_triage 2022-05-06 21:53:16 UTC
Also commenting here for more visibility: this broke at least sysutils/mtpfs and audio/audacity because those build systems now cannot detect this library. Side note, Audacity has their own fork of this library.
Comment 10 Daniel Engberg freebsd_committer freebsd_triage 2022-05-06 22:43:39 UTC
Looking into it, thanks for reporting