Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refuses to build on 0x0e0b04 (14.11.4) but appears to work #42

Closed
anderbubble opened this issue Mar 2, 2015 · 2 comments
Closed

refuses to build on 0x0e0b04 (14.11.4) but appears to work #42

anderbubble opened this issue Mar 2, 2015 · 2 comments

Comments

@anderbubble
Copy link
Contributor

I changed setup.py to allow building against 14.11.4, and it built successfully.

diff --git a/setup.py b/setup.py
index fb1aa70..1ed7e5b 100644
--- a/setup.py
+++ b/setup.py
@@ -24,7 +24,7 @@ logging.basicConfig(level=20)
 #VERSION = imp.load_source("/tmp", "pyslurm/__init__.py").__version__
 __version__ = "14.11.0-0"
 __min_slurm_hex_version__ = "0x0e0b00"
-__max_slurm_hex_version__ = "0x0e0b02"
+__max_slurm_hex_version__ = "0x0e0b04"

 def fatal(logstring, code=1):
        logger.error("Fatal: " + logstring)

Running the "Query Node List" example worked, too.

Can setup.py be updated to allow building against 14.11.4?

@gingergeeks
Copy link
Member

Anderbubble,
You are quite correct with the suggestion. I checked the Slurm 14.11.4 API when it was released and nothing has changed and the min-max hex version check in setup.py is designed for exactly this situation. I will add your change later tonight.

Best Regards

@anderbubble
Copy link
Contributor Author

Any chance I could do the full fork/commit/pull request dance to get my own
commit in the tree? It'd be a first for me. :)

~jonathon

On Tue, Mar 3, 2015 at 1:23 AM, Gingergeek notifications@github.com wrote:

Anderbubble,
You are quite correct with the suggestion. I checked the Slurm 14.11.4 API
when it was released and nothing has changed and the min-max checked in
setup.py is designed for exactly this situation. I will add your change
later tonight.

Best Regards


Reply to this email directly or view it on GitHub
#42 (comment).

anderbubble pushed a commit to anderbubble/pyslurm that referenced this issue Mar 3, 2015
Fixes PySlurm#42

The Slurm API hasn't changed, so the existing code already
supports the latest Slurm release, 14.11.4.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants