search

openmopac / mopac

Molecular Orbital PACkage
http://openmopac.net
GNU Lesser General Public License v3.0
122 stars 32 forks source link

Install mopac-bz only if BUILD_WINMOPAC is set #63

Closed awvwgk closed 2 years ago

awvwgk commented 2 years ago

Building on Windows without BUILD_WINMOPAC enabled results in a missing install target mopac-bz.

CMake Error at CMakeLists.txt:283 (install):
  install TARGETS given target "mopac-bz" which does not exist.

-- Configuring incomplete, errors occurred!

Status

codecov-commenter commented 2 years ago

Codecov Report

Merging #63 (d9782b3) into main (5edb088) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #63   +/-   ##
=======================================
  Coverage   68.24%   68.24%           
=======================================
  Files         330      330           
  Lines       71703    71703           
=======================================
  Hits        48932    48932           
  Misses      22771    22771

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 5edb088...d9782b3. Read the comment docs.

godotalgorithm commented 2 years ago

Thank you, this change is necessary to compile on Windows without the Intel Fortran compiler and its QuickWin library.