Skip to content

Add automatic versioning to Cmake - #505

Merged
yhirose merged 1 commit into
yhirose:masterfrom
sum01:cmake_version
May 28, 2020
Merged

yhirose merged 1 commit into
yhirose:masterfrom
sum01:cmake_version

Conversation

@sum01

@sum01 sum01 commented May 28, 2020

Copy link
Copy Markdown
Contributor

It pulls the version from the user-agent string in the header, so it will not need to be manually adjusted. This version file is installed so that you can check for a specific version with find_package(httplib)

Also added HTTPLIB_INCLUDE_DIR (root path without header name), and HTTPLIB_LIBRARY (only if compiled).

Added HTTPLIB_VERSION, and HTTPLIB_FOUND (although it's recommended to check if the target exists).

Updated CMakeLists documentation for all this.

It pulls the version from the user-agent string in the header, so it
will not need to be manually adjusted. This version file is installed so
that you can check for a specific version with find_package(httplib)

Also added HTTPLIB_INCLUDE_DIR (root path without header name), and
HTTPLIB_LIBRARY (only if compiled).

Added HTTPLIB_VERSION, and HTTPLIB_FOUND (although it's recommended
to check if the target exists).

Updated CMakeLists documentation for all this.
@yhirose

yhirose commented May 28, 2020

Copy link
Copy Markdown
Owner

@sum01, looks awesome!

@yhirose
yhirose merged commit 5fcd8f7 into yhirose:master May 28, 2020
@yhirose yhirose mentioned this pull request Sep 2, 2021
ExclusiveOrange pushed a commit to ExclusiveOrange/cpp-httplib-exor that referenced this pull request May 2, 2023
It pulls the version from the user-agent string in the header, so it
will not need to be manually adjusted. This version file is installed so
that you can check for a specific version with find_package(httplib)

Also added HTTPLIB_INCLUDE_DIR (root path without header name), and
HTTPLIB_LIBRARY (only if compiled).

Added HTTPLIB_VERSION, and HTTPLIB_FOUND (although it's recommended
to check if the target exists).

Updated CMakeLists documentation for all this.
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.

2 participants