Doesn't install the 'crystals' directory when it is supplied by the caller #611
Comments
|
Thanks - it's the endif() in the wrong spot. I'll go fix that - sorry! |
|
When I saw the issue posted, I thought 'that's weird, it's parallel code" - but no, it's not. Thanks for the issue. |
ghutchis
added a commit
to ghutchis/avogadrolibs
that referenced
this issue
Jun 7, 2021
Fixes OpenChemistry#611 - thanks to @yurivict Signed-off-by: Geoff Hutchison <geoff.hutchison@gmail.com>
|
Here are the build results |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Avogadro version: (please complete the following information from the About box):
Desktop version: (please complete the following information):
Describe the bug
Here https://github.com/OpenChemistry/avogadrolibs/blob/master/avogadro/qtplugins/insertfragment/CMakeLists.txt#L48 crystals are only installed when downloaded with git, and not when this directory is supplied by the caller. This is different from the
moleculesdirectory that is (correctly) installed when it is supplied by the caller.The text was updated successfully, but these errors were encountered: