Bug 2370973 - Review Request: faad2 - Library and frontend for decoding MPEG2/4 AAC
Summary: Review Request: faad2 - Library and frontend for decoding MPEG2/4 AAC
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Neal Gompa
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: MultimediaSIG
TreeView+ depends on / blocked
 
Reported: 2025-06-07 22:44 UTC by Dominik 'Rathann' Mierzejewski
Modified: 2025-06-16 03:46 UTC (History)
4 users (show)

Fixed In Version: faad2-2.11.2-3.fc43
Clone Of:
Environment:
Last Closed: 2025-06-11 13:17:57 UTC
Type: ---
Embargoed:
ngompa13: fedora-review+


Attachments (Terms of Use)

Description Dominik 'Rathann' Mierzejewski 2025-06-07 22:44:06 UTC
Spec URL: https://rathann.fedorapeople.org/review/faad2/faad2.spec
SRPM URL: https://rathann.fedorapeople.org/review/faad2/faad2-2.11.2-2.fc43.src.rpm
Description:
FAAD 2 is a LC, MAIN and LTP profile, MPEG2 and MPEG-4 AAC decoder, completely written from scratch.

Fedora Account System Username: rathann

Comment 1 Neal Gompa 2025-06-08 09:02:08 UTC
Taking this review.

Comment 2 Neal Gompa 2025-06-08 09:02:49 UTC
This looks mostly okay, but there's one issue:

> %{_mandir}/man3/libfaad.3*

This should be in the devel package since it's devel documentation.

Comment 3 Nicolas Chauvet (kwizart) 2025-06-09 08:39:19 UTC
Upstream have changed obviously, you can retain the new github projet as URL. 
Specially as the domain was re-assigned in 2021, so it's not even the old site/upstream.

Comment 4 Dominik 'Rathann' Mierzejewski 2025-06-09 11:36:06 UTC
Spec URL: https://rathann.fedorapeople.org/review/faad2/faad2.spec
SRPM URL: https://rathann.fedorapeople.org/review/faad2/faad2-2.11.2-3.fc43.src.rpm

* Mon Jun 09 2025 Dominik Mierzejewski <dominik> - 1:2.11.2-3
- supply a valid upstream URL
- move libfaad manpage to devel subpackage

Comment 5 Neal Gompa 2025-06-09 22:50:14 UTC
> Epoch:		1

Can we drop this?

Comment 6 Dominik 'Rathann' Mierzejewski 2025-06-10 12:58:52 UTC
(In reply to Neal Gompa from comment #5)
> > Epoch:		1
> 
> Can we drop this?

Only if we disregard the fact that RPM Fusion users won't see any updates until they dnf system-upgrade or distro-sync to F44.

Comment 7 Neal Gompa 2025-06-10 21:15:46 UTC
Review notes:

* Package follows Fedora Packaging Guidelines
* Package builds and installs
* Package licensing is correct and license data is installed correctly
* No serious issues from rpmlint

PACKAGE APPROVED.

Comment 8 Fedora Admin user for bugzilla script actions 2025-06-11 12:00:13 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/faad2

Comment 9 Dominik 'Rathann' Mierzejewski 2025-06-11 13:17:57 UTC
https://bodhi.fedoraproject.org/updates/FEDORA-2025-7d83f28cda

Thanks for the review!

Comment 10 Fedora Update System 2025-06-12 09:31:09 UTC
FEDORA-2025-899cbbd0c7 (faad2-2.11.2-3.fc42) has been submitted as an update to Fedora 42.
https://bodhi.fedoraproject.org/updates/FEDORA-2025-899cbbd0c7

Comment 11 Fedora Update System 2025-06-12 09:31:10 UTC
FEDORA-2025-fdc965aa2c (faad2-2.11.2-3.fc41) has been submitted as an update to Fedora 41.
https://bodhi.fedoraproject.org/updates/FEDORA-2025-fdc965aa2c

Comment 12 Fedora Update System 2025-06-13 01:43:05 UTC
FEDORA-2025-899cbbd0c7 has been pushed to the Fedora 42 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2025-899cbbd0c7 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2025-899cbbd0c7

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 13 Fedora Update System 2025-06-13 02:25:00 UTC
FEDORA-2025-fdc965aa2c has been pushed to the Fedora 41 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2025-fdc965aa2c \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2025-fdc965aa2c

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 14 Fedora Update System 2025-06-13 13:56:10 UTC
FEDORA-EPEL-2025-3637b2295c (faad2-2.11.2-3.el9) has been submitted as an update to Fedora EPEL 9.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2025-3637b2295c

Comment 15 Fedora Update System 2025-06-13 13:56:10 UTC
FEDORA-EPEL-2025-432c3df31b (faad2-2.11.2-3.el10_1) has been submitted as an update to Fedora EPEL 10.1.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2025-432c3df31b

Comment 16 Fedora Update System 2025-06-14 00:47:06 UTC
FEDORA-EPEL-2025-3637b2295c has been pushed to the Fedora EPEL 9 testing repository.

You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2025-3637b2295c

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 17 Fedora Update System 2025-06-14 01:09:17 UTC
FEDORA-EPEL-2025-432c3df31b has been pushed to the Fedora EPEL 10.1 testing repository.

You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2025-432c3df31b

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 18 Fedora Update System 2025-06-15 01:48:55 UTC
FEDORA-2025-899cbbd0c7 (faad2-2.11.2-3.fc42) has been pushed to the Fedora 42 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 19 Gary Buhrmaster 2025-06-16 02:25:42 UTC
Was this passed by fedora-legal?  Last I knew some of the AAC patents may not expire for a few more years.  fdk-aac was reviewed by legal for it's specific implementation(s), and approved, but that approval may not apply to faad2.

Comment 20 Fedora Update System 2025-06-16 03:46:31 UTC
FEDORA-2025-fdc965aa2c (faad2-2.11.2-3.fc41) has been pushed to the Fedora 41 stable repository.
If problem still persists, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.