PySlurm / pyslurm

Python Interface to Slurm
https://pyslurm.github.io
GNU General Public License v2.0
474 stars 116 forks source link

Add publish actions #270

Open multimeric opened 1 year ago

multimeric commented 1 year ago
multimeric commented 1 year ago

I need some help adding the command we use for building wheels. python setup.py sdist_wheel doesn't seem to work.

multimeric commented 1 year ago

I think this will need an update to: https://github.com/giovtorres/docker-centos7-slurm before it will work.

tazend commented 1 year ago

@multimeric

I've already worked on improving the docker-centos7-slurm container setup a bit and adding one for 22.05 - I'll try to get the changes up this weekend as a PR.

In regards to building: I wouldn't ship a prebuilt wheel right now. I would just upload the source distribution as tar.gz, by doing python setup.py sdist.

The User is expected to have a valid libslurm.so installed on the System where pyslurm should run (and potentially point the pyslurm installation to it if it is a custom slurm installation in a unusual path). When a user executes pip install pyslurm, the source will be cythonized and compiled on the Users specific system.

multimeric commented 1 year ago

Great, thanks for the update. Agreed that we don't want to ship binary wheels. I for some reason thought there were source wheels but maybe there aren't, and it's just sdist that we want.