[Bug 248815] Review Request: trac-bazaar-plugin - Support for Bazaar source repositories in trac

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]