grame-cncm / libmusicxml

A C/C++ library to support the MusicXML format.
Mozilla Public License 2.0
152 stars 33 forks source link

CMakeLists.txt: Correct target name #47

Closed school510587 closed 2 years ago

school510587 commented 2 years ago

I use cmake and MinGW to build libmusicxml. I obtain liblibmusicxml.dll, which seens not to be an expected output name. Change this line:

set(target libmusicxml)

to

set(target musicxml)

will solve this issue. Thanks.

dfober commented 2 years ago

done. Could you check (I don't use MINGW) and close this issue if it works.

school510587 commented 2 years ago

Hi @dfober,

The variable MSYS is not set in the MinGW environment. The following must work:

  if(MINGW)
    set(target musicxml)
  else()
    set(target libmusicxml)
  endif()
dfober commented 2 years ago

ok, I changed to MINGW.

school510587 commented 2 years ago

OK, it works. Thanks.