Bug 631964

Update to 4.3.0:

http://cern.ch/straylen/rpms/pbs_python/pbs_python.spec
http://cern.ch/straylen/rpms/pbs_python/pbs_python-4.1.0-2.fc13.src.rpm

Update to 4.3.0:

http://cern.ch/straylen/rpms/pbs_python/pbs_python.spec
http://cern.ch/straylen/rpms/pbs_python/pbs_python-4.3.0-1.fc13.src.rpm

Update comment on python3 support to reflect upstream bug on the matter.
No actual change to package.

http://cern.ch/straylen/rpms/pbs_python/pbs_python.spec
http://cern.ch/straylen/rpms/pbs_python/pbs_python-4.3.0-2.fc13.src.rpm

Taking for review.

Instead of pbs_python, I think it would be more natural to call this package python-pbs and name the subpackages respectively python26-pbs and python3-pbs.
Rationale:
 - Python naming guidelines suggest to use the format python-$NAME [1];
 - Naming guidelines prefer '-' as separator [2];
 - If Debian eventually gets around to packaging this, then the binary package
   will be named python-pbs as their guidelines are stricter. For consistency
   it might be nice to keep the naming similar.

[1] http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Addon_Packages_.28python_modules.29
[2] http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Separators


There are lots of conditionals similar to this:
%if %{?rhel}%{!?rhel:0} == 5
I'd suggest to use the following, more readable conditional instead:
%if 0%{?rhel} == 5

See http://fedoraproject.org/wiki/Packaging/DistTag#Conditionals


> # We need to filter _pbs.so out from the provides.
> Source100:     filter-provides.sh
> %global        _use_internal_dependency_generator 0
> %global        __find_provides /bin/bash %{SOURCE100}

I didn't really read the filter-provides.sh, but if you are just trying to filter the _pbs.so Provides from Python private directory, it's easier to use:
%{?filter_provides_in: %filter_provides_in %{python_sitearch}/pbs/.*\.so}
%{?filter_setup}

https://fedoraproject.org/wiki/Packaging:AutoProvidesAndRequiresFiltering#Preventing_files.2Fdirectories_from_being_scanned_for_provides_.28pre-scan_filtering.29

Kalev,

Thanks for the comments:

+ I've renamed to python-pbs.
+ The conditionals have been changed to be as per dist tag example page.
+ I'm using the filter_provides_in macro. Was not aware of that one, thanks.
  (It does not work on RHEL5 but given the state of even the python package
   I don't feel it's significant.)

http://cern.ch/straylen/rpms/pbs_python/python-pbs.spec
http://cern.ch/straylen/rpms/pbs_python/python-pbs-4.3.0-3.fc13.src.rpm

Steve.

Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2625799

Fedora review python-pbs-4.3.0-3.fc13.src.rpm 2010-11-25

+ OK
! needs attention

rpmlint output:
$ rpmlint python-pbs \
          python-pbs-4.3.0-3.fc15.src.rpm \
          python-pbs-debuginfo-4.3.0-3.fc15.i686.rpm
3 packages and 0 specfiles checked; 0 errors, 0 warnings.

+ Rpmlint is quiet
+ The package is named according to the Package Naming Guidelines.
+ Spec file name matches the base package name
+ The package is licensed with a Fedora approved license and meets the
  Licensing Guidelines.
+ The license field in the spec file matches the actual license
! The license file (LICENSE.openpbs) isn't included in the package
+ Spec file is written in American English
+ Spec file is legible
+ Following instructions in the spec file to check out sources
+ Upstream sources match sources in the srpm. md5sum:
  34fada5c56322fac6254d6bf9c9c9106  pbs_python-4.3.0.tar.gz
  34fada5c56322fac6254d6bf9c9c9106  Download/pbs_python-4.3.0.tar.gz
+ The package builds in koji
n/a ExcludeArch bugs filed
+ BuildRequires look sane
n/a The spec file MUST handle locales properly
n/a ldconfig in %post and %postun
+ Package does not bundle copies of system libraries
n/a Package isn't relocatable
+ Package owns all directories it creates
+ No duplicate files in %files
+ Permissions are properly set and %files has %defattr
+ Consistent use of macros
+ The package must contain code, or permissable content.
n/a Large documentation files should go in -doc subpackage
+ Files marked %doc don't affect the package
n/a Header files should be in -devel
n/a Static libraries should be in -static
n/a Library files that end in .so must go in a -devel package
n/a -devel must require the fully versioned base
+ Package doesn't contain any libtool .la files
n/a Packages containing GUI apps must include %{name}.desktop file
+ Directory ownership sane
+ Filenames are valid UTF-8


MUSTFIX:
 - Include the LICENSE.openpbs file in %doc for both subpackages

Some additional nitpicking:
 - Also consider including the AUTHORS and CHANGES files in %doc
 - python-pbs and %{altpython}-pbs subpackages have different Group tags; might
   make sense to put them both into same group. Also, if you use same group,
   then you only have to specify Group tag for main package and the subpackage
   inherits it automatically.
 - %descriptions for both subpackages still refer to 'pbs_python' package name

(In reply to comment #7)

> MUSTFIX:
>  - Include the LICENSE.openpbs file in %doc for both subpackages
> 
> Some additional nitpicking:
>  - Also consider including the AUTHORS and CHANGES files in %doc
>  - python-pbs and %{altpython}-pbs subpackages have different Group tags; might
>    make sense to put them both into same group. Also, if you use same group,
>    then you only have to specify Group tag for main package and the subpackage
>    inherits it automatically.
>  - %descriptions for both subpackages still refer to 'pbs_python' package name

All these comments are valid and should have been trivial for me to have seen..

Thanks.


%changelog
* Sat Nov 27 2010 Steve Traylen <steve.traylen> 4.3.0-4
- Finish renaming package to python-pbs.  rhbz#657027
- Add LICENSE.openpbs and CHANGES files to doc. rhbz#657027
- Consistantly group altpython package in same group.

http://cern.ch/straylen/rpms/pbs_python/python-pbs.spec
http://cern.ch/straylen/rpms/pbs_python/python-pbs-4.3.0-4.fc13.src.rpm

Looks good.

APPROVED

New Package SCM Request
=======================
Package Name: python-pbs
Short Description: PBS/Torque python module
Owners: stevetraylen
Branches: el4 el5 el6 f13 f14 
InitialCC:

Many thanks for the review.

It will go into all branches unmodified except el4 where I will have
to change the %global for %define but I'll do that only in the el4 branch
I think.

Git done (by process-git-requests).

python-pbs-4.3.0-5.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/python-pbs-4.3.0-5.fc13

python-pbs-4.3.0-5.el4.1 has been submitted as an update for Fedora EPEL 4.
https://admin.fedoraproject.org/updates/python-pbs-4.3.0-5.el4.1

python-pbs-4.3.0-5.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/python-pbs-4.3.0-5.fc14

python-pbs-4.3.0-5.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/python-pbs-4.3.0-5.el5

python-pbs-4.3.0-5.el4.1 has been pushed to the Fedora EPEL 4 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-pbs'.  You can provide feedback for this update here: https://admin.fedoraproject.org/updates/python-pbs-4.3.0-5.el4.1

python-pbs-4.3.0-5.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.

python-pbs-4.3.0-5.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.

python-pbs-4.3.0-5.el5 has been pushed to the Fedora EPEL 5 stable repository.  If problems still persist, please make note of it in this bug report.

python-pbs-4.3.0-5.el4.1 has been pushed to the Fedora EPEL 4 stable repository.  If problems still persist, please make note of it in this bug report.