[Bug 583869] Review Request: python-plwm - Modular X window manager for keyboard-loving programmers
bugzilla at redhat.com
bugzilla at redhat.com
Tue Apr 20 05:07:16 UTC 2010
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=583869
Rahul Sundaram <metherid at gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |metherid at gmail.com
--- Comment #1 from Rahul Sundaram <metherid at gmail.com> 2010-04-20 01:07:14 EDT ---
[ Assuming you are only interested in maintaining this for Fedora 13 and above]
You don't have to define the Python macros on top. They are done automatically
by RPM itself in Fedora 13. Read the Fedora Python packaging guidelines
carefully.
https://fedoraproject.org/wiki/Packaging:Python
You should also inquire with upstream about the Python 3 status of this module
You can remove the buildroot definition and cleaning of buildroot in %install
section. These are not necessary anymore. Also remove the commented out
%clean section
You should also add a comment on each of the patches with a short description
of what it does and upstream status of the patches. Of course, you haven't
filed it upstream or informed the upstream developer yet of these changes, now
would be a good time to do so.
--
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 package-review
mailing list