https://bugzilla.redhat.com/show_bug.cgi?id=1359402 --- Comment #14 from Tim Fenn <tim.fenn@xxxxxxxxx> --- (In reply to Antonio Trande from comment #13) > Package Review > ============== > > Legend: > [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated > [ ] = Manual review needed > > > Issues: > ======= > - Package does not use a name that already exists. > Note: A package with this name already exists. Please check > https://admin.fedoraproject.org/pkgdb/package/coot > See: > > https://fedoraproject.org/wiki/Packaging/ > NamingGuidelines#Conflicting_Package_Names > > - Add 'Requires: lib%{name}%{?_isa} = %{version}-%{release}' to > libcoot-devel. > Done. > - Add 'Requires: python2-lib%{name}%{?_isa} = %{version}-%{release}' to > python2-libcoot-devel. > Done. > - 'coot-data' must be a noarch package. > Main package requires 'coot-data' as a noarch dependence. > > Requires: %{name}-data = %{version}-%{release} > Done. > - 'coot-doc' is a stand-alone package that must provide a license file. > Done. > - Following lines can be removed > > %post -n python2-libcoot > /sbin/ldconfig > > %postun -n python2-libcoot > /sbin/ldconfig > > - ldconfig scriptlets can be written in this way > > %post -n libcoot -p /sbin/ldconfig > %postun -n libcoot -p /sbin/ldconfig > Done. > - Please, remove those macros in the comments or silence them with %%. > Done. > - Fix strange-permission warnings (set them with 644) > > coot.src: W: strange-permission reference-structures.tar.gz 444 Fixed. > coot.src: W: strange-permission coot.in 755 > This is set to 755 as its processed by autoconf to generate the coot script that starts the main program. I've also discussed the testing of the build with upstream, and there are several files/custom scripts required to run the tests. I've asked upstream to consider streamlining this in some way, but for now I've disabled the "make check", since by itself it does nothing. Also, the package was updated to version 0.8.7 (which fixes some of the smaller issues, such as removing AM_PROG_CC_STDC). https://timfenn.fedorapeople.org/coot.spec https://timfenn.fedorapeople.org/coot-0.8.7-1.fc24.src.rpm -- 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 -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx