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

CMAKE_OSX_ARCHITECTURES must be set with FORCE #990

Merged
merged 3 commits into from Sep 12, 2016
Merged

CMAKE_OSX_ARCHITECTURES must be set with FORCE #990

merged 3 commits into from Sep 12, 2016

Commits on Sep 12, 2016

  1. CMAKE_OSX_ARCHITECTURES must be set with FORCE 

    `CMAKE_OSX_ARCHITECTURES` is actually assigned on the command line, and if not there, is assigned by CMake to an empty string.  This means that it has a value when the `set` command is encountered, and therefore unless `FORCE` is also specified, the `set` will be ignored.

See #3 for more information.
    @codemercenary
    codemercenary committed Sep 12, 2016
    Copy the full SHA
    ac62ac4 View commit details
    Browse the repository at this point in the history

  2. Only force the assignment if CMAKE_OSX_ARCHITECTURES is empty

    @codemercenary
    codemercenary committed Sep 12, 2016
    Copy the full SHA
    62177a4 View commit details
    Browse the repository at this point in the history

  3. Merge pull request #16 from leapmotion/fix-osxforce 

    CMAKE_OSX_ARCHITECTURES must be set with FORCE
    Jonathan Marsden committed Sep 12, 2016
    Copy the full SHA
    8219036 View commit details
    Browse the repository at this point in the history