Skip to content

Build cvc5 as an unrestricted build - #124

Merged
daniel-larraz merged 1 commit into
cvc5:mainfrom
daniel-larraz:ci-unrestricted-build-type
Sep 5, 2026
Merged

Build cvc5 as an unrestricted build#124
daniel-larraz merged 1 commit into
cvc5:mainfrom
daniel-larraz:ci-unrestricted-build-type

Conversation

@daniel-larraz

@daniel-larraz daniel-larraz commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

cvc5/cvc5#12899 renamed the build types configure.sh accepts and made selecting one mandatory:

before after
production unrestricted
safe-mode safe
stable-mode stable

The old names are rejected outright rather than silently ignored, so the Build cvc5 step fails at once:

*** configure.sh: build type 'production' is no longer available. Use 'unrestricted' to have the previous behavior. Alternatively, the build types 'safe' and 'stable' are available (try -h)

This PR uses the new name. It is the same build, just spelled differently, so nothing else has to change.

One side effect: the cache keys track $CVC5_CONFIG, so this invalidates the cvc5 artifact and dependency caches once. The next run repopulates them.

🤖 Generated with Claude Code

cvc5/cvc5#12899 renamed the build types configure.sh accepts and made
selecting one mandatory: production became unrestricted, safe-mode
became safe, and stable-mode became stable. The old names are rejected
outright rather than silently ignored, so the build step now fails at
once:

  *** configure.sh: build type 'production' is no longer available.
  Use 'unrestricted' to have the previous behavior.

The step that checks out cvc5 names no ref, so it takes the default
branch, and the artifact cache is keyed on the SHA it resolves to. The
build therefore runs on every cvc5 commit, and CI has been broken since
the rename landed.

Use the new name. It is the same build, just spelled differently, so
nothing else has to change.

The cache keys track $CVC5_CONFIG, so this invalidates the cvc5
artifact and dependency caches once. The next run repopulates them.

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>

@alex-ozdemir alex-ozdemir left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@daniel-larraz
daniel-larraz merged commit 726058d into cvc5:main Sep 5, 2026
1 check passed
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

Successfully merging this pull request may close these issues.

2 participants