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

Add mention of -no-haskell-binary to backends tutorial #2863

Merged
merged 3 commits into from
Sep 12, 2022

Conversation

Baltoli
Copy link
Contributor

@Baltoli Baltoli commented Sep 9, 2022

We had a question in the public K channel about this; we should mention the issue in the tutorial for the time being given that it's long-term enough to warrant its own flag.

@Baltoli Baltoli requested a review from radumereuta September 9, 2022 09:55
Copy link
Contributor

@radumereuta radumereuta left a comment

Choose a reason for hiding this comment

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

I made a suggestion to mention the Compact library so it's easier for users to link the issue to the solution.

Also, should we make the default for the M1 macs be --no-haskell-binary? We might need a way to turn it off though which would complicate things a bit.

k-distribution/k-tutorial/1_basic/20_backends/README.md Outdated Show resolved Hide resolved
Co-authored-by: Radu Mereuta <headness13@gmail.com>
@Baltoli
Copy link
Contributor Author

Baltoli commented Sep 12, 2022

I think given that this is (hopefully) a transient bug we can leave the flag optional for now.

@rv-jenkins rv-jenkins merged commit e6fdf47 into master Sep 12, 2022
@rv-jenkins rv-jenkins deleted the add-no-binary branch September 12, 2022 11:17
h0nzZik pushed a commit to h0nzZik/k that referenced this pull request Nov 24, 2022
…untimeverification#2260)

* haskell-backend/src/main/native/haskell-backend: c3296f3e - Update dependency: deps/k_release (runtimeverification#2867)

* haskell-backend/src/main/native/haskell-backend: 7cc4dafc - Update dependency: deps/k_release (runtimeverification#2868)

* haskell-backend/src/main/native/haskell-backend: 3b17155a - Update regression tests (runtimeverification#2863)

* haskell-backend/src/main/native/haskell-backend: b7001f5b - Update regression tests (runtimeverification#2871)

* haskell-backend/src/main/native/haskell-backend: 395b9ec3 - Allow parsing of multi-arg \\or with \\left-assoc (runtimeverification#2861)
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.

3 participants