Bug 710517 - Review Request: gnome-shell-extension-remove-accessibility-icon - A gnome-shell extension for removing the accessibility icon
Summary: Review Request: gnome-shell-extension-remove-accessibility-icon - A gnome-she...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Clint Savage
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-06-03 15:25 UTC by Fabian Affolter
Modified: 2011-10-20 03:50 UTC (History)
6 users (show)

Fixed In Version: gnome-shell-extension-remove-accessibility-icon-20110603-1.fc15
Clone Of:
Environment:
Last Closed: 2011-06-15 05:49:27 UTC
Type: ---
Embargoed:
herlo1: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Fabian Affolter 2011-06-03 15:25:10 UTC
Spec URL: http://fab.fedorapeople.org/packages/SRPMS/gnome-shell-extension-remove-accessibility-icon.spec
SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/gnome-shell-extension-remove-accessibility-icon-20110529-1.fc15.src.rpm

Project URL: http://martin-weusten.de/projects/gnomeshell-extensions/remove-accessibility-icon/

Description:
This simple extension does nothing more than to remove the accessibility
icon in the top right corner of the GNOME panel.

Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3108677

rpmlint output:
[fab@laptop021 SRPMS]$ rpmlint gnome-shell-extension-remove-accessibility-icon*
gnome-shell-extension-remove-accessibility-icon.src: W: invalid-license GPL
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

[fab@laptop021 noarch]$ rpmlint gnome-shell-extension-remove-accessibility-icon*
gnome-shell-extension-remove-accessibility-icon.noarch: W: invalid-license GPL
gnome-shell-extension-remove-accessibility-icon.noarch: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

Upstream was asked about the license. So far no answer.

Comment 1 Clint Savage 2011-06-04 13:04:39 UTC
Licensing issue seems to be the only hold-up. 

Builds just fine in mock.

$ rpmlint rpmbuild/SPECS/gnome-shell-extension-remove-accessibility-icon.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

spec file looks clean

$ rpmlint /var/lib/mock/fedora-15-x86_64/result/gnome-shell-extension-remove-accessibility-icon-20110529-1.fc15.noarch.rpm
gnome-shell-extension-remove-accessibility-icon.noarch: W: invalid-license GPL
gnome-shell-extension-remove-accessibility-icon.noarch: W: no-documentation

SHOULD at least include a README and/or manual page.

Just update here when the licensing is sorted and I'll do another review.

Cheers,

Clint

Comment 2 Fabian Affolter 2011-06-04 13:39:07 UTC
Thanks for your help, Clint.

(In reply to comment #1)
> Licensing issue seems to be the only hold-up. 

License is BSD. Upstream added licensing information to the source.

> SHOULD at least include a README and/or manual page.

Upstream added a COPYING and a README file.

* Sat Jun 04 2011 Fabian Affolter <fabian> - 20110603-1
- README and COPYING added
- License is Modified BSD License

Here are the updated files:
Spec URL:
http://fab.fedorapeople.org/packages/SRPMS/gnome-shell-extension-remove-accessibility-icon.spec
SRPM URL:
http://fab.fedorapeople.org/packages/SRPMS/gnome-shell-extension-remove-accessibility-icon-20110603-1.fc15.src.rpm

Comment 3 Clint Savage 2011-06-04 16:31:23 UTC
$ rpmlint /tmp/gnome-shell-extension-remove-accessibility-icon-20110603-1.fc15.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint /var/lib/mock/fedora-15-x86_64/result/gnome-shell-extension-remove-accessibility-icon-20110603-1.fc15.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint ~/rpmbuild/SPECS/gnome-shell-extension-remove-accessibility-icon.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

License stated as BSD in COPYING, good to go.

Package approved.

Clint

Comment 4 Fabian Affolter 2011-06-04 17:23:27 UTC
New Package VCS Request
=======================
Package Name: gnome-shell-extension-remove-accessibility-icon
Short Description: A gnome-shell extension for removing the accessibility icon
Owners: fab
Branches: F15
InitialCC:

Comment 5 D.S. Ljungmark 2011-06-04 20:21:46 UTC
The one thing I have to say against this is that it removes the accessibility icon for all users without a chance to restore it, which is a bit of a bummer in a shared environment. But for a home user, that shouldn't be an issue.

Comment 6 Gwyn Ciesla 2011-06-05 22:43:33 UTC
New Package SCM Request
=======================
Package Name: gnome-shell-extension-remove-accessibility-icon
Short Description: A gnome-shell extension for removing the accessibility icon
Owners: fab
Branches: f15
InitialCC:

Comment 7 Gwyn Ciesla 2011-06-05 22:44:03 UTC
Git done (by process-git-requests).

Comment 8 Fedora Update System 2011-06-06 18:12:06 UTC
gnome-shell-extension-remove-accessibility-icon-20110603-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/gnome-shell-extension-remove-accessibility-icon-20110603-1.fc15

Comment 9 Fedora Update System 2011-06-07 04:38:28 UTC
gnome-shell-extension-remove-accessibility-icon-20110603-1.fc15 has been pushed to the Fedora 15 testing repository.

Comment 10 Fedora Update System 2011-06-15 05:49:22 UTC
gnome-shell-extension-remove-accessibility-icon-20110603-1.fc15 has been pushed to the Fedora 15 stable repository.

Comment 11 Robert 'Bob' Jensen 2011-06-16 12:57:17 UTC
yum search returns the following short description "gnome-shell-extension-remove-accessibility-icon.noarch : A gnome-shell extensions for removing the accessibility icon" extensions should be changed to extension.

Comment 12 Jeremias Nunez 2011-10-20 03:50:17 UTC
not working in fedora 16 beta, it crashes gnome shell


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