-
Notifications
You must be signed in to change notification settings - Fork 90
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
Faster recompile #2294
Faster recompile #2294
Conversation
By splitting the __TIME__ and __DATE__ to a different header, it makes it much faster to compile. This is especially beneficial in a edit-compile-debug cycle.
* CMake needed new file * avoid race condition in autotools
clang-tidy review says "All clean, LGTM! 👍" |
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.
Nice, thanks @dschwoerer ! One small change please :)
The black test is failing because it's trying to run on everything and two files are not compatible with python 3. |
Co-authored-by: Peter Hill <zed.three@gmail.com>
I am confused why it is trying to do that now. Shouldn't black already be running on next? |
black is failing on this file: https://github.com/boutproject/BOUT-dev/blob/d2750d5cd3fa7b17b75ffbc9001c9163839a2967/tools/tokamak_grids/pyGridGen/aeqdsk.py which no longer exists in |
It seems I forgot to fetch before merging next. Thanks for the catch, it now works 👍 |
Especially nice if working with shared libraries, and only a single, small file needs to be recompiled.
Recompiling
bout++.cxx
can take a long time, and this issue issue is avoided this way.Due to #1933 not useful for cmake ...
I accidentally included 73e7ad8 - but that might be useful anyway to avoid a regression?