https://bugzilla.redhat.com/show_bug.cgi?id=2177305
--- Comment #8 from Kaleb KEITHLEY kkeithle@redhat.com --- (In reply to Jakub Kadlčík from comment #7)
Thank you for the package Kaleb, overall it looks good.
I'd change the Url to https://github.com/civetweb/civetweb but that's a minor thing.
Summary: Civetweb Client Library C and C++ header files Group: Development/Libraries Requires: %{name}%{?_isa} = %{version}-%{release}
Nitpick: Inconsistent indenting
%defattr(-,root,root,-)
I think this is a default and can be deleted
%{_includedir}/* %{_libdir}/cmake/*
Can you make it more specific, please?
Group: Development/Libraries
According to the packaging guidelines, Group tag shouldn't be used anymore https://docs.fedoraproject.org/en-US/packaging-guidelines/#_tags_and_section...
All of the above have been addressed in the new src.rpm (and .spec) at https://kkeithle.fedorapeople.org/civetweb/civetweb-1.15-1.fc39.src.rpm
[ ]: Package must own all directories that it creates. Note: Directories without known owners: /usr/lib/debug/.build-id/8a, /usr/lib/debug/.build-id/56, /usr/lib/debug/.build-id/f3
Can you please take a look at this one?
Those are from the debuginfo build, right? I've never done anything for those in any of the other packages I've added. E.g. last year I packaged liborc and libarrow and didn't do anything to exclude debuginfo in those.
If you can tell me what I'm supposed to do I'll do it, otherwise I haven't got a clue. (No help here https://docs.fedoraproject.org/en-US/packaging-guidelines/Debuginfo/ that I can see.)