Bug 562467

*** Bug 538297 has been marked as a duplicate of this bug. ***

The spec file looks generally clean, with a few nitpicks :
- the -b 0 argument to %setup is useless.
- Why no parallel build ? ("make" rather than "make %{?_smp_mflags}"). If it's failing then that should be added as a comment into the spec.
- the .gz at the end of the manfiles lines in %file should be changed to .* in case the compression format ever change.

Actually, my biggest concern about this review request is it should not be a review request in the first place as openvas-libraries is already packaged. I suggest you submit a patch against the devel branch to Huzaifa, and also that you apply for co-maintainership on openvas-libraries. I'm adding him to the CC list.

I cleaned up spec file and synchronized the libraries with a new upstream version.

As for review request, I did it this way because it was easier for me. I think I am co-maintainer for some parts of the openvas, though I don't know which. Despite this, I didn't want to try to change OpenVAS since it is quite large change in question.

Spec URL:
http://www.zemris.fer.hr/~sgros/stuff/fedora/openvas3/openvas-libraries.spec
SRPM URL:
http://www.zemris.fer.hr/~sgros/stuff/fedora/openvas3/openvas-libraries-3.0.3-1.fc12.src.rpm
Description:

Anyway, Huzaifa is free to do or suggest anything he thinks is the best.

Thanks, i will commit it soon,
I have a having issues with static libs since --disable-static was not working.


http://bugs.openvas.org/855
http://bugs.openvas.org/1125

Are you sure the parser is linked with the dynamic libs and not the static ones, which you are deleting?

I missed that openvas-libnasl has entered the distro. As libnasl has been incorporated in the main libraries package with openvas 3, I think some obsolotes/provides should be added to ensure a clean update path.

openvas-libraries should have :

Obsoletes: openvas-libnasl < 2.0.2-4
Provides: openvas-libnasl = %{version}-%{release}

and openvas-libraries-devel :

Obsoletes: openvas-libnasl-devel < 2.0.2-4
Provides: openvas-libnasl-devel = %{version}-%{release}

Actually openvas-libnasl has not been built yet, so i can just mark this as a dead package.

(In reply to comment #5)
> I missed that openvas-libnasl has entered the distro. As libnasl has been
> incorporated in the main libraries package with openvas 3, I think some
> obsolotes/provides should be added to ensure a clean update path.
> 
> openvas-libraries should have :
> 
> Obsoletes: openvas-libnasl < 2.0.2-4
> Provides: openvas-libnasl = %{version}-%{release}
> 
> and openvas-libraries-devel :
> 
> Obsoletes: openvas-libnasl-devel < 2.0.2-4
> Provides: openvas-libnasl-devel = %{version}-%{release}

openvas-libraries-3.0.3 is built for rawhide now, closing this bug.

Hello Huzaifa,
please - as there is full set of openvas packages available now (libraries, scanner, client), could you consider submitting the openvas-libraries to EL-5 and EL-6 branches so openvas can be build for those platforms as well?

It would be also great if you could include sgros xavierb and me (rebus) as cvs comaintainers of the package so we can submit the build to koji/bodhi.

Thank you
Michal Ambroz

All 4 of us are maintainers of the openvas scanner and client packages, it definitely makes sense to be maintainers of the libs as well, so here's the branch requests for EPEL. Everyone willing to have rights on the Fedora branches will have to apply for them in the pkgdb and Huzaifa will have to approve them.

Package Change Request
======================
Package Name: openvas-libraries
New Branches: EL-5 EL-6
Owners: huzaifas sgros rebus xavierb

CVS done (by process-cvs-requests.py).

Package Change Request
======================
Package Name: openvas-libraries
New Branches: epel7
Owners: huzaifas sgros rebus xavierb fab

Hello SCM team,
plase can you add epel7 branch for the openvas-libraries package?
Thank you
Michal Ambroz

Git done (by process-git-requests).