Bug 225822

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.