https://bugzilla.redhat.com/show_bug.cgi?id=833511

--- Comment #3 from Terje Røsten <[email protected]> ---
Thanks, some more items.

- seems to mix tab and whitespace
- this is two buildrequires on one line: BuildRequires:    cmake python2 :-)
- file listing is recursive by default and you need to own the dir itself,
change 
   %{python_sitelib}/dnf/* to  %{python_sitelib}/dnf/
- with only %config(noreplace) %{confdir}/dnf.conf,  %{confdir} ends up
unowned, add
 a %dir  %{confdir} line to file listing.
- use %global not %define
- still %dist in changelogs?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to