Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 177827
Summary: | Review Request: python-ctypes - Advanced Foreign Function Interface for Python | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Konstantin Ryabitsev <icon> |
Component: | Package Review | Assignee: | Toshio Kuratomi <toshio> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | matt_domsch, nobody+pnasrat, toshio |
Target Milestone: | --- | Flags: | kevin:
fedora-cvs+
|
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2006-05-16 22:22:02 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 163779 |
Description
Konstantin Ryabitsev
2006-01-15 01:15:11 UTC
Hi Konstantin, I tried to download the SRPM but the machine doesn't fully respond. Running wget results in an indefinite wait. The machine is ping-able though, so its at least plugged in... ping. (URLs are still inaccessible.) My apologies! I've had to firewall off that machine according to the McGill policies. Here are the new locations: http://open.mcgill.ca/~icon/fe/python-ctypes-0.9.6-1.src.rpm http://open.mcgill.ca/~icon/fe/python-ctypes.spec Thanks! The spec looks sane enough but there's a new upstream version. Do you want me to start reviewing this version or do you want to update first? Konstantin- are you interested in updating to 0.9.9.6 (not sure is it a devevl version or stable)? http://prdownloads.sourceforge.net/ctypes/ctypes-0.9.9.6.tar.gz?download Toshio are you still interested in reviewing? This will be in python 2.5 so it might be nice to have it in Extras beforehand. Paul: If you're volunteering to review if I say I'm too busy, I'll gladly let you take over :-) Otherwise, I can free up some weekend time to work on reviewing. I'm happy to review this. I'm currently seriously swamped at work. If someone wants to take over this one for me, I'd be more than happy to give up the reigns. :) Hmmm Paul - do you want to take over packaging and I'll review? I'm only swamped with work, not seriously swamped :-) http://people.redhat.com/pnasrat/extras/python-ctypes-0.9.9.6-1.src.rpm http://people.redhat.com/pnasrat/extras/python-ctypes.spec Hey Paul -- your srpm is only 4096 bytes so there's something wrong. Should be better now - sha1sum: f7a7efac9246e3195a6fdb39a10d84d5ed6ff6e6 python-ctypes-0.9.9.6-1.src.rpm APPROVED SHA1SUM: f7a7efac9246e3195a6fdb39a10d84d5ed6ff6e6 python-ctypes-0.9.9.6-1.src.rpm 831a731eebf8950a3dd3eb4cc18dbde8c7a859f5 ctypes-0.9.9.6.tar.gz 20edf3c22969a49e6f0bdff9f321e303fa4928e7 python-ctypes.spec Informational: * Note that the CFLAGS=$RPM_OPT_FLAGS definition doesn't seem to do anything for this package. There's no harm in having it but the CFLAGS are actually being pulled in from python, not from the environment. Good: * All review guidelines passed. Highlights follow: * rpmlint has no output. * python-ctypes conforms to the naming guidelines. * License is MIT and included. * The spec file is clear and legible. * Source matches upstream. * Builds on x86_64. * Permissions set properly on all files. * No extraneous directories; no unowned files or directories. * Builds in mock. * Runs through its test suite on x86_64. Nice job nasrat and icon. Sorry it took me so long to finish the review. Please reassign this bug to the actual reviewer. Package Change Request ====================== Package Name: python-ctypes New Branches: EL-5 Updated EPEL Owners: jkeating cvs done. Package Change Request ====================== Package Name: python-ctypes New Branches: el4 Owners: mdomsch mebrown Git done (by process-git-requests). |