Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: gilboad@gmail.com QAContact: fedora-package-review@redhat.com
Spec URL: http://gilboadavara.thecodergeek.com/gmrun.spec SRPM URL: http://gilboadavara.thecodergeek.com/gmrun-0.9.2-1.src.rpm
Description: A simple program which provides a "run program" window, featuring a bash-like TAB completion. It uses GTK+ interface. Also, supports CTRL-R / CTRL-S / "!" for searching through history.
$ rpmlint -i -v gmrun-0.9.2-1.src.rpm I: gmrun checking
- Gilboa
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
gilboad@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |163778 nThis| |
------- Additional Comments From gilboad@gmail.com 2007-01-13 02:10 EST ------- Added NEED-REVIEW.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
peter@thecodergeek.com changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163778 | nThis| |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
gilboad@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |177841 nThis| |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From gilboad@gmail.com 2007-01-13 03:42 EST ------- Changed NEEDREVIEW to NEEDSPONSOR.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|177841 | nThis| |
------- Additional Comments From mtasaka@ioa.s.u-tokyo.ac.jp 2007-01-15 13:58 EST ------- Removing NEEDSPONSOR (bug 221873)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From gilboad@gmail.com 2007-01-17 00:51 EST ------- %changelog * Sat Jan 13 2007 <gilboad AT gmail DOT com> - 0.9.2-2 - Fix Source0 URL. - Remove redundant %%doc files.
Spec URL: http://gilboadavara.thecodergeek.com/gmrun.spec SRPM URL: http://gilboadavara.thecodergeek.com/gmrun-0.9.2-2.src.rpm
- Gilboa
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From pertusus@free.fr 2007-01-17 16:36 EST ------- * A quick look at the spec file shows that Group: and URL: are remnants from cgdb.
* it doesn't seems to use directly glib2, so maybe the BR for gtk2-devel is sufficient
* in my opinion, you could remove the leading 'A' from summary, and 'for minimal WMs'.
* Once again %{_datadir}/gmrun/gmrunrc is not to be marked with %config.
* the %{_datadir}/gmrun/ directory is unowned.
* the default config file should be tuned a bit for fedora and apropriate dependencies should be added. In fact I think that it could be easier to rewrite the config file from scratch and add the upstream one in %doc
In my opinion, the best integration would be achieved by calling xdg-open on every URL and on any file, whatever its extension is. Currently it is not possible to specify a default handler for url or files. Maybe upstream could be convinced to add that possibility to the config? In the mean time, I think that every entry in the config file should be handled with xdg-open. For URL_*, could be like:
URL_http = xdg-open %u URL_mailto = xdg-open %u URL_man = xdg-open %u URL_info = xdg-open %u URL_pd = xdg-open %u URL_file = xdg-open %u URL_readme = xdg-open %u
I removed URL_sh, I don't think it is that clever to handle such URI. Maybe some %u should in fact be %s, please adjust.
And for EXT:*:
EXT:txt,cc,cpp,h,java,html,htm,epl,tex,latex,js,css,xml,xsl,am,doc,rtf,ps,pdf = xdg-open %s
The terminal should better be xterm in my opinion given the target of gmrun.
* There is a missing BuildRequires on popt. (Hopefully on popt-devel once it is split).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From gilboad@gmail.com 2007-01-18 10:12 EST ------- "* A quick look at the spec file shows that Group: and URL: are remnants from cgdb."
Sigh. /Bangs head against the wall.
"* it doesn't seems to use directly glib2, so maybe the BR for gtk2-devel is sufficient"
Indeed. gtk2-devel REQ glib2-devel. Done.
* xdg-open
Seems too heavy for me. Correct if I'm wrong, but doesn't xdg-open requires an active browser to handle the different URLs? As I'm targeting, well, my Laptop (P2/366, 256MB) I'm doing my best (in IceWM/gmrun/idesk) to remove unnecessary dep-chains and keep things as-lean-as-possible.
* BuildRequires on popt.
Missed it in the configure output. Done.
* Thu Jan 18 2007 <gilboad AT gmail DOT com> - 0.9.2-3 - Fix wrong group and project URLs. - Fix summery, description. - Remove BR: glib2-devel. (REQ by gtk2-devel). - Add BR: popt - Remove gmrunrc from %%config section. - Set gmrunrc to 0644 (by default it's 0600) - Configure gmrunrc to better handle Fedora URI's. - Add REQ: htmlview. (gmrunrc) - Add REQ: xterm. (gmrunrc)
Spec URL: http://gilboadavara.thecodergeek.com/gmrun.spec SRPM URL: http://gilboadavara.thecodergeek.com/gmrun-0.9.2-3.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From pertusus@free.fr 2007-01-18 20:33 EST ------- (In reply to comment #6)
- xdg-open
Seems too heavy for me.
xdg-open is very light, it is only a shell script. Its package (xdg-utils) only brings in indirectly libglib-2.0 which is already a dep.
Correct if I'm wrong, but doesn't xdg-open requires an active browser to handle the different URLs?
xdg-open doesn't do anything itself. It calls the right command in kde, gnome and xfce. Otherwise it tries mimeopen (a perl script from perl-File-MimeInfo) and last it tries a browser (firefox being the default browser).
So I think using that script is in my opinion the most generic way of handling mimetypes (it is similar with htmlview, but more generic).
Now the real question, in my opinion, is: do you want to have a Requires on perl-File-MimeInfo (or %{_bindir}/mimeopen), given that it brings in perl, in order to be sure that every file will be handled right, or do you accept poor handling of URI and files in case the user don't have a way to use the freedesktop stuff?
As I'm targeting, well, my Laptop (P2/366, 256MB) I'm doing my best (in IceWM/gmrun/idesk) to remove unnecessary dep-chains and keep things as-lean-as-possible.
I also think this is a good idea.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From pertusus@free.fr 2007-01-18 20:54 EST ------- NEWS should certainly be in %doc
It is a personal preference, not something mandatory at all, but I personally like to a a training / in %files for directories to show visually that it is a directory and not a file. Here it leads to
%{_datadir}/gmrun/
The %attr line is wrong, root,root is missing.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From gilboad@gmail.com 2007-01-20 02:13 EST ------- * xdg-open:
I stand corrected, but I tried it a couple of months ago (on my laptop) it always seemed to start firefox when it needed to resolve a certain MIME type. Getting a P2/366 laptop start firefox everytime you type man:pthread_create is a -very- frustrating experience... Now, if I could use say, elinks instead, it would have been nice.
NEWS, %{_datadir}:
Fixed.
* A couple of simple question: (More of a user-opinion) - I'm thinking about adding xscreensaver to the dep chain. Autostart it using startup and add Ctrl+Alt+L to activate it. (xscreensaver-command) Do you see any problem with auto-starting xscreensaver by default? - Second, I'm thinking about doing adding menu support to a couple of key system-config-xxx applets. (desktop, screensaver, mouse, sound, etc) - Third. How about adding xrdb -load ~/.Xdefaults by default?
* Sat Jan 20 2007 <gilboad AT gmail DOT com> - 0.9.2-4 - Add missing NEWS to %%doc.
Spec URL: http://gilboadavara.thecodergeek.com/gmrun.spec SRPM URL: http://gilboadavara.thecodergeek.com/gmrun-0.9.2-4.src.rpm
- Gilboa
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From gilboad@gmail.com 2007-01-20 02:14 EST ------- ... Sigh. I suck at multi-threading :(
Ignore the "couple of questions", they belong to icewm.
- Gilboa
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From pertusus@free.fr 2007-01-21 09:43 EST ------- (In reply to comment #9)
- xdg-open:
I stand corrected, but I tried it a couple of months ago (on my laptop) it always seemed to start firefox when it needed to resolve a certain MIME type. Getting a P2/366 laptop start firefox everytime you type man:pthread_create is a -very- frustrating experience... Now, if I could use say, elinks instead, it would have been nice.
As I said above, with perl-File-MimeInfo installed additionally things are better. Although mimeopen -n 'man:pthread_create' leads to Could not determine mimetype for file: man:pthread_create. Same with urls. Ok, so I revert what I said above. xdg-open together with mimeopen may be good for files, not for urls.
It works for file:, and file extensions. So maybe the patch to gmrunrc could have
URL_file = xdg-open %s
EXT:doc,rtf,txt,cc,cpp,h,java,html,htm,epl,tex,latex,js,css,xml,xsl,am,ps,pdf = xdg-open %s
And some requires missing are
Requires: man, info
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From gilboad@gmail.com 2007-01-28 09:02 EST ------- Done.
* Sun Jan 28 2007 <gilboad AT gmail DOT com> - 0.9.2-5 - Missing REQ: man. - Missing REQ: info. - Missing REQ: xdg-open. - Use xdg-open to handle files.
Spec URL: http://gilboadavara.thecodergeek.com/gmrun.spec SRPM URL: http://gilboadavara.thecodergeek.com/gmrun-0.9.2-5.src.rpm
- Gilboa
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From pertusus@free.fr 2007-01-29 08:52 EST ------- The %files line: %attr(0644,-,-) %{_datadir}/gmrun/gmrunrc
is unuseful (and harmful)...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From pertusus@free.fr 2007-01-29 08:54 EST ------- I still think that 'for minimal WMs' shouldn't be in the summary. In my opinion it is wrong. gmrun may be used with any wm.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From gilboad@gmail.com 2007-02-04 04:23 EST ------- Fixed.
* Sun Feb 04 2007 <gilboad AT gmail DOT com> - 0.9.2-6 - Fixed summery. - Remove redundant fileattr.
Spec URL: http://gilboadavara.thecodergeek.com/gmrun.spec SRPM URL: http://gilboadavara.thecodergeek.com/gmrun-0.9.2-6.src.rpm
- Gilboa
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
pertusus@free.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |pertusus@free.fr OtherBugsDependingO|163776 |163779 nThis| |
------- Additional Comments From pertusus@free.fr 2007-02-05 12:05 EST ------- * rpmlint is silent * spec legible, follow guidelines * free software, GPL license included * sane provides * source match upstream 6cef37a968006d9496fc56a7099c603c gmrun-0.9.2.tar.gz * %files section right
APPROVED
The source timestamps are not kept. It is not a blocker, but better. wget -N or spectool allows to keep timestamps.
ls -l gmrun-0.9.2.tar.gz ../SOURCES/gmrun-0.9.2.tar.gz -rw-rw-r-- 1 dumas dumas 66097 nov 16 2003 gmrun-0.9.2.tar.gz -rw-r--r-- 1 dumas dumas 66097 jan 15 11:28 ../SOURCES/gmrun-0.9.2.tar.gz
popt will hopefully be popt-devel someday soon.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From gilboad@gmail.com 2007-02-10 05:45 EST ------- * Sat Feb 10 2007 <gilboad AT gmail DOT com> - 0.9.2-7 - Preserve the source time-stamp.
Spec URL: http://gilboadavara.thecodergeek.com/gmrun.spec SRPM URL: http://gilboadavara.thecodergeek.com/gmrun-0.9.2-7.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
------- Additional Comments From pertusus@free.fr 2007-02-10 06:27 EST ------- This is already approved... You could have fixed the timestamp before importing to cvs. I can confirm that it is indeed fixed.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
notting@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs+
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request:gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
gilboad@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
------- Additional Comments From gilboad@gmail.com 2007-02-20 06:02 EST ------- Done.
Thanks for the review.
- Gilboa
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: gmrun - A lightweight "Run program" window with TAB completion
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |medium Priority|normal |medium
Christian.Iseli@licr.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request:gmrun - A |Review Request: gmrun - A |lightweight "Run program" |lightweight "Run program" |window with TAB completion |window with TAB completion
------- Additional Comments From Christian.Iseli@licr.org 2007-06-05 12:27 EST ------- Make Summary parser happy.
package-review@lists.fedoraproject.org