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

creusot-setup: Print issues on stderr instead of stdout when bailing #1335

Merged
merged 1 commit into from
Feb 3, 2025

Conversation

Lysxia
Copy link
Collaborator

@Lysxia Lysxia commented Feb 1, 2025

@xldenis that would provide a more informative error message in #1294 (comment)

@xldenis
Copy link
Collaborator

xldenis commented Feb 1, 2025

Testing creusot-contracts ... failure
Error: Why3 has version 1.7.2, expected version is 1.8.0
Error: why3find has version 1.0.0, expected version is 1.1.1
Error: Please run 'cargo creusot setup status' to diagnostic and fix the issue(s)
thread 'main' panicked at creusot/tests/creusot-contracts.rs:120:9:
output of creusot-contracts failed
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

ah! opam strikes again.

@Lysxia Lysxia merged commit cb81007 into master Feb 3, 2025
6 checks passed
@Lysxia Lysxia deleted the setup-eprintln branch February 3, 2025 09:41
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