Skip to content

clingo: fix i out of n#40861

Closed
haampie wants to merge 1 commit intospack:developfrom
haampie:clingo-unconditionally-depend-on-setuptools
Closed

clingo: fix i out of n#40861
haampie wants to merge 1 commit intospack:developfrom
haampie:clingo-unconditionally-depend-on-setuptools

Conversation

@haampie
Copy link
Copy Markdown
Member

@haampie haampie commented Nov 3, 2023

The old concretizer just drops py-setuptools. Add it unconditionally then.

Might get us back to the issue with Python 3.6 :) 🎉

@haampie
Copy link
Copy Markdown
Member Author

haampie commented Nov 3, 2023

Sigh. @alalazo suggestions? Either python 3.6 fails cause the old concretizer doesn't know how to deal with conflicts on setuptools, or it just entirely ignores the conditional depends on for 3.12.

@alalazo
Copy link
Copy Markdown
Member

alalazo commented Nov 3, 2023

Wondering if we can use a variant, and construct the specs in a way that depends on the interpreter version when bootstrapping.

@alalazo
Copy link
Copy Markdown
Member

alalazo commented Nov 3, 2023

Also, it seems GA is in the middle of a migration, so we're getting Python versions randomly on macOS. This one https://github.com/spack/spack/actions/runs/6745185158/job/18336769272?pr=40861 (from the PR) is Python 3.11

@haampie
Copy link
Copy Markdown
Member Author

haampie commented Nov 3, 2023

You can try concretization locally I guess

@haampie
Copy link
Copy Markdown
Member Author

haampie commented Nov 3, 2023

Bit frustrating that Spack can barely self-bootstrap :)

@alalazo
Copy link
Copy Markdown
Member

alalazo commented Nov 3, 2023

Bit frustrating that Spack can barely self-bootstrap :)

Yes, it's frustrating to support Python 3.6 and Python 3.12 together. But bootstrapping started failing on a single case over eleven tested, so "can barely self-bootstrap" seems a bit too much 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants