Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CMakeLists improvements #162

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

CMakeLists improvements #162

wants to merge 3 commits into from

Commits on Apr 18, 2017

  1. Added CMake error when ImageMagick is not found and tests are to be b… 

    …uilt.
    @wmamrak
    wmamrak authored Apr 18, 2017
    Configuration menu
    Copy the full SHA
    357e211 View commit details
    Browse the repository at this point in the history

  2. Don't add generate-embedded-font and embedded-font targets when freet… 

    …ype-gl_BUILD_MAKEFONT is false.
    @wmamrak
    wmamrak authored Apr 18, 2017
    Configuration menu
    Copy the full SHA
    600a6b6 View commit details
    Browse the repository at this point in the history

  3. Fixed missing _USE_MATH_DEFINES preprocessor definition 

    When generating solution by multiple configure, then generate combo instead of single configure, generate combo or generate alone, _USE_MATH_DEFINES preprocessor definition is not populated.
Alternative solution is to add unset(HAVE_M_PI CACHE) in the else block just after add_definitions(-D_USE_MATH_DEFINES), but the proposed way avoids unnecessary checks each time the project is configured.
    @wmamrak
    wmamrak authored Apr 18, 2017
    Configuration menu
    Copy the full SHA
    049c38e View commit details
    Browse the repository at this point in the history