added top-level build script for all additivefoam solvers and utilities#75
Conversation
|
As well build out applications and utilities, we may want to reconsider where the additivefoam gitInfo header is written for printing additivefoam version information to log files |
| # Parse arguments for library compilation | ||
| . $WM_PROJECT_DIR/wmake/scripts/AllwmakeParseArguments | ||
|
|
||
| wmake -all $targetType solvers |
There was a problem hiding this comment.
Is this the same as the following?
cd solvers/additiveFoam
AllwmakeThere was a problem hiding this comment.
It is more generalized form, but the behavior will be the same in the current context. This implementation is taken from (https://github.com/OpenFOAM/OpenFOAM-10/blob/89f925dc708d068f84f51e8317e3bebcda64f5cb/applications/Allwmake#L21), to build any libraries and applications inside of the specified directory tree.
There was a problem hiding this comment.
See the source script for wmake, which will check for Allwmake scripts in each directory:
gknapp1
left a comment
gknapp1
left a comment
There was a problem hiding this comment.
Looks good, will approve once you clarify the behavior shouldn't change from the current Allwmake script.
2 participants
No description provided.