This service will be undergoing maintenance at 00:00 UTC, 2016-09-28. It is expected to last about 1 hours
Bug 230585 - Review Request: xdg-user-dirs - Handling of well known folders in homedir
Review Request: xdg-user-dirs - Handling of well known folders in homedir
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: manuel wolfshant
Fedora Package Reviews List
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-03-01 12:15 EST by Alexander Larsson
Modified: 2007-11-30 17:11 EST (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-03-06 04:47:52 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
wolfy: fedora‑review+
dennis: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Alexander Larsson 2007-03-01 12:15:25 EST
Spec URL: http://people.redhat.com/alexl/temp/xdg-user-dirs.spec
SRPM URL: http://people.redhat.com/alexl/temp/xdg-user-dirs-0.3-1.fc7.src.rpm
Description: 
Contains xdg-user-dirs-update that updates folders in a users
homedirectory based on the defaults configured by the administrator.

See http://freedesktop.org/wiki/Software_2fxdg_2duser_2ddirs for more info
Comment 1 manuel wolfshant 2007-03-01 14:03:19 EST
- rpmlint is silent both on source and binary rpm
- package meets naming guidelines
- package meets packaging guidelines 
- license (GPL ) is OK, matches source 
- spec file legible, in am. english
- source matches upstream,is latest available version, sha1sum
ddb6299faae1e23da524b113eb5d7a27c734630a  xdg-user-dirs-0.3.tar.gz
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- locales properly handled via find_lang
- not relocatable
- owns all files/directories that it creates, does not take ownership of foreign
files/directories
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file 
- no scriptlets
- no static, .la or .pc files

SHOULD
- package builds in mock/devel/x86_64
- works as advertised

TODO 
- I suggest including the Changelog (including the one for translations)
- I think that it would be a good idea to use "-p" when installing %Source1, in
order to preserve the timestamp
- MUSTFIX: include the license file, it exists in the tar.gz but is not included
in %doc


Package is APPROVED but please add the license before uploading to CVS
Comment 2 Alexander Larsson 2007-03-02 02:47:45 EST
New Package CVS Request
=======================
Package Name: xdg-user-dirs
Short Description: Handling of user well-known folders
Owners: alexl@redhat.com
Branches: devel
InitialCC: 
Comment 3 Dennis Gilmore 2007-03-02 08:06:40 EST
CVS done
Comment 4 manuel wolfshant 2007-03-06 04:33:24 EST
Alex, the package has been successfuly built, could you please close this ticket ?

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