Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 451773 - Review Request: window-picker-applet - an alternative window chooser for small screens
Summary: Review Request: window-picker-applet - an alternative window chooser for smal...
Keywords:
Status: CLOSED DUPLICATE of bug 523326
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Tim Lauridsen
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 498200 (view as bug list)
Depends On:
Blocks: FedoraMini
TreeView+ depends on / blocked
 
Reported: 2008-06-17 10:01 UTC by Yaakov Nemoy
Modified: 2009-09-14 21:51 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-09-14 21:51:31 UTC
Type: ---
Embargoed:
tla: fedora-review?


Attachments (Terms of Use)
spec file cleanup (1.84 KB, patch)
2008-06-17 16:25 UTC, Peter Robinson
no flags Details | Diff

Description Yaakov Nemoy 2008-06-17 10:01:27 UTC
SPEC File: http://ynemoy.fedorapeople.org/repo/window-picker-applet.spec
SRPM File:
http://ynemoy.fedorapeople.org/repo/window-picker-applet-0.2-1.fc9.src.rpm

window-picker-applet is an applet for the Gnome Panel that provides an alternate
window chooser for small screens.  It displays a small single icon button for
each window and combines a title button for the currently active window.

Comment 1 Peter Robinson 2008-06-17 16:25:40 UTC
Created attachment 309634 [details]
spec file cleanup

Attached is an initial patch to cleanup the spec file somewhat. With this the
srpm created passes rpmlint. It probably needs some extra build reqs. You also
need to change the name/email in the Changelog entry.

I think there possibly should also be some post install gconf stuff too but
need to check that out further. Someone in the know should be able to confirm
it.

Comment 2 Jonathan Roberts 2008-09-18 16:37:38 UTC
Updated the spec file to make it comply with packaging guidelines better, as
well as renamed to remove Ubuntu from the name. 

SPEC File: http://ynemoy.fedorapeople.org/repo/window-picker-applet.spec
SRPM File:
http://ynemoy.fedorapeople.org/repo/window-picker-applet-0.2-1.fc9.src.rpm

Have spoken with Yaakov, and will likely co-maintain these packages.

Comment 3 Tim Lauridsen 2008-09-19 14:03:58 UTC
I will review this one:

Initial comments:

Requires and BuildRequires are missing

libpanelapplet-2.0
gtk+-2.0
gconf-2.0
libwnck-1.0
and maybe others

URL should point to upstream home page not to source

URL should be
Source0: http://launchpadlibrarian.net/14942872/window-picker-applet_%(version).tar.gz

Comment 4 Tim Lauridsen 2008-09-19 14:09:02 UTC
forgot the URL
https://launchpad.net/netbook-remix
is the closest to and upstream homepage.

and Source0 shall point to 
http://launchpadlibrarian.net/14942872/window-picker-applet_%(version).tar.gz

Comment 5 Jonathan Roberts 2008-09-19 15:16:18 UTC
I've spoken to upstream and new source tarballs are due out next week, so I'll hold on making these fixes and changing the source details until then. They've also committed to making more regular releases that will be better to use.

Comment 6 Tim Lauridsen 2008-09-20 15:26:29 UTC
Sounds good, maybe you should talk to upstream about making the source tarballs
more standard windows-picker-applet-%{version}.tar.gz, instead of _%{version}.
in ume-launcer it is even more needed the fix the tarball to
ume-launcher-%{version}.tar.gz.

Comment 7 Jonathan Roberts 2008-09-20 22:42:06 UTC
Have done so already :) They said that with the current speed of development, they're really pushed for time and not always able to do make dist for every release and update configure.ac. They have said though that they'll endeavour to make more releases in the future, now that they know others are using their code.

Comment 8 Christoph Wickert 2008-10-24 14:31:54 UTC
Just like in bug #451768 I suggest to rename this package to gnome-applet-window-picker (with a "Provides: window-picker-applet"). See
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Addon_Packages_.28General.29

Comment 9 Bastien Nocera 2009-07-26 22:58:31 UTC
*** Bug 498200 has been marked as a duplicate of this bug. ***

Comment 10 Michel Alexandre Salim 2009-09-14 07:20:56 UTC
Yaakov, are you still interested in packaging this? I've been running my own window-picker-applet for a while, I can take over the packaging if you want. Desperately need the extra vertical pixels that w-p-a + maximus gives :)

Comment 11 Yaakov Nemoy 2009-09-14 11:50:07 UTC
Feel free to take this over, i have no interest in this package, since i don't use Gnome anymore.

Comment 12 Michel Alexandre Salim 2009-09-14 21:51:31 UTC
Thanks; I just filed the review request.

Tim, would you like to continue with the review? I addressed your concern with package naming. And the 0.5 series is much more visually attractive too.

*** This bug has been marked as a duplicate of bug 523326 ***


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