Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=395651
Summary: Review Request: python-enum - Robust enumerated type support in Python Product: Fedora Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: mcepl@redhat.com QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
Spec URL: http://mcepl.fedorapeople.org/rpms/python-enum.spec SRPM URL: http://koji.fedoraproject.org/koji/getfile?taskID=254391&name=python-enu... Description: This package provides a module for robust enumerations in Python.
An enumeration object is created with a sequence of string arguments to the Enum() constructor::
from enum import Enum Colours = Enum('red', 'blue', 'green') Weekdays = Enum('mon', 'tue', 'wed', 'thu', 'fri', 'sat', 'sun')
The return value is an immutable sequence object with a value for each of the string arguments. Each value is also available as an attribute named from the corresponding string argument::
pizza_night = Weekdays[4] shirt_colour = Colours.green
The values are constants that can be compared only with values from the same enumeration; comparison with other values will invoke Python's fallback comparisons::
pizza_night == Weekdays.fri
True
shirt_colour > Colours.red
True
shirt_colour == "green"
False
Each value from an enumeration exports its sequence index as an integer, and can be coerced to a simple string matching the original arguments used to create the enumeration::
str(pizza_night)
'fri'
shirt_colour.index
2
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: python-enum - Robust enumerated type support in Python
https://bugzilla.redhat.com/show_bug.cgi?id=395651
------- Additional Comments From mcepl@redhat.com 2007-11-22 08:34 EST ------- Scratch built in koji http://koji.fedoraproject.org/koji/taskinfo?taskID=254391 (src.rpm is also available at http://mcepl.fedorapeople.org/rpms)
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: python-enum - Robust enumerated type support in Python
https://bugzilla.redhat.com/show_bug.cgi?id=395651
tim.lauridsen@googlemail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |tim.lauridsen@googlemail.com Status|NEW |ASSIGNED Flag| |fedora-review?
------- Additional Comments From tim.lauridsen@googlemail.com 2007-11-22 08:55 EST ------- I will review this tomorrow, but i have some prelimynary comments.
Just write the what the package does in the decription in the spec, not how it does it. That info should be placed in some DOCS.
for F8 you have to use BuildRequires: python-setuptools-devel
Something like this will work.
%if 0%{?fedora} >= 8 BuildRequires: python-setuptools-devel %else BuildRequires: python-setuptools %endif
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: python-enum - Robust enumerated type support in Python
https://bugzilla.redhat.com/show_bug.cgi?id=395651
------- Additional Comments From mcepl@redhat.com 2007-11-22 10:49 EST ------- New build of the package is on http://koji.fedoraproject.org/koji/taskinfo?taskID=254503 new src.rpm is http://koji.fedoraproject.org/koji/getfile?taskID=254503&name=python-enu...
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: python-enum - Robust enumerated type support in Python
https://bugzilla.redhat.com/show_bug.cgi?id=395651
------- Additional Comments From tim.lauridsen@googlemail.com 2007-11-23 00:06 EST ------- ? rpmlint must be silent.
$rpmlint /data/dnl/python-enum-0.4.3-2.fc9.src.rpm
$ rpmlint python-enum-0.4.3-2.fc8.noarch.rpm python-enum.noarch: W: incoherent-version-in-changelog 0.4.3-2.fc9 0.4.3-2.fc8
caused by the 0.4.3-2.fc9 in the Changelog entries, just use 0.4.3-2.
* source match upstream source
[tim@localhost ~]$ md5sum /data/dnl/enum-0.4.3.tar.gz f0e2bd1a0a581897ea3ed245a1db6c8e /data/dnl/enum-0.4.3.tar.gz [tim@localhost ~]$ md5sum rpmbuild/SOURCES/enum-0.4.3.tar.gz f0e2bd1a0a581897ea3ed245a1db6c8e rpmbuild/SOURCES/enum-0.4.3.tar.gz [tim@localhost ~]$
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: python-enum - Robust enumerated type support in Python
https://bugzilla.redhat.com/show_bug.cgi?id=395651
------- Additional Comments From tim.lauridsen@googlemail.com 2007-11-23 00:28 EST ------- MUST: * package is named appropriately * it is legal for Fedora to distribute this ? license field matches the actual license. Package is dual licensed GPLv2+ or Python, just use GPLv2+ in the License field. * license is open source-compatible. * specfile name matches %{name} * summary and description fine * correct buildroot * %{?dist} is used * license text included in package and marked with %doc * package meets FHS (http://www.pathname.com/fhs/) ? changelog format fine See earlier comment. * Packager tag not used * Vendor tag not used * Distribution tag not used * License used and not Copyright * Summary tag does not end in a period * specfile is legible * package successfully compiles and builds on at least x86 * make sure lines are <= 80 characters * specfile written in American English * no -doc sub-package necessary * no libraries * no rpath * not a GUI app * no -devel sub-package necessary * macros used appropriately and consistently * no %makeinstall * install section must begin with rm -rf $RPM_BUILD_ROOT or %{buildroot} * no locale data * split Requires(pre,post) into two separate lines * package not relocatable * package contains code * package owns all directories and files * no %files duplicates * %defattrs present ( %defattr(-, root, root, -)) * %clean present * %doc files do not affect runtime
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: python-enum - Robust enumerated type support in Python
https://bugzilla.redhat.com/show_bug.cgi?id=395651
tim.lauridsen@googlemail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
------- Additional Comments From tim.lauridsen@googlemail.com 2007-11-23 01:40 EST ------- * verify the final provides and requires of the binary RPMs
$ rpm -q -R -p rpmbuild/RPMS/noarch/python-enum-0.4.3-2.fc8.noarch.rpm python(abi) = 2.5 rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1
$ rpm -q --provides -p rpmbuild/RPMS/noarch/python-enum-0.4.3-2.fc8.noarch.rpm python-enum = 0.4.3-2.fc8
SHOULD: * package should include license text in the package and mark it with %doc * package should build on i386 ? package should build in mock - I haven't tried, but I don't think it'll be a problem
Fixes the license field and the the changelog entry versions and it is
APPROVED.
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: python-enum - Robust enumerated type support in Python
https://bugzilla.redhat.com/show_bug.cgi?id=395651
------- Additional Comments From mcepl@redhat.com 2007-11-23 06:33 EST ------- (In reply to comment #5)
MUST: ? license field matches the actual license. Package is dual licensed GPLv2+ or Python, just use GPLv2+ in the License field.
That's incorrect -- I can and I should use what I did. http://fedoraproject.org/wiki/PackagingDrafts/LicenseTag#head-35bf3683af78c7... 'the spec must reflect this by using "or" as a separator.'
? changelog format fine See earlier comment.
OK, fixed.
? package should build in mock
- I haven't tried, but I don't think it'll be a problem
It was built in koji, which is even better than mock (because it is multiplatform build -- not relevant in this case, true).
New and upgraded package is available at http://koji.fedoraproject.org/koji/getfile?taskID=255432&name=python-enu... (build page in koji is http://koji.fedoraproject.org/koji/taskinfo?taskID=255432)
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: python-enum - Robust enumerated type support in Python
https://bugzilla.redhat.com/show_bug.cgi?id=395651
mcepl@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From mcepl@redhat.com 2007-11-23 06:49 EST ------- New Package CVS Request ======================= Package Name: python-enum Short Description: Robust enumerated type support in Python Owners: mcepl Branches: F-7 F-8 InitialCC: Cvsextras Commits: yes
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: python-enum - Robust enumerated type support in Python
https://bugzilla.redhat.com/show_bug.cgi?id=395651
------- Additional Comments From tim.lauridsen@googlemail.com 2007-11-23 08:33 EST -------
That's incorrect -- I can and I should use what I did.
http://fedoraproject.org/wiki/PackagingDrafts/LicenseTag#head-35bf3683af78c7...
'the spec must reflect this by using "or" as a separator.'
Yes, you are right.
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: python-enum - Robust enumerated type support in Python
https://bugzilla.redhat.com/show_bug.cgi?id=395651
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2007-11-23 11:41 EST ------- cvs done.
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: python-enum - Robust enumerated type support in Python
https://bugzilla.redhat.com/show_bug.cgi?id=395651
mcepl@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
------- Additional Comments From mcepl@redhat.com 2007-11-23 12:20 EST ------- build done http://koji.fedoraproject.org/koji/taskinfo?taskID=255762
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=395651
Adam Miller maxamillion@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |maxamillion@fedoraproject.o | |rg Flag|fedora-cvs+ |fedora-cvs?
--- Comment #12 from Adam Miller maxamillion@fedoraproject.org 2010-05-07 13:04:02 EDT --- Package Change Request ====================== Package Name: python-enum New Branches: EL-5 Owners: maxamillion
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=395651
Dennis Gilmore dennis@ausil.us changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #13 from Dennis Gilmore dennis@ausil.us 2010-05-13 18:48:56 EDT --- CVS Done
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=395651
Adam Miller maxamillion@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs+ |fedora-cvs?
--- Comment #14 from Adam Miller maxamillion@fedoraproject.org 2011-01-25 13:41:06 EST --- Package Change Request ====================== Package Name: python-enum New Branches: el6 Owners: maxamillion
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=395651
--- Comment #15 from Jason Tibbitts tibbs@math.uh.edu 2011-01-25 18:40:18 EST --- Git done (by process-git-requests).
package-review@lists.fedoraproject.org