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



--- Comment #2 from Michael Schwendt (Fedora Packager Sponsors Group) 
<[email protected]> ---
> Summary: RADIUS protocol client library
> Group: Applications/Internet

The Group tag for base runtime libraries is "System Environment/Libraries" for
many years.

Nowadays the tag is obsolete:
https://fedoraproject.org/wiki/Packaging:Guidelines#Group_tag


> Conflicts: freeradius-client-devel, radiusclient-ng-devel

https://fedoraproject.org/wiki/Packaging:Guidelines#Conflicts
 -> https://fedoraproject.org/wiki/Packaging:Conflicts
  -> https://fedoraproject.org/wiki/Packaging:Conflicts#Header_Name_Conflicts
  ->
https://fedoraproject.org/wiki/Packaging:Conflicts#Other_Uses_of_Conflicts:


> %dir %{_sysconfdir}/radcli
> %config(noreplace) %{_sysconfdir}/%{name}/radiusclient.conf

Why do you spell out "radcli" in one line but use %{name} elsewhere?


> %files devel
> 
> %{_includedir}/freeradius-client.h
> %{_includedir}/radiusclient-ng.h
> %{_includedir}/radcli/radcli.h
> %{_includedir}/radcli/version.h
> %{_libdir}/libfreeradius-client.so
> %{_libdir}/libradiusclient-ng.so
> %{_libdir}/libradcli.so
> %{_mandir}/man3/*
> %{_libdir}/pkgconfig/*.pc

https://fedoraproject.org/wiki/Packaging:Guidelines#File_and_Directory_Ownership

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to