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

Use a relative src dir path in containers-tests instead of a symlink. #886

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

Conversation

AndreasPK
Copy link
Contributor

@AndreasPK AndreasPK commented Dec 23, 2022

This should make working on containers easier when on windows.

Fixes #885

This should make working on containers easier when on windows.
@AndreasPK
Copy link
Contributor Author

Worked for me locally not sure why this failes on CI.

@treeowl
Copy link
Contributor

treeowl commented Dec 23, 2022

Let me know when you get it working and I'll merge.

@treeowl
Copy link
Contributor

treeowl commented Jan 9, 2023

Any further ideas?

@AndreasPK
Copy link
Contributor Author

Best I can tell CI runs the actual testsuite by moving the container-tests folder to some other place which invalidates the ../containers relative path.

I couldn't yet find the motivation to dig into the ci config. If I run into the issue again I probably will but it's not urgent for me personally.

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.

intset-benchmarks no longer build on windows.
2 participants