Skip to content

Fix macos ci#201

Merged
misonijnik merged 4 commits intomainfrom
misonijnik/fix-macos-ci
Oct 24, 2024
Merged

Fix macos ci#201
misonijnik merged 4 commits intomainfrom
misonijnik/fix-macos-ci

Conversation

@misonijnik
Copy link
Collaborator

No description provided.

With Python3.12, brew requires a virtualenv for external python packages.
Setup and use such an environment for testing under MacOS.

While there is a `tabulate` package in brew, installing it won't install the library only the command line tool `tabulate`.
Therefore, we install `tabulate` via `pip install`.

Remove unnecessary package upgrade.
The previous implementation gobbled up the first entry of the PATH by interpreting the second argument as pattern.
New implementation avoids this and is more robust in detecting existing KLEE installations.
@misonijnik misonijnik force-pushed the misonijnik/fix-macos-ci branch from d75e7dc to 2541da0 Compare October 23, 2024 09:45
@misonijnik misonijnik merged commit 6e2c242 into main Oct 24, 2024
metametamoon pushed a commit to metametamoon/klee that referenced this pull request Jun 21, 2025
* Use a virtual environment for Python3 while testing under MacOS

With Python3.12, brew requires a virtualenv for external python packages.
Setup and use such an environment for testing under MacOS.

While there is a `tabulate` package in brew, installing it won't install the library only the command line tool `tabulate`.
Therefore, we install `tabulate` via `pip install`.

Remove unnecessary package upgrade.

* Fix bug in PATH cleaning, more robust installation detection

The previous implementation gobbled up the first entry of the PATH by interpreting the second argument as pattern.
New implementation avoids this and is more robust in detecting existing KLEE installations.

* Mark variadic argument tests as requiring x86_64

* Update VarArgByValOld.c

---------

Co-authored-by: Martin Nowack <m.nowack@imperial.ac.uk>
(cherry picked from commit 6e2c242)
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