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: redet - Regular expression development and execution tool
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=241075
dev(a)nigelj.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag| |fedora-cvs?
------- Additional Comments From dev(a)nigelj.com 2007-06-08 00:01 EST -------
(In reply to comment #7)
Sorry it's taken me three days to get back to this.
It's fine if you put off the manpage bit; it's kind of pointless to compress it
yourself but I don't think it's a blocker.
Unfortunately this fails to build in mock for me; you just need to add a build
dependency on desktop-file-utils and everything goes through fine.
Ahh! I'll
fix that when I get home.
I installed on my desktop and found a problem with the desktop file: It has no
GenericName entry, so instead of "Redet (Regular expression whatever)" I just
get "Redet". I would recommend replacing "Comment" with
"GenericName" in your
desktop file, but the desktop entry spec doesn't require GenericName so I don't
think it's a blocker.
I'll fix this tonight as well.
I am not sure if TCL actually supports StartupNotify=true, but startup
notification seems to work well enough for me under KDE so I suppose it's OK.
So since there's only one remaining trivial blocker and it will keep you from
actually building the package, I'll go ahead and approve this and you can fix it
when you check in.
APPROVED
New Package CVS Request
=======================
Package Name: redet
Short Description: Regular expression development and execution tool
Owners: dev(a)nigelj.com
Branches: FC-6 F-7 EL-4 EL-5
InitialCC:
--
Configure bugmail:
https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.