Spec URL: http://people.redhat.com/green/FE/FC5/fluidsynth-dssi.spec SRPM URL: http://people.redhat.com/green/FE/FC5/fluidsynth-dssi-0.9.1-2.src.rpm Description: This is an implementation of the FluidSynth soundfont-playing software synthesizer as a DSSI plugin. It makes use of DSSI's run_multiple_synths() interface to allow sharing of resources between multiple plugin instances -- soundfont data is shared between instances, and FluidSynth's usual voice allocation methods are applied across multiple instances as if each were a FluidSynth channel.
I guess to avoid rpmlint symlink errors/warnings you can change ln -s %{_bindir}/jack-dssi-host $RPM_BUILD_ROOT%{_bindir}/fluidsynth-dssi to cd %{_bindir} ln -s jack-dssi-host fluidsynth-dssi cd - because there is no need to make the symlinks across the filesystem if source and target are in the same directory. This concerns also your other dssi plugin(s).
The specfile uses "make" instead of "make %{?_smp_mflags}"
Thanks - I've updated the SRPM here: Spec URL: http://people.redhat.com/green/FE/FC5/fluidsynth-dssi.spec SRPM URL: http://people.redhat.com/green/FE/FC5/fluidsynth-dssi-0.9.1-3.src.rpm
The .desktop file should be in a separate file and not inside the spec. You do not need: Requires(post): desktop-file-utils Requires(postun): desktop-file-utils make install DESTDIR="$RPM_BUILD_ROOT" instead of %makeinstall should work, so you must not use %makeinstall see: http://fedoraproject.org/wiki/Packaging/Guidelines#head-fcaf3e6fcbd51194a5d0dbcfbdd2fcb7791dd002 You do not package COPYING with contains the license. The license seems to be GPL but you wrote LGPL in the spec.
(In reply to comment #4) > The .desktop file should be in a separate file and not inside the spec. I decided to delete the .desktop file, as well as the fluidsynth-dssi binary, since it is redundant. FE already has many other ways to run fluidsynth (qsynth, etC). > make install DESTDIR="$RPM_BUILD_ROOT" > instead of %makeinstall should work, so you must not use %makeinstall Fixed. > You do not package COPYING with contains the license. Fixed. > The license seems to be GPL but you wrote LGPL in the spec. Fixed. Updated bits here: Spec URL: http://people.redhat.com/green/FE/FC5/fluidsynth-dssi.spec SRPM URL: http://people.redhat.com/green/FE/FC5/fluidsynth-dssi-0.9.1-4.src.rpm
BuildRequires: alsa-lib-devel liblo-devel ladspa-devel is not needed since dssi-devel already depends on them and is in BuildRequires rpmlint: ok naming: ok packaging guidelines: ok license: ok sources: ok 06bce40ec6c86545d0587e9364bba116 fluidsynth-dssi-0.9.1.tar.gz 06bce40ec6c86545d0587e9364bba116 fluidsynth-dssi-0.9.1.tar.gz.1 files and directories: ok mock build: ok Change the buildrequires and the package is APPROVED
(In reply to comment #6) > Change the buildrequires and the package is > > APPROVED Done, thanks!