-
Notifications
You must be signed in to change notification settings - Fork 16
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
Tests fail to configure: string sub-command REPLACE requires at least four arguments. #207
Comments
camke use |
Port doesn't use git - it uses the source tarball. |
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. |
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 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. |
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:
|
This should be fixed with #236. |
Version: 2023.03.0
cmake-3.25.1
FreeBSD 13.1
The text was updated successfully, but these errors were encountered: