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=236642
Summary: Review Request: <main package name here> - <short summary here> Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: jonathansteffan@gmail.com QAContact: fedora-package-review@redhat.com
Spec URL: http://files.damaestro.us/revisor/revisor.spec SRPM URL: http://files.damaestro.us/revisor/revisor-2.0.1-2.fc7.src.rpm Description: Fedora "Spin" Graphical User Interface
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
jonathansteffan@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: <main |Review Request: Revisor - |package name here> - <short |Revisor GUI |summary here> |
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
kanarip@kanarip.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |kanarip@kanarip.com
------- Additional Comments From kanarip@kanarip.com 2007-04-16 18:10 EST ------- Adding myself to CC
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
kanarip@kanarip.com changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |163776 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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
jspaleta@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |jspaleta@gmail.com Flag| |fedora-review?
------- Additional Comments From jspaleta@gmail.com 2007-04-18 04:25 EST ------- Okay initial review comments. Attached you will find an updated specfile which fixes some simple problems in the specfile and the important rpmlint error message concerning consolehelper symlink. Fixes in updated specfile:
- remove makeinstall macro - add usermode and pam requires - make symlink to consolehelper relative path instead of absolute path - removed desktop-update-database scriptlets and associated deps Refer to http://fedoraproject.org/wiki/Packaging/ScriptletSnippets
One thing I can't easily fix is the SOURCE0 tag. Please refer to http://fedoraproject.org/wiki/Packaging/SourceURL and follow the directions for Using Revision Control to add the appropriate comment blog which gives the necessary instructions to rebuild the source tarball locally. The goal is to provide the steps necessary for me to rebuild the tarball so that I can do the appropriate md5sum check for the tarball you have included.
Please regenerate a new spec and srpm with these corrections and I should be able to run through the full formal review.
-jef
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jspaleta@gmail.com 2007-04-18 04:27 EST ------- Created an attachment (id=152880) --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=152880&action=vie...) Updated Spec with review inspired corrections
This is an updated spec which includes corrections found during submission 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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From kanarip@kanarip.com 2007-04-18 06:11 EST ------- Thank you for taking the time to review our package.
Updated Source RPM: http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.1-3.fc7.src....
Updated Spec file: http://revisor.fedoraunity.org/releases/revisor-2.0/revisor.spec
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jspaleta@gmail.com 2007-04-19 02:06 EST ------- You really really really need to do something about the SOURCE tag as mentioned in comment #2.
Since the source tarball is being generated from a source control system (git in this case if im reading the revisor development webpages correctly) and its not available at an established static URL address as an identifiable 'release' then the spec file must contain a comment blog which details the actual steps that need to be taken to pull the appropriated tagged git entity and then repackage it as a tarball.
http://fedoraproject.org/wiki/Packaging/SourceURL has an example of how to do a comment block for a public svn server. You should do something similar for your git tree.
I cannot initiate a formal review if I can't confirm that the md5sum of the tarball is as it should be.
-jef
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From kanarip@kanarip.com 2007-04-19 03:59 EST ------- I'm sorry, obviously you are right. I didn't regenerate the .spec after editting the .spec.in.
I've updated the .spec file on the location mentioned above.
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From kanarip@kanarip.com 2007-04-19 04:11 EST ------- No I've not updated, for some reason it won't update.
New location is http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.1.spec
Thank you Jef, for your patience ;-)
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jspaleta@gmail.com 2007-04-20 02:34 EST ------- hmm im getting a name resolution failure for revisor.fedoraunity.org at the moment. Is the result of a problem on my end or is the machine down currently?
-jef
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jonathansteffan@gmail.com 2007-04-20 02:47 EST ------- Yeah, the VM has been moved to another host due to Bob needing to use the physical machine for another purpose (to replace a server having hardware issues for $work) and as a side effect the DNS servers are down at the moment. Minimally we are going to add my primary DNS server as a slave to help prevent this in the future but we are also evaluating just moving the plone instance to my dedicated zope server. Sorry Jef, it'll be back tomorrow I assume.
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jspaleta@gmail.com 2007-04-22 15:52 EST ------- Okay, http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.1.spec still isn't resolving correctly. that URL gets redirected to index.html for the fedoraunity site.
Please provide an updated url for both the current srpm and spec.
-jef
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From kanarip@kanarip.com 2007-04-23 04:41 EST ------- We're sorry for the inconvenience, backups were trashed.
Files are up now; SPEC: http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.1.spec SRPM: http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.1-3.fc7.src....
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jspaleta@gmail.com 2007-04-26 04:27 EST ------- Okay the SOURCE0 tag still isn't good enough. I need a valid url at which I can find the revisor-2.0.1.tar.gz tarball or I need a comment block in the specfile which tells me how to generate the revisor-2.0.1.tar.gz using git commands.
-jef
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From kanarip@kanarip.com 2007-04-26 05:05 EST ------- Source0 tag is now: Source0: http://revisor.fedoraunity.org/releases/revisor-2.0/%%7Bname%7D-%%7Bversion%...
I'm not sure I understand what is wrong here, because that URL gives you the proper tarball, located at
http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.1.tar.gz
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jspaleta@gmail.com 2007-04-26 14:41 EST ------- oops sorry, I had a stale srpm here that i was looking at. The new SOURCE0 looks fine I should be able to start the formal review.
One thing, you should consider changing how the consolehelper symlink is created in the release tarball for the next release, so we can avoid the hack to fix it in the specfile.
-jef
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jspaleta@gmail.com 2007-04-26 14:47 EST ------- Problem: md5sum from SOURCE0 url and included sources do not match. Please doublecheck that you are including the tarball as listed from the SOURCE0 location.
Here's my local actions for md5sum test
wget
http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.1-3.fc7.src....
rpm -ivh revisor-2.0.1-3.fc7.src.rpm
md5sum rpmbuild/SOURCES/revisor-2.0.1.tar.gz
1ae8d73266079f93e3d543631ce59704 rpmbuild/SOURCES/revisor-2.0.1.tar.gz
spectool rpmbuild/SPECS/revisor.spec
Source0: http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.1.tar.gz
wget http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.1.tar.gz md5sum revisor-2.0.1.tar.gz
6acf0e69808e014683607ab9628b5586 revisor-2.0.1.tar.gz
-jef
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jonathansteffan@gmail.com 2007-05-01 06:31 EST ------- SRPM: http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.1-4.fc7.src.... Tar: http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.1.tar.gz Spec: http://revisor.fedoraunity.org/releases/revisor-2.0/revisor.spec
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jspaleta@gmail.com 2007-05-04 03:50 EST ------- The GOOD + naming is good + specfile name matches base package name + specfile written in english-ese and is legible + included source md5sum checks with upstream source as listed in SOURCE0 url 9b1bb4207f9c8a64609d1007420955ef revisor-2.0.1.tar.gz + builds on x86 fedora-development in mock + no locales + not relocatable + clean section is okay + consistent use of macros + permissible code and content + items in doc are not runtime necessary + does not obviously own files from another package + directory ownership of parent directories is accounted for in package deps. + No .la files + No devel subpackage + Need need to for shared libraries sciptlets + no need for scriptlets.
The BAD - Licensed as GPL but COPYING file NOT included in docs!!
- install section has a problem needs to include a desktop-file-install stanza to install the desktop file correctly, and require desktop-file-utils Refer to http://fedoraproject.org/wiki/Packaging/Guidelines#desktop
? permissions on pilgrim.py Is pilgrim.py meant to be run stand-alone as an executable. You should either strip the shell invocation at the top of the file or make it executable.
The Suggestions: You can remove pam from the requires list, usermode requires pam. I know I know i suggested it originally, based on just the directory ownership crap. But taking a closer look the pam requires is redundant.
I haven't actually used this yet. Is there a simple example walkthrough on usage? Like how to make a stupidly simple livedvd image or something, so I dont have to think about the package selection but I can test the wizard interface.
rpmlint revisor-2.0.1-4.fc7.noarch.rpm E: revisor non-executable-script /usr/lib/python2.5/site-packages/revisor/pilgrim.py 0644 W: revisor non-conffile-in-etc /etc/revisor/conf.d/pungi-fc6-i386.conf W: revisor non-conffile-in-etc /etc/revisor/conf.d/revisor-fc6-ppc.conf W: revisor non-conffile-in-etc /etc/revisor/conf.d/pungi-f7-i386.conf W: revisor non-conffile-in-etc /etc/revisor/conf.d/pungi-fc6-ppc.conf W: revisor non-conffile-in-etc /etc/revisor/conf.d/pungi-fc6-x86_64.conf W: revisor non-conffile-in-etc /etc/revisor/conf.d/revisor-fc6-x64_86.conf W: revisor non-conffile-in-etc /etc/revisor/conf.d/revisor-fc6-i386.conf W: revisor non-conffile-in-etc /etc/revisor/conf.d/sample-ks.cfg W: revisor non-conffile-in-etc /etc/revisor/revisor.conf W: revisor non-conffile-in-etc /etc/security/console.apps/revisor W: revisor non-conffile-in-etc /etc/revisor/conf.d/revisor-f7-i386.conf W: revisor non-conffile-in-etc /etc/pam.d/revisor
-- warnings are bogus, might want to patch the pilgrim.py to either be executable or to strip the intepreter from the first line.
rpmlint revisor-2.0.1-4.fc7.src.rpm W: revisor strange-permission revisor.spec 0600 -- not important, but you might consider making it world and group readable by default.
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From kanarip@kanarip.com 2007-05-10 16:43 EST ------- Spec: http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.2.spec Tarball: http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.2.tar.gz SRPM: http://revisor.fedoraunity.org/releases/revisor-2.0/revisor-2.0.2-1.fc7.src....
The only error I get with rpmlint is on the SRPM which still reports strange-permission (I don't know how to solve that)
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From mspevack@redhat.com 2007-05-22 16:06 EST ------- Can we PLEASE do whatever it takes to get this thing through review and into the Fedora repo BEFORE the F7 release? (meaning, ASAP)
People are going to be looking for it. Let's make sure they can have it. We've spent lots of time and energy building it and promoting the hell out of it. It would be a shame if people can't find and use it.
We're going to have a feature article in RH Magazine that talks about Revisor, and we made a big deal about it in the press interviews that are ongoing and all F7 release material.
What's the biggest hold up? I assume between Jef, Steffan, and Jeroen, we can get this thing done. You three have enough combined brainpower to make Zod himself nervous.
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |tibbs@math.uh.edu
------- Additional Comments From tibbs@math.uh.edu 2007-05-22 16:20 EST ------- All that's necessary is for Jef to verify that the issues he saw have been fixed and give his approval. Then the process of getting it branched, checked in and built can happen.
I'm happy to tale a look if Jef is on vacation or something.
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From kanarip@kanarip.com 2007-05-22 17:24 EST ------- As I understand it Mr. Tibbitts is going to take Revisor to a nice Indian restaurant, only 6 blocks from here, so I'm gonna stand-by online and wait for the ping ;-)
Thank you Mr. Tibbitts!
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|jspaleta@gmail.com |tibbs@math.uh.edu OtherBugsDependingO|163776 | nThis| |
------- Additional Comments From tibbs@math.uh.edu 2007-05-22 20:41 EST ------- OK, assigning to me with visions of real, hot vindaloo (as opposed to "painful for a Yankee, boring for a Texan" Boston-style vindaloo).
I'm going to build on top of Jef's review here, check the issues he raised and trust him for the rest. Just let me run this through mock....
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From tibbs@math.uh.edu 2007-05-22 21:36 EST ------- OK, that didn't take long.
You're right, rpmlint is now silent except for:
W: revisor strange-permission revisor.spec 0600
which I don't think is a blocker, but should be easily fixed by just "chmod 644 revisor.spec". It shouldn't make much difference once it gets checked into CVS anyway.
There is a copy of the license in the tarball; that needs to be included in the final package as %doc.
As Jef pointed out above, I see no mention of desktop-file-install; you'll need to add BR: desktop-file-utils and then something like this in %install:
desktop-file-install --vendor="fedora" \ --delete-original \ --dir=%{buildroot}%{_datadir}/applications \ %{buildroot}/%{_datadir}/applications/foo.desktop
The package installs fine on a handy rawhide system, shows up in the KDE menu and starts without problems. I note, though, that the menu entries do nothing for me, and if I select "Rescue Image" as the type of media to compose, I get an error dialog telling me "No media selected". I'd wager that these are not issues related to packaging and so aren't blockers, but you may know better than I.
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From mspevack@redhat.com 2007-05-22 23:18 EST ------- Thanks Jason. Looks like we should be able to get this thing into CVS pretty easily at this point, which is going to be great.
All you, Jeroen!
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jonathansteffan@gmail.com 2007-05-23 14:23 EST ------- For a note about "W: revisor strange-permission revisor.spec 0600":
revisor.spec is generated by autmake. If there is anyone that knows how to fix this, we are more then willing to fix it. I just have not figured out how.
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jonathansteffan@gmail.com 2007-05-23 14:24 EST ------- s/autmake/automake/
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From tibbs@math.uh.edu 2007-05-23 14:51 EST ------- rpmlint is complaining about the spec that's in the SRPM, not anything generated by the build process, but I guess you could be building the SRPM with rpmbuild -ts on the tarball. To add to oddness, though, the spec in the tarball has mode 664.
In any case, it's really not a big deal. The other issues are much more interesting, but should also be much easier to fix.
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From tibbs@math.uh.edu 2007-05-24 23:02 EST ------- I received an updates SRPM via IRC. Here's a quick recap of the IRC discussion for the record:
The desktop file gets installed properly, although desktop-file-install does emit a warning:
/var/tmp/revisor-2.0.2-3.fc7-root/usr/share/applications/fedora-revisor.desktop: warning: The 'Application' category is not defined by the desktop entry specification. Please use one of "AudioVideo", "Audio", "Video", "Development", "Education", "Game", "Graphics", "Network", "Office", "Settings", "System", "Utility" instead
I don't think this is a big deal, although it would be good to fix at some point. Either "Development" or "System" seem OK to me.
The COPYING file still doesn't get included in the package. All you need to do is change: %doc README NEWS AUTHORS to %doc README NEWS AUTHORS COPYING in your %files section.
That should be it.
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
------- Additional Comments From tibbs@math.uh.edu 2007-05-25 00:10 EST ------- OK, I've received an updated package via IRC.
rpmlint is silent except for the aforementioned strange-permission warning on the spec which isn't a blocker.
The desktop-file-install issue is fixed.
COPYING is included in the package.
So we're done.
APPROVED
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jonathansteffan@gmail.com 2007-05-25 02:00 EST ------- New Package CVS Request ======================= Package Name: revisor Short Description: Fedora "Spin" Graphical User Interface Owners: jonathansteffan@gmail.com Branches: F-7 InitialCC:
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
jonathansteffan@gmail.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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
tcallawa@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From tcallawa@redhat.com 2007-05-25 16:24 EST ------- cvs done.
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From tibbs@math.uh.edu 2007-06-01 01:39 EST ------- Please remember to close this bug once the package has been built, thanks. (My open ticket list is getting long.)
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
jonathansteffan@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RELEASE_PENDING
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
jonathansteffan@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RELEASE_PENDING |CLOSED Resolution| |NEXTRELEASE
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
kanarip@kanarip.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs+ |fedora-cvs?
------- Additional Comments From kanarip@kanarip.com 2007-06-16 12:48 EST ------- Package Change Request ====================== Package Name: revisor Updated Fedora Owners: jonathansteffan@gmail.com,kanarip@kanarip.com Updated Fedora CC: jonathansteffan@gmail.com,kanarip@kanarip.com
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
------- Additional Comments From jonathansteffan@gmail.com 2007-06-16 17:14 EST ------- Yes, this should be done.
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: Revisor - Revisor GUI
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236642
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From tibbs@math.uh.edu 2007-06-18 12:40 EST ------- Co-maintainer added.
package-review@lists.fedoraproject.org