You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@zookeeper.apache.org by andschwa <gi...@git.apache.org> on 2018/03/09 21:03:05 UTC

[GitHub] zookeeper pull request #483: ZOOKEEPER-2997: CMake should not force static C...

GitHub user andschwa opened a pull request:

    https://github.com/apache/zookeeper/pull/483

    ZOOKEEPER-2997: CMake should not force static CRT linking

    By removing this code, CMake will use its own defaults for the CRT
    flags (e.g., `/MDd` for debug configurations). With it removed, the
    user can override this behavior by setting the `CMAKE_CXX_FLAGS`
    manually when configuring ZooKeeper.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/andschwa/zookeeper master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zookeeper/pull/483.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #483
    
----
commit 45a997ad0e281d967460f4eb205a4b18e4219ecd
Author: Andrew Schwartzmeyer <an...@...>
Date:   2018-03-09T21:00:48Z

    ZOOKEEPER-2997: CMake should not force static CRT linking
    
    By removing this code, CMake will use its own defaults for the CRT
    flags (e.g., `/MDd` for debug configurations). With it removed, the
    user can override this behavior by setting the `CMAKE_CXX_FLAGS`
    manually when configuring ZooKeeper.

----


---

[GitHub] zookeeper pull request #483: ZOOKEEPER-2997: CMake should not force static C...

Posted by andschwa <gi...@git.apache.org>.

Github user andschwa closed the pull request at:

    https://github.com/apache/zookeeper/pull/483


---