[Bug 1028165] Review Request: globus-gram-job-manager-slurm - Globus Toolkit - SLURM Job Manager Support

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=1028165



--- Comment #4 from Mattias Ellert <mattias.ellert at fysast.uu.se> ---
(In reply to Adrien Devresse from comment #3)

> Issues:
> =======
> - Contains file under BSD license but only ASL 2.0 is signaled

A very valid complaint. I had already fixed this locally, but somehow forgot
the update the review request. Which of course was entirely my fault.

> - no ownership defined on /etc/globus

Oops. /etc/globus was supposed to be owned by globus-gram-job-manager which
this package requires. Somehow that got lost somewhere. I have submitted an
update of globus-gram-job-manager that properly owns the /etc/globus directory
to rectify this.

> - Macro could be used in "/usr/bin/srun", "/usr/bin/sbatch", etc.

Thanks - applied.

New version:

http://www.grid.tsl.uu.se/review/globus-gram-job-manager-slurm-1.2-2.fc19.src.rpm
http://www.grid.tsl.uu.se/review/globus-gram-job-manager-slurm.spec

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component