Fedora Merge Review: gnome-media http://cvs.fedora.redhat.com/viewcvs/devel/gnome-media/ Initial Owner: alexl
A couple of issues for gnome-media; NEEDSWORK: * rpmlint (on the binary package, the -devel subpackage, and the srpm) produces a number of warnings/errors. Most of it can possibly be ignored, but I think the rpath issue need to be fixed >> [deji@agape reviews]$ rpmlint gnome-media-2.17.90-1.fc7.x86_64.rpm E: gnome-media tag-not-utf8 %changelog E: gnome-media obsolete-not-provided gnome E: gnome-media binary-or-shlib-defines-rpath /usr/lib64/libglade/2.0/libgnome-media-profiles.so ['/usr/lib64'] E: gnome-media binary-or-shlib-defines-rpath /usr/bin/gnome-audio-profiles-properties ['/usr/lib64'] E: gnome-media binary-or-shlib-defines-rpath /usr/bin/gnome-cd ['/usr/lib64'] E: gnome-media binary-or-shlib-defines-rpath /usr/libexec/cddb-track-editor ['/usr/lib64'] W: gnome-media non-conffile-in-etc /etc/gconf/schemas/CDDB-Slave2.schemas W: gnome-media non-conffile-in-etc /etc/gconf/schemas/gnome-audio-profiles.schemas W: gnome-media non-conffile-in-etc /etc/gconf/schemas/gnome-volume-control.schemas W: gnome-media non-conffile-in-etc /etc/gconf/schemas/gnome-cd.schemas [deji@agape reviews]$ rpmlint gnome-media-devel-2.17.90-1.fc7.x86_64.rpm E: gnome-media-devel tag-not-utf8 %changelog W: gnome-media-devel no-documentation [deji@agape reviews]$ rpmlint gnome-media-2.17.90-1.fc7.src.rpm E: gnome-media tag-not-utf8 %changelog E: gnome-media non-utf8-spec-file gnome-media.spec W: gnome-media unversioned-explicit-obsoletes gnome W: gnome-media macro-in-%changelog _datadir W: gnome-media mixed-use-of-spaces-and-tabs (spaces: line 12, tab: line 22) * scrollkeeper-update is not being properly called in the post and postun sections according to http://fedoraproject.org/wiki/Packaging/ScriptletSnippets?action=show&redirect=ScriptletSnippets#head-a4ea5e1946bc113d19d24b4f5bfb543c579e5fc8 * "--add-category X-Redhat-Base" option to desktop-file-install is no longer necesary Not so importantly, the "disable-schemas-install" option can be pass to the configure script instead of setting and unsetting GCONF_DISABLE_MAKEFILE_SCHEMA_INSTAL env.
The packaging guildelines on removing rpath is here; http://fedoraproject.org/wiki/Packaging/Guidelines?highlight=%28rpath%29#head-7cea8c7aa96400a4687e843156354476434ff883
(In reply to comment #1) > A couple of issues for gnome-media; > > NEEDSWORK: > * rpmlint (on the binary package, the -devel subpackage, and the srpm) produces > a number of warnings/errors. Most of it can possibly be ignored, but I think the > rpath issue need to be fixed > >> > [deji@agape reviews]$ rpmlint gnome-media-2.17.90-1.fc7.x86_64.rpm > E: gnome-media tag-not-utf8 %changelog Fixed > E: gnome-media obsolete-not-provided gnome > E: gnome-media binary-or-shlib-defines-rpath > /usr/lib64/libglade/2.0/libgnome-media-profiles.so ['/usr/lib64'] > E: gnome-media binary-or-shlib-defines-rpath > /usr/bin/gnome-audio-profiles-properties ['/usr/lib64'] > E: gnome-media binary-or-shlib-defines-rpath /usr/bin/gnome-cd ['/usr/lib64'] > E: gnome-media binary-or-shlib-defines-rpath /usr/libexec/cddb-track-editor > ['/usr/lib64'] > W: gnome-media non-conffile-in-etc /etc/gconf/schemas/CDDB-Slave2.schemas > W: gnome-media non-conffile-in-etc /etc/gconf/schemas/gnome-audio-profiles.schemas > W: gnome-media non-conffile-in-etc /etc/gconf/schemas/gnome-volume-control.schemas > W: gnome-media non-conffile-in-etc /etc/gconf/schemas/gnome-cd.schemas Fixed. > [deji@agape reviews]$ rpmlint gnome-media-devel-2.17.90-1.fc7.x86_64.rpm > E: gnome-media-devel tag-not-utf8 %changelog > W: gnome-media-devel no-documentation > [deji@agape reviews]$ rpmlint gnome-media-2.17.90-1.fc7.src.rpm > E: gnome-media tag-not-utf8 %changelog > E: gnome-media non-utf8-spec-file gnome-media.spec Fixed. > W: gnome-media unversioned-explicit-obsoletes gnome > W: gnome-media macro-in-%changelog _datadir Fixed. > W: gnome-media mixed-use-of-spaces-and-tabs (spaces: line 12, tab: line 22) Fixed. > * scrollkeeper-update is not being properly called in the post and postun > sections according to > http://fedoraproject.org/wiki/Packaging/ScriptletSnippets?action=show&redirect=ScriptletSnippets#head-a4ea5e1946bc113d19d24b4f5bfb543c579e5fc8 Fixed > * "--add-category X-Redhat-Base" option to desktop-file-install is no longer > necesary That's commented. > Not so importantly, the "disable-schemas-install" option can be pass to the > configure script instead of setting and unsetting > GCONF_DISABLE_MAKEFILE_SCHEMA_INSTAL env. Fixed. All of this is fixed in gnome-media-2.17.90-3.fc7, I'll need to work on the rpath issue now.
And fixed the rpath issue in gnome-media-2.17.90-4.fc7. Let me know if anything else needs work.
And will really be fixed in gnome-media-2.17.90-5.fc7, I forgot to *actually* apply the configure changes...
I was looking at the cvs commit mail, and it seems you forgot to actually commit the change ;). I'm waiting for anonymous cvs to sync up so I can properly test it.
The rpath issue is still present; [deji@agape reviews]$ rpmlint gnome-media-2.17.90-5.fc7.x86_64.rpm E: gnome-media tag-not-utf8 %changelog E: gnome-media obsolete-not-provided gnome E: gnome-media binary-or-shlib-defines-rpath /usr/lib64/libglade/2.0/libgnome-media-profiles.so ['/usr/lib64'] E: gnome-media binary-or-shlib-defines-rpath /usr/bin/gnome-audio-profiles-properties ['/usr/lib64'] E: gnome-media binary-or-shlib-defines-rpath /usr/bin/gnome-cd ['/usr/lib64'] E: gnome-media binary-or-shlib-defines-rpath /usr/libexec/cddb-track-editor ['/usr/lib64'] W: gnome-media conffile-without-noreplace-flag /etc/gconf/schemas/CDDB-Slave2.schemas W: gnome-media conffile-without-noreplace-flag /etc/gconf/schemas/gnome-audio-profiles.schemas W: gnome-media conffile-without-noreplace-flag /etc/gconf/schemas/gnome-cd.schemas W: gnome-media conffile-without-noreplace-flag /etc/gconf/schemas/gnome-volume-control.schemas The problem is caused by the package using it's local copy of libtool, as can be seen from the build log (excerpt below). I believe this can be fixed by having a buildrequire on libtool and forcing the build to use the system libtool ("make LIBTOOL=/usr/bin/libtool ...") ( /bin/sh ../libtool --mode=install /usr/bin/install -c 'cddb-slave2-properties' '/var/tmp/gnome-media-2.17.90-5.fc7-root-mockbuild/usr/bin/cddb-slave2-properties' /usr/bin/install -c cddb-slave2-properties /var/tmp/gnome-media-2.17.90-5.fc7-root-mockbuild/usr/bin/cddb-slave2-properties test -z "/usr/libexec" || mkdir -p -- "/var/tmp/gnome-media-2.17.90-5.fc7-root-mockbuild/usr/libexec" /bin/sh ../libtool --mode=install /usr/bin/install -c 'CDDBSlave2' '/var/tmp/gnome-media-2.17.90-5.fc7-root-mockbuild/usr/libexec/CDDBSlave2' /usr/bin/install -c CDDBSlave2 /var/tmp/gnome-media-2.17.90-5.fc7-root-mockbuild/usr/libexec/CDDBSlave2 /bin/sh ../libtool --mode=install /usr/bin/install -c 'cddb-track-editor' '/var/tmp/gnome-media-2.17.90-5.fc7-root-mockbuild/usr/libexec/cddb-track-editor' ) Probably an upstream issue, I noticed the install section of the build doesn't respect the configure option to disable scrollkeeper; it still variously attempts to update the scrollkeeper.
I checked with --disable-rpath on i386, and it didn't happen. I'll try and work on that upstream instead for now.
That explains it, I was testing on x86_64
I've made some local changes, but the gist of it is that we need to force a new libtool in the tarball to get rid of the rpath warnings. As for the scrollkeeper bits, we'll need to remove them ourselves from the omf.make as well, as gnome-media doesn't use gnome-doc-utils, which provides and respects the --disable-scrollkeeper argument: http://bugzilla.gnome.org/show_bug.cgi?id=405821
The simple-minded approach to the scrollkeeper problem is to just rm -rf $RPM_BUILD_ROOT/var/scrollkeeper in %build.
(In reply to comment #11) > The simple-minded approach to the scrollkeeper problem is to just > rm -rf $RPM_BUILD_ROOT/var/scrollkeeper in %build. Actually, it tries to update the database in /var/scrollkeeper, not $RPM_BUILD_ROOT/var/scrollkeeper... It's all built in gnome-media-2.17.90-6.fc7
GOOD: * Build Ok in mock (x86_64) * License (GPL) and rpm Group tag OK * Naming meets the packaging guildlines * Spec file clean and legible * Handles locales correctly * Build dependencies properly listed * Source file matches upstream's 094a92f01a3581de7b3567f85320df7b gnome-media-2.17.90.tar.bz2 * Own its files and directories correctly rpmlint warnings/error: This can mostly be ignored [deji@agape reviews]$ rpmlint gnome-media-2.17.90-6.fc7.x86_64.rpm E: gnome-media obsolete-not-provided gnome W: gnome-media conffile-without-noreplace-flag /etc/gconf/schemas/CDDB-Slave2.schemas W: gnome-media conffile-without-noreplace-flag /etc/gconf/schemas/gnome-audio-profiles.schemas W: gnome-media conffile-without-noreplace-flag /etc/gconf/schemas/gnome-cd.schemas W: gnome-media conffile-without-noreplace-flag /etc/gconf/schemas/gnome-volume-control.schemas [deji@agape reviews]$ rpmlint gnome-media-2.17.90-6.fc7.src.rpm W: gnome-media unversioned-explicit-obsoletes gnome [deji@agape reviews]$ rpmlint gnome-media-devel-2.17.90-6.fc7.x86_64.rpm W: gnome-media-devel no-documentation However I wonder if it's not just OK to drop that Obsoletes on gnome, it seems superfluous of gnome-media to Obsolete gnome as it cannot also provide it (gnome). I believe you can fix that, so APPROVED. APPROVED.
This review is done.