Spec URL: http://cern.ch/straylen/rpms/pbs_python/pbs_python.spec SRPM URL: http://cern.ch/straylen/rpms/pbs_python/pbs_python-4.1.0-2.fc13.src.rpm Description: The pbs_python package is a wrapper class for the Torque C library. With this package you now can write utilities/extensions in Python instead of C.
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.