[Bug 226337] Merge Review: pyparted
bugzilla at redhat.com
bugzilla at redhat.com
Fri Apr 20 14:16:07 UTC 2007
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Merge Review: pyparted
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226337
bugzilla at redhat.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|normal |medium
Priority|normal |medium
wolfy at nobugconsulting.ro changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |NEEDINFO
AssignedTo|nobody at fedoraproject.org |wolfy at nobugconsulting.ro
Flag| |fedora-review?,
| |needinfo?(dcantrell at redhat.c
| |om)
------- Additional Comments From wolfy at nobugconsulting.ro 2007-04-20 10:15 EST -------
I am planning on reviewing this one, but I have a few questions/suggestions,
before doing an official full review:
- rpmlint has a couple of complaints on the src.rpm:
W: pyparted no-url-tag
W: pyparted mixed-use-of-spaces-and-tabs (spaces: line 4, tab: line 27)
The "no-url-tag" is of course propagated to the binary rpms, too.
Maybe you could fix those?
- I notice CFLAGS is explicitely set :
export CFLAGS="-fPIC -O2 -g2"
If there is a good reason to override the default compiler options (including
the -fortify and so on), please be as kind as to explain it.
- SMP flags are not used; if their usage breaks compilation, please state that
in the spec, otherwise please use them
- The theory says that %makeinstall should be avoided, if possible. I assume it
is a must in this case? If so, could you please add a comment in the spec, to
make it clear ?
- I suggest adding AUTHORS and Changelog to the list of files, so that we know
who to bug in case of trouble :)
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.
More information about the package-review
mailing list