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

V4.5 #2

Merged
merged 13 commits into from
Apr 14, 2023
Merged

V4.5 #2

merged 13 commits into from
Apr 14, 2023

Conversation

joy4eg
Copy link
Collaborator

@joy4eg joy4eg commented Apr 14, 2023

No description provided.

mfontanini and others added 13 commits February 17, 2022 06:55
)

Co-authored-by: Bill Willcox <billwcorp@gmail.com>
Using a submodule to download and build GTest is a great approach for
most circumstances, but some may prefer to use the system-provided GTest
if it is available.

This change adds a fallback to using the system's GTest if the submodule
is absent.
On some platforms (like RedHat ones), CMAKE_INSTALL_LIBDIR is set to
`lib64` instead of `lib`. The CMake files should also be installed to
`lib64`, but because CONF_CMAKE_INSTALL_DIR is set unconditionally to
use `lib`, the proper path can't be configured.

This change makes CONF_CMAKE_INSTALL_DIR use the configured
CMAKE_INSTALL_LIBDIR value, which defaults to `lib`.
…tanini#484) (mfontanini#486)

According to the C/C++ Standard, for shift operations, the behavior is undefined if the right operand is equal to the width of the promoted left operand.
On a 64-bit Windows machine, this causes IP addresses 0.0.0.0 and 255.255.255.255 to have the same internal representation, leading to various issues when using a /0 prefix.
Due to changes in GCC13 need fix include.
`<algorithm>` is needed for std::fill (error in clang-16)
@joy4eg joy4eg self-assigned this Apr 14, 2023
@joy4eg joy4eg merged commit 5c92d5d into master Apr 14, 2023
@joy4eg joy4eg deleted the v4.5 branch April 14, 2023 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

8 participants