-
Notifications
You must be signed in to change notification settings - Fork 1
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
Document .git_archival.txt #38
Comments
For anyone else who might be stuck on this: The link on this line is dead, it did link to here. There is a related issue here (somebody else trying to figure out how to use the file) and some potential documentation here. I made the last one a permalink so this doesn't get lost. Will post here if I figure this out, but any of those links would be nice to have in the documentation. |
Sorry about that, I was aware of I've noticed that it is missing the The git submodule functionality is something I haven't thought of how to support, thanks for the idea. Btw, for CMake projects it is generally recommended to use FetchContent instead. |
I really need to put more time into finishing the online documentation. Right now the documentation doesn't build properly, but I have an idea of how to fix it. Can you ping me after 2 weeks, I'll finish that and the linking to the documentation. Also here's the new link |
Yes, FetchContent would be ideal, but in this scenario, I'm trying to build libint within another project (Psi4) in order to test modifications against the Psi4 codebase, so its easiest (development-wise) to include the modified libint (along with the modified Psi4) in the source tree. I think I have a handle on the DynamicVersion cmake file at this point, so I think I can figure out how to make the right edits to the git archival file. Thank you for getting back so quickly. |
That workflow is supported and I use it all the time. Just add |
In this case, Psi4 is using TBH, I think just adding that link you posted to the CMake Extra Utils docs would solve a lot of issues due to the unfortunate "unsearchability" of
|
Also, it seems that DynamicVersion is compatible with git submodules, but was only having trouble accessing the .git directory for my whole project in this case. Once fixed, the submodule version info is correctly gathered. For anyone trying something similar, note that GitHub forks do not grab tags from the upstream repo, you need to do it manually. |
Hmm, did I write it properly at that time? If I wrote it with
Not quite, let me walk you through:
I may have 1 and 2 switched, don't remember. There's one more workflow using
Regarding when |
I am trying to build libint as a Git submodule in another project. It is currently not building because of DynamicVersion.cmake, which I have traced here. The short document describing it here states:
I have a feeling that this is the solution to my build problems, unless I hack DynamicVersion to silence errors. But what is it? A web search for
.git_archival.txt
doesn't yield anything other thangit archive
commands, which seems to generate a .zip, not a .txt file.The text was updated successfully, but these errors were encountered: