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=200760
Summary: Review Request: dogtail Product: Fedora Core Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: notting@redhat.com ReportedBy: ezannoni@redhat.com QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com
Spec URL: http://people.redhat.com/zcerza/dogtail/
This is a request to move Dogtail from Extras to Core. The review for inclusion into Extras was done here:
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=182306
Reason: Dogtail is needed by the frysk testsuite, and it run in headless mode at Frysk buildtime.
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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
ezannoni@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |198806 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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
jkeating@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|notting@redhat.com |jkeating@redhat.com OtherBugsDependingO|188265 |188267 nThis| |
------- Additional Comments From jkeating@redhat.com 2006-07-31 11:10 EST ------- Er, can I get a direct link to the spec and to a srpm build from said spec? That'll speed up the 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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
------- Additional Comments From jkeating@redhat.com 2006-07-31 11:14 EST ------- This will require pyspi to be in Core as well. Please open up a review to move this from Extras to Core and have it block this bug. We'll focus on pyspi first.
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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
ezannoni@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |200772
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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
------- Additional Comments From ezannoni@redhat.com 2006-07-31 11:49 EST ------- I can find these: http://people.redhat.com/zcerza/dogtail/releases/ but I wonder if there is anything newer.
http://download.fedora.redhat.com/pub/fedora/linux/extras/development/SRPMS/... seems to be the most recent build.
Please also see the old review for extras inclusion (bug 182306).
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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
jkeating@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO_REPORTER OtherBugsDependingO|188267 |188268 nThis| |
------- Additional Comments From jkeating@redhat.com 2006-07-31 14:37 EST ------- Package looks good, rpmlint only has this to say:
E: dogtail non-executable-script /usr/lib/python2.4/site-packages/dogtail/config.py 0644 E: dogtail non-executable-script /usr/lib/python2.4/site-packages/dogtail/tree.py 0644
These are due to #! being in these files. Are these files executable on their own, do they actually do anything? If not, remove the sha-bang. If so, than ignore this.
W: dogtail macro-in-%changelog post
Macros in changelog are usually unsafe as they'll get expanded.
Other than that looks good, approving. Bill already gave tech ack, so who gets to own this? (and who will coordinate its removal from Extras)
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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
ezannoni@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO_REPORTER |ASSIGNED
------- Additional Comments From ezannoni@redhat.com 2006-07-31 14:53 EST ------- Zack is the owner.
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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
------- Additional Comments From zcerza@redhat.com 2006-07-31 18:12 EST ------- tree.py and config.py don't really need to have the shebang line; I just removed them upstream.
I didn't realize putting '%post' in the changelog would be a problem; I'll obscure 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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
------- Additional Comments From paul@city-fan.org 2006-08-01 07:49 EST ------- (In reply to comment #6)
I didn't realize putting '%post' in the changelog would be a problem; I'll obscure that.
Just escape macros in the changelog (e.g. use %%post instead of %post) and you'll get the desired effect.
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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
------- Additional Comments From zcerza@redhat.com 2006-08-01 17:43 EST ------- dogtail just built successfully into FC-devel.
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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
------- Additional Comments From zcerza@redhat.com 2006-08-02 15:22 EST ------- How do I get dogtail removed from FC6's Extras?
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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
------- Additional Comments From jkeating@redhat.com 2006-08-02 15:36 EST ------- Please see http://fedoraproject.org/wiki/Extras/FC6Status
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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
------- Additional Comments From zcerza@redhat.com 2006-08-02 16:21 EST ------- Thanks. FWIW, though, it seems like the page should be called 'FE6Requests' 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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
jkeating@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |RAWHIDE
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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
Bug 200760 depends on bug 200772, which changed state.
Bug 200772 Summary: Review Request: pyspi: Move from Extras to Core https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200772
What |Old Value |New Value ---------------------------------------------------------------------------- Resolution| |RAWHIDE Status|ASSIGNED |CLOSED
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: dogtail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200760
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |medium Priority|normal |medium Product|Fedora Core |Fedora
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=200760
Zack Cerza zcerza@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #12 from Zack Cerza zcerza@redhat.com 2010-07-12 17:03:36 EDT --- Package Change Request ====================== Package Name: dogtail New Branches: EL-6 Owners: zmc
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=200760
--- Comment #13 from Tom "spot" Callaway tcallawa@redhat.com 2010-07-13 11:37:15 EDT --- CVS done (by process-cvs-requests.py).
package-review@lists.fedoraproject.org