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: trac-bazaar-plugin - Support for Bazaar source repositories in trac https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=248815 ------- Additional Comments From toshio@xxxxxxxxxxxxxxx 2007-07-28 12:36 EST ------- Thanks tibbs! (In reply to comment #1) > > You can probably remove the initial comment from the spec. > Done. > The only thing I can complain about is inconsistent macroization (%{__python} > but not %{__rm}). I vaguely remember something special about python that > exempted it from the usual "consistent use of macros" rule, but I can't recall > the details. In any case, I trust you to fix things up as appropriate. > I'm unable to find details either so I made this consistent as well. > * %check is not present; no test suite upstream. I haven't the means to test this package. This is running on the Fedora Infrastructure trac instance ATM:: https://hosted.fedoraproject.org/projects/packagedb/browser URLs for packages that will be imported:: http://fedorapeople.org/~toshio/packages/trac-bazaar-plugin.spec http://fedorapeople.org/~toshio/packages/trac-bazaar-plugin-0.2-2.20070717bzr180.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review