Open yurivict opened 1 year ago
camke use git ls-files tests
to generate a list of files that need to be moved to the test directory. It looks like this command might be failing and not generating that test list correctly. What version of git is on the system? Are you building from a clone or using the release tar file?
Port doesn't use git - it uses the source tarball. Generally port systems can't use git clones because of the need to fingerprint the input.
I've updated the release tar to include git information so hopefully this should work for you now. This will be appropriately fixed in either the next release or a patch release.
@yurivict Did the updated release tar solve your issue?
Closing due to inactivity.
@mdavis36
Sorry for missing your response earlier.
2023.03.1 still has the same problem.
yep no git info, did you try the updated tar file for 2023.03.0?
No, I didn't try Spheral-v2023.03.0.tar.gz For the latest release Spheral-v2023.03.1.tar.gz isn't available.
Can you make git info optional, or always release the tarball with it, or make cmake to print an informative message during project's (not tests') configure saying something like "No version info, please set the SPHERAL_VERSION variable" ?
For the meantime, git info is going to be a requirement. This might change in a later release, if you can try the updated 2023.03.0 tar file then we can release a similar one for 2023.03.1 (w/ git info). 2023.03.1 only has changes that are pertinent to building at LLNL so I would stick with 2023.03.0.
@mdavis36
Spheral-v2023.03.0.tar.gz has exactly same failure during the tests configuration phase.
This is because it fails to configure when the tarball is used.
This patch works:
--- cmake/SetupSpheral.cmake.orig 2023-07-06 16:44:17 UTC
+++ cmake/SetupSpheral.cmake
@@ -176,7 +176,7 @@ if (ENABLE_TESTS)
# Find the test files we want to install
execute_process(
- COMMAND git ls-files tests
+ COMMAND find tests -type f
WORKING_DIRECTORY ${SPHERAL_ROOT_DIR}
OUTPUT_VARIABLE test_files1)
string(REPLACE "\n" " " test_files ${test_files1})
This should be fixed with #236.
Version: 2023.03.0 cmake-3.25.1 FreeBSD 13.1