Bug 487263 - Review request: preferences-menus - Categorized submenus for the Preferences menu
Summary: Review request: preferences-menus - Categorized submenus for the Preferences ...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-02-25 05:51 UTC by Matthias Clasen
Modified: 2009-03-04 05:11 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-03-04 05:11:18 UTC
Type: ---
Embargoed:
mclasen: fedora_requires_release_note+
panemade: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Matthias Clasen 2009-02-25 05:51:37 UTC
Spec URL: http://people.redhat.com/mclasen/preferences-menus.spec
SRPM URL: http://people.redhat.com/mclasen/preferences-menus-1-1.fc11.src.rpm

Description:

This package recreates the Preferences submenus that Fedora used to have
by default in F10 and earlier.

Comment 1 Parag AN(पराग) 2009-02-25 06:19:03 UTC
1) Good to add versioned requires

2) From http://fedoraproject.org/wiki/Packaging/SourceURL#We_are_Upstream , 
    Will you please add comment in SPEC file. Otherwise I see no way to verify source here.

3) From review guidelines,
   SHOULD: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it.

4) Though for such small package where no upstream release happened, it looks not really needed but good to follow guidelines
http://fedoraproject.org/wiki/PackagingGuidelines#Timestamps
==> Add -p to install command

Comment 2 Matthias Clasen 2009-02-25 14:43:30 UTC
I've added the versioned requires, add a comment and even did the -p.

Then, I've queried upstream (myself), but upstream said no.

New files in the same location.

Comment 3 Matthias Clasen 2009-02-26 03:13:47 UTC
We should probably release-note this:

The Preferences menu does no longer have submenus. This change was done since deep menu hierarchies are difficult to navigate for many users, and the category-based submenus were not terribly obvious. If you are used to the submenus, and prefer to keep the Peferences menu organized in this way, install the preferences-menus package.

Comment 4 Parag AN(पराग) 2009-02-26 03:41:03 UTC
(In reply to comment #2)
> I've added the versioned requires, add a comment and even did the -p.
> 
OK.

> Then, I've queried upstream (myself), but upstream said no.
 Nice comment.

> 
> New files in the same location.

thanks.

koji build => http://koji.fedoraproject.org/koji/taskinfo?taskID=1182091
+ rpmlint is Not silent but ok to accept.
preferences-menus.noarch: W: no-documentation
preferences-menus.noarch: W: no-url-tag
+ Rest looks following packaging guidelines.

APPROVED.

Comment 5 Matthias Clasen 2009-02-26 04:42:42 UTC
New Package CVS Request
=======================
Package Name: preferences-menus 
Short Description: Categorized submenus for the Preferences menu
Owners: mclasen
Branches: 
InitialCC:

Comment 6 Kevin Fenzi 2009-02-27 00:09:39 UTC
cvs done.

Comment 7 Parag AN(पराग) 2009-03-04 05:11:18 UTC
looks this package is already built for requested branches therefore closing this review now.


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