[Bug 225834] Merge Review: gnome-python2 (stalled)

bugzilla at redhat.com bugzilla at redhat.com
Sun Apr 25 22:09:24 UTC 2010

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


Thomas Spura <tomspur at fedoraproject.org> changed:

           What    |Removed                     |Added
             Status|MODIFIED                    |NEW
                 CC|                            |tomspur at fedoraproject.org

--- Comment #4 from Thomas Spura <tomspur at fedoraproject.org> 2010-04-25 18:09:20 EDT ---
- The devel package only contains text files
  -> can be noarch

- Don't use %define inside a macro (first line).
  It would be better, if you'd use the macro from:

- Please use parallel make:

- When building I get:
*** WARNING: You are using a deprecated build system (autotools).
***          You should be using WAF instead; see the file INSTALL.WAF.
***          The autotools build system will be removed in a future version.

- Please preserve timestamps:
  make install DESTDIR=$RPM_BUILD_ROOT INSTALL='install -p'

- I don't see atm what the '%defattr(644,root,root,755)' in some %files
  do... Can this be left out?

- Could you please be consistend with python2-devel vs python-devel?
  e.g. BR python2-devel, but the devel subpackage R python-devel.

Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.

More information about the package-review mailing list