Red Hat Bugzilla – Bug 467489
Review Request: nwsclient - NetWorkSpaces Client for Python
Last modified: 2008-12-13 10:05:24 EST
Spec URL: http://www.auroralinux.org/people/spot/review/new/nwsclient.spec
SRPM URL: http://www.auroralinux.org/people/spot/review/new/nwsclient-1.6.3-1.fc10.src.rpm
NetWorkSpaces (NWS) is a powerful, open-source software package that makes it
easy to use clusters from within scripting languages like Python, R, and
Matlab. It uses a Space-based approach, similar to JavaSpaces (TM) for example,
that makes it easier to write distributed applications.
NetWorkSpaces for Python is the Python API to the NetWorkSpaces server. It
allows different Python scripts to communicate and coordinate with each other,
and (with some restrictions) with scripts written in other languages, such as
R and Matlab. The restriction is that only strings can be passed between
different languages. NetWorkSpaces doesn't provide a standard way to serialize
objects between different languages, but by allowing strings to be used,
programmers can choose their own mechanism (XML or YAML, for example).
Here are the results of my review on this package:
* rpmlint says:
nwsclient.noarch: W: non-executable-in-bin /usr/bin/PythonNWSSleighWorker.pyc 0644
nwsclient.noarch: W: non-executable-in-bin /usr/bin/PythonNWSSleighWorker.pyo 0644
These files should be excluded. Please see:
* Since this package is a python module, we have to obey the python module naming guidelines:
? Shall we package the README* files?
? Why is only a subset of the scripts in the scripts directory installed?
? It would be nice if you make a %check section and run as many possible tests as possible from the test directory.
* Source0 gives 404
* Providing Eggs for non-setuptools packages is slightly different than for setuptools packages. Please follow:
This package isn't a python module, its a python application that includes its own module, thus, it doesn't need to be renamed.
The tests can't be run without an nwsserver, so they're not overly useful in %check.
The README files are packaged, look closely. :)
The PySleighService.py isn't installed on Linux because it is windows specific. All the other scripts are installed.
I fixed the Source0 URL, used setuptools-devel, and got rid of the byte compiled files.
New SRPM: http://www.auroralinux.org/people/spot/review/new/nwsclient-1.6.3-2.fc11.src.rpm
New SPEC: http://www.auroralinux.org/people/spot/review/new/nwsclient.spec
All good, except I have an issue with the naming. I know that the package contains a python application, but the doc files, especially
gave me the impression that this package's main purpose is usage as a python module. Am I wrong?
The main purpose of this package is to provide a python client to communicate with the nwsserver. The module enables you to write python scripts to send jobs to the nwsserver, but it is primarily the python client.
ok, convinced :)
This package (nwsclient) is APPROVED by oget
New Package CVS Request
Package Name: nwsclient
Short Description: NetWorkSpaces Client for Python
Branches: F-9 F-10 devel
... and it's done.
nwsclient-1.6.3-2.fc9 has been submitted as an update for Fedora 9.
nwsclient-1.6.3-2.fc10 has been submitted as an update for Fedora 10.
nwsclient-1.6.3-2.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report.
nwsclient-1.6.3-2.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.