[Bug 433778] Review Request: fsvs - full system versioning using subversion repositories

bugzilla at redhat.com bugzilla at redhat.com
Wed Mar 26 11:48:41 UTC 2008


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: fsvs - full system versioning using subversion repositories


https://bugzilla.redhat.com/show_bug.cgi?id=433778





------- Additional Comments From wolfy at nobugconsulting.ro  2008-03-26 07:48 EST -------
I have one cosmetic suggestion for this package: if you do not mind, please move
the three "%defines" from %install to the top of the package. Makes life easier
for people examining the spec, most people do not expect to find "%defines" in
the middle of the spec.
Not to mention that you could ditch them completely, all files from "%doc_dir"
(and the directory itself) could be included with a single %doc directive and
the other two defines are used only twice each.

-- 
Configure bugmail: https://bugzilla.redhat.com/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.




More information about the Fedora-package-review mailing list