Needed to support the psi4 quantum chemistry package
Details
Diff Detail
- Repository
- rP FreeBSD ports repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Good morning :)
It does not like 10.3:
-- Could NOT find OpenMP_C (missing: OpenMP_C_FLAGS OpenMP_C_LIB_NAMES) (found version "1.0") -- Could NOT find OpenMP_CXX (missing: OpenMP_CXX_FLAGS OpenMP_CXX_LIB_NAMES) (found version "1.0") CMake Error at cmake/custom_cxxstandard.cmake:52 (message): CLANG version must be at least 3.6! Call Stack (most recent call first): CMakeLists.txt:74 (include) -- Configuring incomplete, errors occurred! See also "/wrkdirs/usr/ports/math/ambit/work/.build/CMakeFiles/CMakeOutput.log". See also "/wrkdirs/usr/ports/math/ambit/work/.build/CMakeFiles/CMakeError.log". *** Error code 1
math/ambit/Makefile | ||
---|---|---|
30 ↗ | (On Diff #35250) | ^ I'm not sure whether hard-depending on one version of clang is a great idea as they get dropped eventually. |
math/ambit/Makefile | ||
---|---|---|
26 ↗ | (On Diff #35250) | ^ I think using <bsd.port.pre.mk> (and <bsd.port.post.mk> at the bottom) would be more proper than using <bsd.options.mk>, as you are really not using any option specific stuff here. https://www.freebsd.org/doc/en/books/porters-handbook/dads-after-port-mk.html |
math/ambit/Makefile | ||
---|---|---|
30 ↗ | (On Diff #35250) | I am not sure how this can be solved. Eventually, every specific version will be dropped. The right way would have been having USES=compiler:clang>3.6. This already came up before, without a clear solution. |
math/ambit/Makefile | ||
---|---|---|
32 ↗ | (On Diff #35350) | ^ you need += here. After including bsd.pre.mk BUILD_DEPENDS already contains cmake:devel/cmake from the USES=cmake line. Which you override here :) So this port will fail to configure on FreeBSD < 11, as cmake is not installed. root@103amd32-yuri:/usr/ports/math/ambit # make -VBUILD_DEPENDS clang38:lang/clang38 /usr/local/bin/ccache:devel/ccache |
math/ambit/Makefile | ||
---|---|---|
32 ↗ | (On Diff #35350) | Thanks for catching this! |