Bug 331081 - Review Request: python-pp - Parallel execution of python on smp
Summary: Review Request: python-pp - Parallel execution of python on smp
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michel Lind
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-10-14 01:09 UTC by Steve Milner
Modified: 2009-09-29 20:27 UTC (History)
2 users (show)

Fixed In Version: 1.4.5-3.fc8
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-12-06 20:46:44 UTC
Type: ---
Embargoed:
michel: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Steve Milner 2007-10-14 01:09:19 UTC
Spec URL: http://stevemilner.org/tmp/python-pp.spec
SRPM URL: http://stevemilner.org/tmp/python-pp-1.4.5-1.fc7.src.rpm
Description: 
pp is a python module which provides mechanism for parallel execution of 
python code on SMP (systems with multiple processors or cores) and clusters
(computers connected via network).

[steve@psycho SPECS]$ rpmlint /home/steve/rpmbuild/RPMS/noarch/python-pp-1.4.5-1.fc7.noarch.rpm
[steve@psycho SPECS]$

Comment 1 Michel Lind 2007-10-14 02:46:19 UTC
Two things:
- Files in %{_bindir} should not have .py extension, so that they won't be
byte-compiled. The byte-compiled files should be excluded:

http://fedoraproject.org/wiki/Packaging/Python

- The if-block you have in %install tried to move files from /usr/bin to
%{_bindir}, which in all Fedora/RHEL installs I've seen are the same place?

Comment 2 Steve Milner 2007-10-14 18:38:01 UTC
Updated ...

Spec URL: http://stevemilner.org/tmp/python-pp.spec
SRPM URL: http://stevemilner.org/tmp/python-pp-1.4.5-2.fc7.src.rpm

[steve@psycho SPECS]$ rpmlint
/home/steve/rpmbuild/RPMS/noarch/python-pp-1.4.5-2.fc7.noarch.rpm 
[steve@psycho SPECS]$ 



Comment 3 Steve Milner 2007-10-21 23:59:32 UTC
Is there anything else that needs updated?

Comment 4 Michel Lind 2007-11-03 19:07:11 UTC
The following are unnecessary:
* -O1 flag for in %install (This is done automatically)
* Minimum version requirement (all current Fedora versions have Python >= 2.3)

(unless you're targeting RHEL 4 as well:
http://fedoraproject.org/wiki/Packaging/Python)

APPROVED


Comment 5 Michel Lind 2007-11-13 08:04:36 UTC
Ping?

Comment 6 Steve Milner 2007-11-15 03:04:07 UTC
Sorry I got side tracked :-)

Comment 7 Steve Milner 2007-11-15 03:08:07 UTC
New Package CVS Request
=======================
Package Name: python-pp
Short Description: pp is a python module which provides mechanism for parallel
execution of python code on SMP and clusters.
Owners: smilner
Branches: F-7 F-8 EL-5
Cvsextras Commits: yes

Comment 8 Kevin Fenzi 2007-11-15 17:14:07 UTC
cvs done.

Comment 9 Michel Lind 2007-11-18 18:28:41 UTC
Steve, have you gone to the Fedora Update System and push the builds for Fedora
7 and 8? They are still not showing up:

https://admin.fedoraproject.org/updates/search/python-pp

You can toggle the switch so that this bug entry is automatically closed when
the Fedora 8 package is marked stable.

Comment 10 Steve Milner 2007-11-23 03:14:35 UTC
They are now listed. Thanks for the tip about automatically closing ... I didn't
notice that in the past :-).

Comment 11 Fedora Update System 2007-11-26 18:50:59 UTC
python-pp-1.4.5-3.fc8 has been pushed to the Fedora 8 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update python-pp'

Comment 12 Fedora Update System 2007-12-06 20:46:42 UTC
python-pp-1.4.5-3.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 13 Steve Milner 2009-09-29 19:13:34 UTC
Package Change Request
======================
Package Name: python-pp
New Branches: EL-4
Owners: smilner

Comment 14 Kevin Fenzi 2009-09-29 20:27:00 UTC
cvs done.


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