Comment # 26 on bug 2736 from Sérgio Basto

fedora-review --other-bz https://bugzilla.rpmfusion.org -b 2736 -m
fedora-rawhide-x86_64-rpmfusion_nonfree

no issues , just not sure is documentation aacDecoder.pdf and aacEncoder.pdf
should be in devel package , at least in -doc sub package ... 


Rpmlint (debuginfo)
-------------------
Checking: fdk-aac-debuginfo-0.1.5-0.1.gita0bd8aa.fc26.x86_64.rpm
fdk-aac-debuginfo.x86_64: W: invalid-license FDK-AAC
1 packages and 0 specfiles checked; 0 errors, 1 warnings.


Rpmlint (installed packages)
----------------------------
fdk-aac-debuginfo.x86_64: W: invalid-license FDK-AAC
fdk-aac-devel.x86_64: W: invalid-license FDK-AAC
fdk-aac-devel.x86_64: W: only-non-binary-in-usr-lib
fdk-aac.x86_64: W: spelling-error Summary(en_US) Fraunhofer -> Fraudster
fdk-aac.x86_64: W: invalid-license FDK-AAC
3 packages and 0 specfiles checked; 0 errors, 5 warnings.


invalid license is not an problem, only-non-binary-in-usr-lib usr/lib have a
symlink correctly and other warnings can be ignored.

fdk-aac.spec looks good , source field could be more simple :
https://github.com/mstorsjo/%{name}/archive/%{commit0}/%{name}-%{shortcommit0}.tar.gz

So just need clarify documentation and I will approve it .


Thanks