Skip to content

MSVC preview version breaks clingo build#41185

Merged
alalazo merged 2 commits intospack:developfrom
johnwparent:win/prevent-msvc-ver-breaking-clingo-bootstrap
Nov 20, 2023
Merged

MSVC preview version breaks clingo build#41185
alalazo merged 2 commits intospack:developfrom
johnwparent:win/prevent-msvc-ver-breaking-clingo-bootstrap

Conversation

@johnwparent
Copy link
Copy Markdown
Contributor

Prevent preview versions from being used for bootstrapping clingo until patch is ready

Prevent preview versions from being used for bootstrapping clingo until
patch is ready
@johnwparent johnwparent requested a review from alalazo November 20, 2023 19:43
@spackbot-app spackbot-app bot added the core PR affects Spack core functionality label Nov 20, 2023
@johnwparent
Copy link
Copy Markdown
Contributor Author

johnwparent commented Nov 20, 2023

@alalazo This should resolve the bootstrapping issue, 19.38 is the MSC_VER that adds the breaking change to the standard library, so we just bootstrap with 19.37 now.

Edit: clingo patch coming soon, just don't want to hold any other PRs up on this.

Use correct version constraints

Co-authored-by: Harmen Stoppels <[email protected]>
@alalazo
Copy link
Copy Markdown
Member

alalazo commented Nov 20, 2023

Thanks!

@alalazo alalazo merged commit 5154d69 into spack:develop Nov 20, 2023
haampie added a commit that referenced this pull request Nov 23, 2023
@haampie haampie mentioned this pull request Nov 23, 2023
36 tasks
gabrielctn pushed a commit to gabrielctn/spack that referenced this pull request Nov 24, 2023
mtaillefumier pushed a commit to mtaillefumier/spack that referenced this pull request Dec 14, 2023
alalazo pushed a commit that referenced this pull request Jan 10, 2024
RikkiButler20 pushed a commit to RikkiButler20/spack that referenced this pull request Jan 31, 2024
vjranagit pushed a commit to vjranagit/spack that referenced this pull request Jan 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core PR affects Spack core functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants