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

ReadMe, alive-tv --help: correct timeout, BUILD_SHARED_LIBS optional #992

Conversation

FlashSheridan
Copy link
Contributor

  • Note the -smt-to: option to the alive executable; corrects my c46e7ef.
  • Add “in ms” to alive-tv --help for -smt-to.
  • Note in build instructions that BUILD_SHARED_LIBS may not be necessary; it wasn’t for our build system, and caused severe problems when enabled.
    • Move its CMake problems to the instructions from Troubleshooting.
  • Convert alive-tv example instructions into text which can be triple-clicked and dragged. (I’ve started using it a lot; it found an important bug which had been fixed while I was fiddling with the build system.)
  • Convert Compiler Explorer instance URL into a Markdown clickable link. (Though I expect it will be made clickable anyway for most viewers by GitHub.)

may not be necessary; it wasn’t for our build system, and caused severe problems when enabled.
Move its CMake problems to the instructions from Troubleshooting.
into text which can be triple-clicked and dragged.  (I’ve started using it a lot; it found an important bug which had been fixed while I was fiddling with the build system.)
• Convert Compiler Explorer instance URL into a Markdown clickable link.  (Though I expect it will be made clickable anyway by GitHub for most viewers.)
README.md Outdated
@@ -63,7 +63,13 @@ Building and Running Translation Validation

Alive2's `opt` and `clang` translation validation requires a build of LLVM with
RTTI and exceptions turned on.
LLVM can be built targeting X86 in the following way. (You may prefer to add `-DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++` to the CMake step if your default compiler is `gcc`.)
LLVM can be built targeting X86 in the following way.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Better handled now by rikhuijzer’s #998.

@nunoplopes nunoplopes merged commit 64fd5fb into AliveToolkit:master Jan 3, 2024
15 checks 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