[Bug 537587] Review Request: dspam - bayesian filtering daemon, client, library and web ui

bugzilla at redhat.com bugzilla at redhat.com
Thu Nov 19 01:43:54 UTC 2009


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=537587


Rahul Sundaram <sundaram at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |sundaram at redhat.com




--- Comment #14 from Rahul Sundaram <sundaram at redhat.com>  2009-11-18 20:43:53 EDT ---

Please remove the conditionals especially things like

"# use "rpmbuild --with single_dyn_drv" or "rpm --define '_with_single_dyn_drv
1'" (for RPM 3.x)"

It doesn't make any sense whatsoever for a new package in Fedora to have a
conditional for a non current RPM release. All the conditionals just looks very
convoluted. You can keep a minimum number of them if really required with
comments on where this is useful.

-- 
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.




More information about the Fedora-package-review mailing list