Bug 1293049

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.