Bug 2389958

Summary: Review Request: fdkaac - command line frontend for libfdk-aac encoder
Product: [Fedora] Fedora Reporter: Dominik 'Rathann' Mierzejewski <dominik>
Component: Package ReviewAssignee: Neal Gompa <ngompa13>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: ngompa13, package-review
Target Milestone: ---Keywords: AutomationTriaged
Target Release: ---Flags: ngompa13: fedora-review+
Hardware: All   
OS: Linux   
URL: https://github.com/nu774/fdkaac
Whiteboard:
Fixed In Version: Doc Type: ---
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2026-04-20 00:44:56 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 2218117    
Attachments:
Description Flags
The .spec file difference from Copr build 9451688 to 10094561 none

Description Dominik 'Rathann' Mierzejewski 2025-08-20 22:33:59 UTC
Spec URL: https://rathann.fedorapeople.org/review/fdkaac/fdkaac.spec
SRPM URL: https://rathann.fedorapeople.org/review/fdkaac/fdkaac-1.0.6-5.fc44.src.rpm
Description:
fdkaac reads linear PCM audio in either WAV, raw PCM, or CAF format,
and encodes it into either M4A / AAC file.

If the input file is "-", data is read from stdin. Likewise, if the
output file is "-", data is written to stdout if one of streamable AAC
transport formats are selected by -f.

When CAF input and M4A output is used, tags in CAF file are copied into
the resulting M4A.

Fedora Account System Username: rathann

Comment 1 Fedora Review Service 2025-08-20 22:40:27 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/9451688
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2389958-fdkaac/fedora-rawhide-x86_64/09451688-fdkaac/fedora-review/review.txt

Please take a look if any issues were found.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 2 Neal Gompa 2025-08-27 02:14:37 UTC
Taking this review.

Comment 3 Neal Gompa 2025-08-27 02:16:15 UTC
Spec review notes:

> %build
> autoreconf -fiv
> export LDFLAGS="%{?__global_ldflags} -L%{_libdir}/fdk-aac"
> %configure

This should be in %conf, and instead of messing with "$LDFLAGS", it's better to update %build_ldflags instead.

Comment 4 Dominik 'Rathann' Mierzejewski 2026-02-04 22:29:30 UTC
Spec URL: https://rathann.fedorapeople.org/review/fdkaac/fdkaac.spec
SRPM URL: https://rathann.fedorapeople.org/review/fdkaac/fdkaac-1.0.6-6.fc44.src.rpm

- update build_ldflags instead of messing with LDFLAGS

Comment 5 Fedora Review Service 2026-02-04 22:34:54 UTC
Created attachment 2128224 [details]
The .spec file difference from Copr build 9451688 to 10094561

Comment 6 Fedora Review Service 2026-02-04 22:35:04 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/10094561
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2389958-fdkaac/fedora-rawhide-x86_64/10094561-fdkaac/fedora-review/review.txt

Please take a look if any issues were found.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 7 Dominik 'Rathann' Mierzejewski 2026-04-09 08:56:01 UTC
Anything else to fix or is this ready to be approved?

Comment 8 Neal Gompa 2026-04-09 13:59:12 UTC
Review notes:

* Package follows Fedora packaging guidelines
* Package builds and installs
* Package licensing is correctly marked and license data is installed
* No serious rpmlint issues

PACKAGE APPROVED.

Comment 9 Fedora Admin user for bugzilla script actions 2026-04-10 07:17:53 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/fdkaac

Comment 10 Fedora Update System 2026-04-10 07:56:14 UTC
FEDORA-2026-b26110ebfd (fdkaac-1.0.6-6.fc42) has been submitted as an update to Fedora 42.
https://bodhi.fedoraproject.org/updates/FEDORA-2026-b26110ebfd

Comment 11 Fedora Update System 2026-04-10 07:56:15 UTC
FEDORA-2026-bb32c80cd0 (fdkaac-1.0.6-6.fc43) has been submitted as an update to Fedora 43.
https://bodhi.fedoraproject.org/updates/FEDORA-2026-bb32c80cd0

Comment 12 Fedora Update System 2026-04-10 07:56:15 UTC
FEDORA-2026-aed7714b25 (fdkaac-1.0.6-6.fc44) has been submitted as an update to Fedora 44.
https://bodhi.fedoraproject.org/updates/FEDORA-2026-aed7714b25

Comment 13 Fedora Update System 2026-04-12 16:12:07 UTC
FEDORA-2026-b26110ebfd 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-2026-b26110ebfd \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2026-b26110ebfd

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

Comment 14 Fedora Update System 2026-04-12 16:34:38 UTC
FEDORA-2026-aed7714b25 has been pushed to the Fedora 44 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2026-aed7714b25 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2026-aed7714b25

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

Comment 15 Fedora Update System 2026-04-12 16:54:14 UTC
FEDORA-2026-bb32c80cd0 has been pushed to the Fedora 43 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --refresh --advisory=FEDORA-2026-bb32c80cd0 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2026-bb32c80cd0

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

Comment 16 Fedora Update System 2026-04-20 00:44:56 UTC
FEDORA-2026-bb32c80cd0 (fdkaac-1.0.6-6.fc43) has been pushed to the Fedora 43 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 17 Fedora Update System 2026-04-20 01:04:33 UTC
FEDORA-2026-b26110ebfd (fdkaac-1.0.6-6.fc42) has been pushed to the Fedora 42 stable repository.
If problem still persists, please make note of it in this bug report.