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 - Review Request: python-ctypes - Advanced Foreign Function Interface for Python
Summary: Review Request: python-ctypes - Advanced Foreign Function Interface for Python
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Toshio Kuratomi
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-01-15 01:15 UTC by Konstantin Ryabitsev
Modified: 2011-02-03 19:49 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-05-16 22:22:02 UTC
Type: ---
Embargoed:
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Konstantin Ryabitsev 2006-01-15 01:15:11 UTC
Spec Name or Url: http://rakta.wsg.mcgill.ca/fe/python-ctypes.spec
SRPM Name or Url: http://rakta.wsg.mcgill.ca/fe/python-ctypes-0.9.6-1.src.rpm
Description:
ctypes is an advanced ffi (Foreign Function Interface) package for
Python 2.3 and higher.
ctypes allows to call functions exposed from dlls/shared libraries
and has extensive facilities to create, access and manipulate
simple and complicated C data types in Python - in other words:
wrap libraries in pure Python.  It is even possible to implement C
callback functions in pure Python.

Comment 1 Ed Hill 2006-01-29 22:42:25 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...

Comment 2 Toshio Kuratomi 2006-03-04 04:50:43 UTC
ping.

(URLs are still inaccessible.)

Comment 3 Konstantin Ryabitsev 2006-03-04 05:26:26 UTC
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

Comment 4 Toshio Kuratomi 2006-03-04 20:02:12 UTC
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?

Comment 5 Paul Nasrat 2006-04-26 18:43:55 UTC
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.

Comment 6 Toshio Kuratomi 2006-04-26 19:35:44 UTC
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.

Comment 7 Paul Nasrat 2006-04-26 19:40:48 UTC
I'm happy to review this.

Comment 8 Konstantin Ryabitsev 2006-04-27 03:46:28 UTC
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. :)

Comment 9 Toshio Kuratomi 2006-04-27 16:11:36 UTC
Hmmm Paul - do you want to take over packaging and I'll review?  I'm only
swamped with work, not seriously swamped :-)

Comment 11 Toshio Kuratomi 2006-05-01 18:01:53 UTC
Hey Paul -- your srpm is only 4096 bytes so there's something wrong.

Comment 12 Paul Nasrat 2006-05-01 19:10:11 UTC
Should be better now - sha1sum:

f7a7efac9246e3195a6fdb39a10d84d5ed6ff6e6  python-ctypes-0.9.9.6-1.src.rpm


Comment 13 Toshio Kuratomi 2006-05-13 06:08:50 UTC
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.

Comment 14 Toshio Kuratomi 2006-05-13 06:11:58 UTC
Nice job nasrat and icon.  Sorry it took me so long to finish the review.

Comment 15 Warren Togami 2006-05-15 19:27:53 UTC
Please reassign this bug to the actual reviewer.

Comment 16 Jesse Keating 2007-12-21 15:51:52 UTC
Package Change Request
======================
Package Name: python-ctypes
New Branches: EL-5
Updated EPEL Owners: jkeating

Comment 17 Kevin Fenzi 2007-12-21 19:38:54 UTC
cvs done.

Comment 18 Matt Domsch 2011-02-03 15:33:49 UTC
Package Change Request
======================
Package Name: python-ctypes
New Branches: el4
Owners: mdomsch mebrown

Comment 19 Kevin Fenzi 2011-02-03 19:49:48 UTC
Git done (by process-git-requests).


Note You need to log in before you can comment on or make changes to this bug.