Skip to content

Commit

Permalink
[CMake] do not build in each dir anymore. No need. (#8494)
Browse files Browse the repository at this point in the history
  - The CMake configuration setup has changed in #8485. Which means
    almost all targets are enabled by default and all of them are part
    of the `all` make target.

    So there is no need to go into each dir and issue installation. The
    fact that they are part of `all` means all of them will be built and
    installed.

    The excpetion is the `testsuite-depends` target which includes the
    targets:
      - libs-for-testing: installation of Modelica libs for testing.
      - reference-files: extraction of reference files.
  • Loading branch information
mahge committed Feb 2, 2022
1 parent 5f36b2b commit cfbcffb
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions .CI/common.groovy
Expand Up @@ -333,12 +333,7 @@ void buildOMC_CMake(cmake_args, cmake_exe='cmake') {
else {
sh "mkdir ./build_cmake"
sh "${cmake_exe} -S ./ -B ./build_cmake ${cmake_args}"
sh "${cmake_exe} --build ./build_cmake/OMCompiler --parallel ${numPhysicalCPU()} --target install"
sh "${cmake_exe} --build ./build_cmake/OMParser --parallel ${numPhysicalCPU()} --target install"
sh "${cmake_exe} --build ./build_cmake/OMEdit --parallel ${numPhysicalCPU()} --target install"
sh "${cmake_exe} --build ./build_cmake/OMShell --parallel ${numPhysicalCPU()} --target install"
sh "${cmake_exe} --build ./build_cmake/OMNotebook --parallel ${numPhysicalCPU()} --target install"
sh "${cmake_exe} --build ./build_cmake/testsuite --parallel ${numPhysicalCPU()} --target install"
sh "${cmake_exe} --build ./build_cmake --parallel ${numPhysicalCPU()} --target install"
sh "${cmake_exe} --build ./build_cmake --parallel ${numPhysicalCPU()} --target testsuite-depends"
}
}
Expand Down

0 comments on commit cfbcffb

Please sign in to comment.