Spec URL: http://coral.thiell.com/uploads/clustershell-1.3.spec SRPM URL: http://pypi.python.org/packages/source/C/ClusterShell/clustershell-1.3-1.fc12.src.rpm Description: Event-based Python library to execute commands on cluster nodes in parallel depending on the selected engine and worker mechanisms. The library provides also advanced NodeSet and NodeGroups handling methods to ease and improve the administration of large compute clusters or server farms. Also, clush, clubak and nodeset, three convenient command-line tools are included. Note: ClusterShell and especially its useful "clush" and "nodeset" commands are currently being used in production on several large Red Hat-based clusters at CEA in France and by some other HPC sites worldwide that uses Shine, a command line tool designed to setup and manage the Lustre file system. We would like to see ClusterShell added to Fedora Everything. I will do my best to meet your packaging requirements. This is my first package and I need a sponsor! Thanks.
I've incluced the library API epydoc (HTML) documentation in the RPM itself, but it's quite large (6.2M) and should not always be needed. So the resulting RPM size is 1.8MB on F12. Building a RPM without the epydoc documentation makes it 128KB only so it's probably better. Maybe I should separate the HTML documentation in another RPM (like clustershell-docs, clustershell-apidoc or something similiar), or just give a pointer to a web site to browse and download API documentation. Advice welcomed! Be my sponsor!
Seems like you have used python distutils to create the spec file. Don't do that, it produce a lot a unwanted stuff. See my comments and advice in bug #620862 You also need to add vim as buildreq, don't compress mans (rpm will do for you). List mans like this (rpm will mark as %doc itself): %{_mandir}/man1/clubak.1* Use of --record=INSTALLED_FILES is not allowed. Again, read comments in bug #620862
Thank you Terje for this first review. I've been able to fix many things, please take a look at the release 2: Spec URL: http://coral.thiell.com/uploads/fedora/2/clustershell.spec SRPM URL: http://coral.thiell.com/uploads/fedora/2/clustershell-1.3-2.fc11.src.rpm I've also removed the epydoc HTML doc from this RPM as it's pretty big and there is a way to access the library documentation with pydoc. rpmlint result: clustershell.noarch: W: file-not-utf8 /usr/share/doc/clustershell-1.3/Licence_CeCILL-C_V1-fr.txt clustershell.noarch: W: file-not-utf8 /usr/share/doc/clustershell-1.3/Licence_CeCILL-C_V1-en.txt 1 packages and 0 specfiles checked; 0 errors, 2 warnings.
After reading other package submissions, I fixed some issues in this one (sorry for that, I should have done that before): - Source0 is now a full URL to sourceforge package, so that we can check for upstream source matching (it's ok) - converted license files text-encoding from latin1 to UTF-8 (with iconv) so that rpmlint reports no error at all: $ rpmlint clustershell-1.3-3.fc11.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. Updated files: Spec URL: http://coral.thiell.com/uploads/fedora/3/clustershell.spec SRPM URL: http://coral.thiell.com/uploads/fedora/3/clustershell-1.3-3.fc11.src.rpm
Nice software... I'm no sponsor, so just some comments for now: - it would be much more readable, if you use the same indentation everywhere (but that's just costmetic) - Buildroot tag is wrong: https://fedoraproject.org/wiki/EPEL/GuidelinesAndPolicies#BuildRoot_tag But if you don't want to make this a package in el5, you could also delete it completely. - Please preserve timestamps, when copying files around (cp -a or install -p). - First line: prefer global vs define, See: https://fedoraproject.org/wiki/Packaging/Guidelines#.25global_preferred_over_.25define You could use this macro: https://fedoraproject.org/wiki/Packaging:Python#Macros - Please be a bit more explicit in %files: When using: %{python_sitelib}/ClusterShell/ %{python_sitelib}/ClusterShell-*-py?.?.egg-info you know, when building the egg failed.
Many thanks Thomas for your review. Updated files: Spec URL: http://coral.thiell.com/uploads/fedora/4/clustershell.spec SRPM URL: http://coral.thiell.com/uploads/fedora/4/clustershell-1.3-4.fc11.src.rpm Notes: - I want to keep BuildRoot for now so that the spec can be used for el5 also. But I fixed it. - I changed the %define for a %global definition for pythin_sitelib, as you told me. I took the recommended macro but removed the pythin_sitearch definition as I'm not using it.
Hi Stephane, your package already looks pretty good. Here are just a couple of further things to consider: - it's sufficient to use "%setup -q" in %prep - I recommend to place the vim files in %{_datadir}/vim/vimfiles/syntax and %{_datadir}/vim/vimfiles/ftdetect, respectively. Thus, it's not necessary to detect the current vim version and the corresponding folder name. Also, you can drop BR: vim-common - For proper directory ownership, I suggest to create a -vim subpackage containing the .vim files. This package should require vim-common in order to create the vim data directories. - Your package should also own the directory %{_sysconfdir}/clustershell and not only its contents. Otherwise, rpm won't remove the directory when uninstalling the package. Hence, add %dir %{_sysconfdir}/clustershell to the %files section.
Thanks Martin, much appreciated. Updated files (also for new version 1.3.1): Spec URL: http://coral.thiell.com/uploads/fedora/5/clustershell.spec SRPM URL: http://coral.thiell.com/uploads/fedora/5/clustershell-1.3.1-1.fc13.src.rpm
I'll be sponsoring Stephane. Before I accept the request, there's something Martin asked for that you did not do in the last version: the vim subpackage. That would allow clustershell itself not to depend on vim. Thanks a lot to all those who participated in this review.
Sorry about that, I've uploaded updated files: Spec URL: http://coral.thiell.com/uploads/fedora/6/clustershell.spec SRPM URL: http://coral.thiell.com/uploads/fedora/6/clustershell-1.3.1-2.fc13.src.rpm The spec file now includes a -vim subpackage. Thank you, Aurélien.
Looks good, thanks.
New Package SCM Request ======================= Package Name: clustershell Short Description: Python framework for efficient cluster administration Owners: sthiell Branches: f12 f13 f14 el5 el6 InitialCC:
Git done (by process-git-requests).
clustershell-1.3.2-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/clustershell-1.3.2-1.fc14
clustershell-1.3.2-1.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/clustershell-1.3.2-1.fc13
clustershell-1.3.2-1.fc12 has been submitted as an update for Fedora 12. https://admin.fedoraproject.org/updates/clustershell-1.3.2-1.fc12
clustershell-1.3.2-1.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/clustershell-1.3.2-1.el5
clustershell-1.3.2-1.el5 has been pushed to the Fedora EPEL 5 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 clustershell'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/clustershell-1.3.2-1.el5
clustershell-1.3.2-1.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report.
clustershell-1.3.2-1.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.
clustershell-1.3.2-1.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
clustershell-1.3.2-1.el5 has been pushed to the Fedora EPEL 5 stable repository. If problems still persist, please make note of it in this bug report.