OpenChemistry / avogadrolibs

Avogadro libraries provide 3D rendering, visualization, analysis and data processing useful in computational chemistry, molecular modeling, bioinformatics, materials science, and related areas.
https://two.avogadro.cc/
BSD 3-Clause "New" or "Revised" License
461 stars 172 forks source link

fileformatScripts require Molequeue #241

Closed ghutchis closed 6 years ago

ghutchis commented 6 years ago

From https://github.com/OpenChemistry/avogadrolibs/blob/master/avogadro/qtplugins/scriptfileformats/CMakeLists.txt

# Pull in MoleQueue -- just for QtJson right now...
find_package(MoleQueue REQUIRED NO_MODULE)
include_directories(${MoleQueue_INCLUDE_DIRS})

Why should writing an IO script require Molequeue?

cryos commented 6 years ago

It was for QJson (mentioned in the comments in the CMake file). Removing, I will let the buildbot confirm, but worked locally with removal of the CMake lines requiring MoleQueue.