-
-
Notifications
You must be signed in to change notification settings - Fork 359
Use Scons in CI with container #959
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
Conversation
Other than the fact that the container doesn't have cargo it works |
Alright, let's try and I'll revert if it doesn't run correctly |
@ntindle here is the full problem: The probable cause is as follows:
As such, the solution is to do the path manipulation ourselves in the Dockerfile to ensure consistent paths, even with |
Yes, I forgot to end a "
reminds me of my youth
but once you do, its home please squash this when merging
in the docker options?
this keeps tech debt in place. Consult actions/runner#1525 for fixes
Okay if whoever merges this doesn't squash and merge, I will cry. THank you for coming to my ted talk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, this looks fine for me; however, I am not sure how to test this properly.
I am happy to merge though, I trust your judgement
To test, read the details of an action and see if scons passes |
No description provided.