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
Collected small changes for the next patch release #1361
Conversation
if(GIT_FOUND) | ||
execute_process(COMMAND ${GIT_EXECUTABLE} describe HEAD | ||
RESULT_VARIABLE temp_in_git_checkout | ||
if(GIT_FOUND AND EXISTS ${CMAKE_CURRENT_SOURCE_DIR}/../.git) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The other option and maybe the safer one would be to do ${GIT_EXECUTABLE} -C ${CMAKE_CURRENT_SOURCE_DIR}/../.git
everywhere.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In case you were wondering of using IS_DIRECTORY
instead of EXISTS
, this was on purpose, as git submodules only have .git
file not a directory. And lammps is used as a submodule e.g. in: https://gitlab.com/exaalt/exaalt
…ies resulting in segfaults
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK
Summary
This pull request combines multiple small changes and bugfixes
Related Issues
implements a suggestion from the discussion of #1357
fixes #1362
Author(s)
Axel Kohlmeyer (Temple U)
Licensing
By submitting this pull request, I agree, that my contribution will be included in LAMMPS and redistributed under either the GNU General Public License version 2 (GPL v2) or the GNU Lesser General Public License version 2.1 (LGPL v2.1).
Backward Compatibility
yes.
Implementation Notes
The following individual changes are implemented:
group
command options that access unavailable atom attributes.Post Submission Checklist
Please check the fields below as they are completed after the pull request has been submitted
Further Information, Files, and Links
Put any additional information here, attach relevant text or image files, and URLs to external sites (e.g. DOIs or webpages)