Spec URL: http://www.coolbits.it/fedora/libpasastro.spec SRPM URL: http://www.coolbits.it/fedora/libpasastro-1.0-2.20151219svn.fc23.src.rpm Description: libpasastro provides shared libraries to interface Pascal programs with standard astronomy libraries. It provides common code to programs like Skychart and Virtual Planets Atlas. Fedora Account System Username: mattia
mattia's scratch build of libpasastro-1.0-2.20151219svn.fc23.src.rpm for f24 completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12255912
naming: ok license: ok scriptlets: ok Arches: NOT ok. Looks like fpc has ExclusiveArch: %{arm} %{ix86} x86_64 ppc ppc64 and lazaras: ExcludeArch: s390 s390x I'd guess those should match closer, but in the least the .spec comment is inaccurate: # fpc and lazarus are not working on s390, ppc and aarch64 1. MUST: either clarify this comment or adjust to match fpc's ExclusiveArch: %{arm} %{ix86} x86_64 ppc ppc64 sources: NOT ok, not verifiable 2. MUST give better instructions to create reproducible sources or justify why not using upstream tarball source, http://sourceforge.net/projects/libpasastro/files/version%201.0/libpasastro-1.0-src.tar.xz/download 3. SHOULD replace fragile/manual creation of library soname links: #ln -fs %{_libdir}/libpasplan404.so.%{sover}.0 %{buildroot}/%{_libdir}/libpasplan404.so.%{sover} #ln -fs %{_libdir}/libpasgetdss.so.%{sover}.0 %{buildroot}/%{_libdir}/libpasgetdss.so.%{sover} #ln -fs %{_libdir}/libpaswcs.so.%{sover}.0 %{buildroot}/%{_libdir}/libpaswcs.so.%{sover} with: /sbin/ldconfig -n %{buildroot}%{_libdir} macros: NOT ok 4. MUST use standard $RPM_OPT_FLAGS/%{optflags}, currently uses hard-coded mixture of "-O3 -fPIC -g" and "-fPIC -g"
As an aside, as far as I can tell neither lazarus or fpc are actually used during the build process for this package, and it seems to build ok without them being present in the buildroot, so. 5. SHOULD verify BuildRequires: fpc lazarus is really needed (and the accompanying ExcludeArch/ExclusiveArch tag).
Spec URL: http://www.coolbits.it/fedora/libpasastro.spec SRPM URL: http://www.coolbits.it/fedora/libpasastro-1.0-5.20151222svn.fc23.src.rpm Ok, I've removed fpc and lazarus BR, so ExcludeArch tag is no more necessary. I've fixed the soname links creation and passed CFLAGS and CXXFLAGS in make by using %{optflags}, I guess it can now be the right way for that. About sources, I use svn and not the package available upstream because I asked the developer to fix some things for packaging into Fedora and these things are available only in svn at the moment. I clarified that a bit in .spec file, tell me if I need to specify something more. Thanks
Thanks, looks good, approved.
Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/libpasastro
libpasastro-1.0-5.20151222svn.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-f874adea82
libpasastro-1.0-5.20151222svn.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-f874adea82
libpasastro-1.0-5.20151222svn.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report.