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

Check that everything works with ptracer and pt-run from /usr/local #20

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hugomg
Copy link
Member

@hugomg hugomg commented Aug 10, 2024

Get rid of the "../../pt-run", by adding it to the PATH. The tricky bit is that ./run-tests no longer is a trivial wrapper around busted, because it sets the PATH. Thus, I'm not 100% sold that we actually want to do this. Anyway, I'm leaving this here for feedback.

It's inconvenient to do make install every time we want to run the test suite. So I set it up to run the /usr/local tests only in the Github CI. To test that the compilation works with /usr/local version of ptracer.h, clear the CPPFLAGS variable when compiling the tests.

It's inconvenient to do make install every time we want to run the test suite.
So I set it up to run the /usr/local tests only in the Github CI. To test that
the compilation works with /usr/local version of ptracer.h, clear the CPPFLAGS
variable when compiling the tests.
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.

1 participant