[Bug 2736] Review Request: fdk-aac - Fraunhofer FDK AAC Codec Library

[RPM Fusion Bugzilla]

http://bugzilla.rpmfusion.org/show_bug.cgi?id=2736

--- Comment #28 from Nicolas Chauvet <kwizart at gmail.com> ---
(In reply to Sérgio Basto from comment #26)
> 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 ... 
Theses files are developers documentation of the library, the packaging
guideline doesn't mandate from which size a sub-package should be made, but
since it's already in a -devel I think it's fine to have them in -devel. (it's
less than 1Mo) 

> fdk-aac.spec looks good , source field could be more simple :
> https://github.com/mstorsjo/%{name}/archive/%{commit0}/%{name}-
> %{shortcommit0}.tar.gz
Fixed this one locally along with using Source0 instead of Source:

Thx for the review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.rpmfusion.org/pipermail/rpmfusion-developers/attachments/20160908/f70e8368/attachment.html>