https://bugzilla.rpmfusion.org/show_bug.cgi?id=4441

--- Comment #12 from Nicolas Chauvet <kwiz...@gmail.com> ---
(In reply to Andrew Bauer from comment #11)
...
> We should probably add the requires shown in this spec file, but add them
> one per line please:
> https://github.com/RPM-Outpost/discord/blob/master/discord-stable.spec
> Gives props to the electronwill for figuring that part out.
Please reminds to use the %{_isa} notation for each dependencies (one by line
for better readability). 
Some dependencies can be automatically detected if you remove AutoReqProv:   
no
Also BuildRequires:  libXScrnSaver is probably Requires: libXScrnSaver%{_isa}
instead.

Please use desktop-file-validate
https://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-install_usage

Also please consider to provide appdata files
https://fedoraproject.org/wiki/Packaging:AppData

As this is a pre-built binary package, you need to use both of theses macros:
---
%global        debug_package %{nil}
%global        __strip /bin/true
---

> Note, that I'd send an email to Chris Marsh regarding the license question.
Thx!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
_______________________________________________
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

Reply via email to