Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Minimum required version set to
3.18.4
, released October 2020; available on Debian 11 (oldstable
)int2ssl-lib
) and executable (int2ssl
) targetsDifferent approach to integrating
CFalloutScript
into external application; instead of messing with code first, i'm messing with build process. That makes integration available faster as long external application provides everything required for linking.int2ssl-lib
is truly independent ofint2ssl
Added
int2ssl-lib-test
target (not built by default) which tries to use library target with absolute minimum setup required. Generated only for out-of-source builds.Added helper function
int2ssl_build_option()
which allows...- selecting if flag should be added to compiler or linker
- filtering out flags which are irrelevant (disabled options, different platforms, etc.)
- marking flag as required
...all in single line, making flags easier to read/find/add/remove, as they're all in one place
Added option
INT2SSL_BINARY
(default: on)Added compiler flags
-Wall -Wextra -Wpedantic
Added option
INT2SSL_STRICT
(default: off) which turns on compiler flag-Werror
Extras
project()
commandIf someone includes int2ssl in their application source, finding this repository will be one click away :)
Added option
INT2SSL_BINARY_STATIC
(default: off)Required compiler flag(s) are currently checked in
if(UNIX)
block only, simply because i'm not using MinGW right now, and have no way to test all little details. For crossplatform compilation it should probably be changed to something likeif(INT2SSL_BINARY_STATIC AND CMAKE_CXX_COMPILER_ID MATCHES "GNU")
or something like that, but someone needs to test it first~Clean worktree is happy worktree