-
-
Notifications
You must be signed in to change notification settings - Fork 319
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
Updated taskmaster to check srcnode for variant dir files without builders #4153
base: master
Are you sure you want to change the base?
Conversation
CI failed the "Linux Testing" build with an oddity - I've never seen F77PATH.py fail. The test log isn't terribly informative, but seems to be building something when no build was expected - the test line (#152) is: test.up_to_date(arguments = args) and from the log of what actually ran there was: ...
> scons: `variant/prog' is up to date.
> f77 -o subdir/prog.o -c -x f77 -Isubdir/include -Isubdir -Isubdir subdir/prog.f77
> gfortran -o subdir/prog subdir/prog.o
> scons: done building targets. |
Yeah its weird because when I run the test with manual commands, I can't reproduce it. Been looking into it, but because I can not reproduce it manually, I am force to debug through the test which is not very productive. When I added --debug=explain, it said rebuilding because it doesn't exist, that's where I am at now. |
Yes, that's a real pain. @bdbaddog has in the past tried to suggest some techniques for interacting with the CI, which I've never gotten to work. |
Do you have an f77 command? I don't.... I'm not used to the command even running. Github isn't as clear on how the images are provisioned as AppVeyor is (or as Travis was). |
I should be more clear, I can reproduce this via the test on a local machine, which is much better than needing to debug through CI. However I am perplexed how me manually running the same commands works, but the test running them fails. |
Usually, that means Environment vars sound like they're a red herring:
|
fixes #2908
https://discord.com/channels/571796279483564041/971831743944491008/976472063990718494
Contributor Checklist:
CHANGES.txt
(and read theREADME.rst
)