Bug 492398

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

> $ pkg-config --libs slv2
> -lrdf -lrasqal -lraptor -lslv2  

> $ pkg-config --libs redland
> -lrdf -lrasqal -lraptor

The slv2.pc file is optimised for static linking in that it adds the @REDLAND_LIBS@ libs which the shared libslv2 is linked with already. Can be fixed in both packages.


> export CFLAGS="%{optflags}"
> export CPPFLAGS="%{optflags}"

CPPFLAGS duplicates the CFLAGS during compilation. You want to define CXXFLAGS instead, although this project is in plain C.


> slv2_plugininstance.h_r1948.diff

Does any slv2 header actually require definitions found within <stdlib.h>? Or is it just the plugins that ought to include stdlib.h instead? The comment in the spec doesn't explain this.



* Main slv2 package contains API html documentation, which belongs into the -devel package. EasyFix.


* Do we have any dependencies for SLV2 already to test with? [Audacity has dropped SLV2/Redland usage prior to releasing 1.3.7-beta]

Thank you for the review!

(In reply to comment #2)
> > $ pkg-config --libs slv2
> > -lrdf -lrasqal -lraptor -lslv2  
> 
> > $ pkg-config --libs redland
> > -lrdf -lrasqal -lraptor
> 
> The slv2.pc file is optimised for static linking in that it adds the
> @REDLAND_LIBS@ libs which the shared libslv2 is linked with already. Can be
> fixed in both packages.
> 

I removed @REDLAND_*@ from the .pc file

> 
> > export CFLAGS="%{optflags}"
> > export CPPFLAGS="%{optflags}"
> 
> CPPFLAGS duplicates the CFLAGS during compilation. You want to define CXXFLAGS
> instead, although this project is in plain C.
> 
> 

Sorry, I missed this. Fixed.

> > slv2_plugininstance.h_r1948.diff
> 
> Does any slv2 header actually require definitions found within <stdlib.h>? Or
> is it just the plugins that ought to include stdlib.h instead? The comment in
> the spec doesn't explain this.
> 
> 

It was me who reported this upstream. The file plugininstance.h refers to "NULL" which is undefined unless stdlib.h is included.

Ref: http://dev.drobilla.net/ticket/341

> 
> * Main slv2 package contains API html documentation, which belongs into the
> -devel package. EasyFix.
> 
> 

Done.

> * Do we have any dependencies for SLV2 already to test with? [Audacity has
> dropped SLV2/Redland usage prior to releasing 1.3.7-beta]  

You can re-compile ardour with SLV2 support. We will do this after this package hits to repos. Also there is traverso in rpmfusion that you can try out:
https://bugzilla.rpmfusion.org/show_bug.cgi?id=traverso


Spec URL: http://oget.fedorapeople.org/review/slv2.spec
SRPM URL: http://oget.fedorapeople.org/review/slv2-0.6.2-2.fc10.src.rpm

Changelog: 0.6.2-2
- Remove redland flags from the .pc file
- Change CPPFLAGS to CXXFLAGS
- Move API documentation to the -devel package

For NULL you would want to include <stddef.h> not <stdlib.h>

[...]

Will take a final look at the pkg tomorrow...hopefully.

* Would have kept %{_docdir}/%{name}-%{version} as the single docdir instead of creating a separate %{_docdir}/%{name}-devel-%{version} for the -devel pkg, but that's a matter of taste.

* Have restricted myself to retesting the slv2 executables with LV2 swh-plugins. Encountered a few error conditions with the JACK hosts and even a segfault, but that one is inside a plugin.

[...]

APPROVED: slv2-0.6.2-2.fc10.src.rpm

Thanks Michael! I'll do more tests and file tickets upstream if necessary.

New Package CVS Request
=======================
Package Name: slv2
Short Description: LV2 host library
Owners: oget
Branches: F-10
InitialCC:

cvs done.

slv2-0.6.2-2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/slv2-0.6.2-2.fc10

slv2-0.6.2-2.fc10 has been pushed to the Fedora 10 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update slv2'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-3234

slv2-0.6.2-3.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/slv2-0.6.2-3.fc10

slv2-0.6.2-3.fc10 has been pushed to the Fedora 10 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update slv2'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-3497

slv2-0.6.2-3.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.