Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 230586 - Review Request: xdg-user-dirs-gtk - gnome/gtk+ integration of xdg-user-dirs
Summary: Review Request: xdg-user-dirs-gtk - gnome/gtk+ integration of xdg-user-dirs
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: manuel wolfshant
QA Contact: Fedora Package Reviews List
Depends On:
TreeView+ depends on / blocked
Reported: 2007-03-01 17:17 UTC by Alexander Larsson
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2007-03-05 10:13:47 UTC
wolfy: fedora-review+
wtogami: fedora-cvs+

Attachments (Terms of Use)

Description Alexander Larsson 2007-03-01 17:17:21 UTC
Spec URL:
Contains some integration of xdg-user-dirs with the gnome
desktop, including creating default bookmarks and detecting
locale changes.

review bug for xdg-user-dirs in bug 230585

Comment 1 manuel wolfshant 2007-03-01 19:10:37 UTC
Build in mock/devel/x86_64 failed:

checking for xgettext... /usr/bin/xgettext
checking for intltool >= 0.35.0... 0.35.5 found
checking for perl... /usr/bin/perl
checking for XML::Parser... configure: error: XML::Parser perl module is
required for intltool
error: Bad exit status from /var/tmp/rpm-tmp.11780 (%build)

Comment 2 manuel wolfshant 2007-03-01 19:15:11 UTC
another stop after that:
checking for x86_64-redhat-linux-gnu-pkg-config... (cached) no
checking pkg-config is at least version 0.9.0... ./configure: line 6758: no:
command not found
checking for GTK... configure: error: The pkg-config script could not be found
or is too old.  Make sure it
is in your PATH or set the PKG_CONFIG environment variable to the full
path to pkg-config.

Comment 3 manuel wolfshant 2007-03-01 19:20:24 UTC
And the show goes on...
checking for x86_64-redhat-linux-gnu-pkg-config... (cached) /usr/bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
checking for GTK... configure: error: Package requirements (gtk+-2.0) were not met:

No package 'gtk+-2.0' found

Comment 4 manuel wolfshant 2007-03-01 19:41:15 UTC
That was it. gtk2-devel, pkgconfig and perl-XML-Parser seem to be needed.
After adding those the package builds in mock, yielding
[wolfy@wolfy xdg-user-dirs-gtk]$ rpmlint ~/xdg-user-dirs-gtk-0.1*rpm
[wolfy@wolfy xdg-user-dirs-gtk]$ rpmlint xdg-user-dirs-gtk-0.1-1.fc7.x86_64.rpm
W: xdg-user-dirs-gtk non-conffile-in-etc

Please submit a fixed spec/rpm and I'll do the review.

Comment 6 manuel wolfshant 2007-03-02 10:41:01 UTC

- rpmlint checks do not return anything either on source or on binary package
- package meets naming guidelines
- package meets packaging guidelines
- license ( GPL ) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream, sha1sum 
09ccb33dd147d969ec33a9e3d08daa1ca0c37202  xdg-user-dirs-gtk-0.1.tar.gz
- package compiles on devel (x86_64)
- no missing BR
- MINOR: unnecessary BR: pkgconfig is pulled in by gtk2-devel
- no locales, despite what the build log seems to say
- not relocatable
- owns all files/directories that it creates, does not take ownership of foreign
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- MUSTFIX the .desktop file does not contain all needed entries:
/usr/bin/desktop-file-validate *desktop
user-dirs-update-gtk.desktop: error: required key "Type" not found
user-dirs-update-gtk.desktop: warning: non-standard key "OnlyShownIn" lacks the
"X-" prefix
According to there
are a few missing entries
- MUSTFIX: still according to, "It is not simply
enough to just include the .desktop file in the package, one MUST run
desktop-file-install in %install (and have BuildRequires: desktop-file-utils),
to help ensure .desktop file safety and spec-compliance."
- no .la, .pc, static content
- devel not needed


- builds without problems in mock/devel/x86_64
- does not segfault at run. I am not a Gnome user, so I cannot really test if it
creates what it supposed to create; a dry run in strace seems to appear OK

Please fix the desktop issues (I suggest either a patch or going with the
simpler version of a separate file included as Source1) and I will approve the
package. You might also remove pkgconfig from BR if you wish since it's not
really needed to be included separately.

Comment 7 Alexander Larsson 2007-03-02 12:03:11 UTC
I fixed the desktop file upstream and made a new release:

However, i disagree with using desktop-file-install. I'm using a desktop file as
per the autostart spec, not for something in the menus, so the guidelines don't
really affect this use. (And in fact, it would be pretty weird to use it in this

Comment 8 manuel wolfshant 2007-03-02 12:42:53 UTC
Except for Source0 which is wrong (you have left 0.1 instead of 0.2) the rest is
OK. New sha1sum for source file is
4184311e587a49b73a1fc5a00811bcc2d2df2fa7  /tmp/xdg-user-dirs-gtk-0.2.tar.gz
4184311e587a49b73a1fc5a00811bcc2d2df2fa7  /home/wolfy/xdg-user-dirs-gtk-0.2.tar.gz

Your choice of using the desktop file seems (to me at least) to correspond with
so I guess that in this particular case the guidelines do not apply.

Package APPROVED. Please do not forget to fix Source0 before uploading to cvs.

Comment 9 Alexander Larsson 2007-03-02 13:56:26 UTC
New Package CVS Request
Package Name: xdg-user-dirs-gtk
Short Description: Gnome integration for xdg-user-dirs
Branches: devel

Comment 10 Alexander Larsson 2007-03-05 10:13:47 UTC

Comment 11 Warren Togami 2007-03-05 13:41:33 UTC
I see fedora-cvs in the ? state, but it is already added to owners.list and
devel branch is created.  I don't see any further action needing to be taken. 
Setting to +.

Note You need to log in before you can comment on or make changes to this bug.