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.

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

More cmake options #177

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

More cmake options #177

wants to merge 1 commit into from

Conversation

xpol
Copy link

@xpol xpol commented Nov 28, 2016

Add follow 3 option to CMakeLists.txt:

option(ZLIB_BUILD_SHARED "Enable building zlib shared library" ON)
option(ZLIB_BUILD_STATIC "Enable building zlib static library" ON)
option(ZLIB_BUILD_EXAMPLES "Enable building zlib examples" ON)

@xpol xpol changed the base branch from master to develop November 28, 2016 07:56
- ZLIB_BUILD_SHARED: Enable building zlib shared library
- ZLIB_BUILD_STATIC: Enable building zlib static library
- ZLIB_BUILD_EXAMPLES: Enable building zlib examples
@xpol xpol force-pushed the more-cmake-options branch from 1230137 to 243930e Compare November 28, 2016 08:00
@warmwaffles
Copy link

@madler this would be a nice addition.

@dooglz
Copy link

dooglz commented Jan 2, 2017

@madler please consider merging😄 , Without these I'm getting unnecessary Example binaries in my parent project.

@jonathonracz
Copy link

I'd like to bump this and hopefully bring it to @madler 's attention - for those of us including zlib as a CMake sub-project, this is extremely useful for keeping generated projects + build outputs clean and is a no-risk change since the default behavior is identical to the current behavior.

Vollstrecker pushed a commit to cmake-remake/zlib that referenced this pull request Nov 30, 2024
@Vollstrecker Vollstrecker mentioned this pull request Dec 11, 2024
@Neustradamus Neustradamus mentioned this pull request Jan 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants