Build cleanup #85
Merged
Build cleanup #85
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
6ce107f to
f054eeb
Compare
This entry should have been removed in the referenced commit. Fixes: 6639d08
This change imports ac_cxx_compile_stdcxx.m4 from gnu,org and makes use of the `AX_CXX_COMPILE_STDCXX` macro in configure.ac to ensure that the compiler specified supports C++-14 (at bare minimum). This is being done to quell some issues reported by scan-build about the code using C++ range-based for-loops (a feature added in C++-11).
These manpages have been deprecated for at least a release. Remove them and all of the logic associated with them. Signed-off-by: Enji Cooper <ngie@FreeBSD.org>
WCOREDUMP is considered an extension to the POSIX spec on multiple platforms, and thus is not automatically exposed on all platforms. Add the relevant preprocessor defines to config.h via autoconf, then leverage them in atf-c(3). This helps ensure that the platforms which support WCOREDUMP properly expose the macro. Signed-off-by: Enji Cooper <ngie@FreeBSD.org>
This feature is being detected so several functions can be appropriately marked as taking non-NULL/-nullptr parameters, and the compiler and static analyzers can (in turn) make intelligent decisions when optimizing and analyzing code, respectively. Signed-off-by: Enji Cooper <ngie@FreeBSD.org>
This was accidentally deleted post-0.21 release. It's still needed by tests and build infrastructure. Signed-off-by: Enji Cooper <ngie@FreeBSD.org>
The libcalls used have been in the POSIX standard since 2008.1. Require them to be present instead of conditionally hoping they're present. This fixes an issue where the autoconf code was messing up with a combination of clang tools, which resulted in the autoconf code failing to properly determine whether or not the functions were available. Signed-off-by: Enji Cooper <ngie@FreeBSD.org>
This particular check unfortunately doesn't work when ATF_BUILD_CXX contains multiple CXXFLAGS, or contains a path with spaces in it. Remove this check to unbreak the dependent tests post-793d4640031dc06ce8a239ffa9ab61322104c4ca. Signed-off-by: Enji Cooper <ngie@FreeBSD.org>
The C++ toolchain is sanity checked when C++14 conformance is checked; there's no reason why we need to check whether or not the C++ toolchain works again. Signed-off-by: Enji Cooper <ngie@FreeBSD.org>
f054eeb to
de512c3
Compare
emaste
reviewed
Dec 29, 2024
emaste
reviewed
Dec 29, 2024
emaste
reviewed
Dec 29, 2024
emaste
reviewed
Dec 29, 2024
| @@ -62,6 +62,7 @@ AC_USE_SYSTEM_EXTENSIONS | |||
|
|
|||
| AC_PROG_CXX | |||
| AC_LANG_COMPILER(C++) | |||
| AC_PROG_CPP | |||
There was a problem hiding this comment.
OK. Is this where it belongs though, it seems the CXX and C++ bits should be together without CPP in the middle
There was a problem hiding this comment.
I'll have to run through it again, but the order for invoking the macros is a bit fragile (autoconf/automake whine a lot about not calling things in the right order).
emaste
reviewed
Dec 29, 2024
emaste
reviewed
Dec 29, 2024
|
I've marked the files I've reviewed so far |
ngie-eign
added a commit
that referenced
this pull request
Mar 29, 2025
* Garbage collect stray reference to auto_array This entry should have been removed in the referenced commit. Fixes: 6639d08 * Require C++-14 at bare minimum This change imports ac_cxx_compile_stdcxx.m4 from gnu,org and makes use of the `AX_CXX_COMPILE_STDCXX` macro in configure.ac to ensure that the compiler specified supports C++-14 (at bare minimum). This is being done to quell some issues reported by scan-build about the code using C++ range-based for-loops (a feature added in C++-11). * Stop shipping atf-*-api(3) manpages These manpages have been deprecated for at least a release. Remove them and all of the logic associated with them. Signed-off-by: Enji Cooper <ngie@FreeBSD.org> * Expose WCOREDUMP(..) in a deterministic manner WCOREDUMP is considered an extension to the POSIX spec on multiple platforms, and thus is not automatically exposed on all platforms. Add the relevant preprocessor defines to config.h via autoconf, then leverage them in atf-c(3). This helps ensure that the platforms which support WCOREDUMP properly expose the macro. Signed-off-by: Enji Cooper <ngie@FreeBSD.org> * Add compiler feature detection for `__attribute__((nonnull))` This feature is being detected so several functions can be appropriately marked as taking non-NULL/-nullptr parameters, and the compiler and static analyzers can (in turn) make intelligent decisions when optimizing and analyzing code, respectively. Signed-off-by: Enji Cooper <ngie@FreeBSD.org> * Restore AC_PROG_CPP This was accidentally deleted post-0.21 release. It's still needed by tests and build infrastructure. Signed-off-by: Enji Cooper <ngie@FreeBSD.org> * atf-c/detail/env.c: remove unnecessary complexity The libcalls used have been in the POSIX standard since 2008.1. Require them to be present instead of conditionally hoping they're present. This fixes an issue where the autoconf code was messing up with a combination of clang tools, which resulted in the autoconf code failing to properly determine whether or not the functions were available. Signed-off-by: Enji Cooper <ngie@FreeBSD.org> * Remove ATF_BUILD_CXX require.progs check This particular check unfortunately doesn't work when ATF_BUILD_CXX contains multiple CXXFLAGS, or contains a path with spaces in it. Remove this check to unbreak the dependent tests post-793d4640031dc06ce8a239ffa9ab61322104c4ca. Signed-off-by: Enji Cooper <ngie@FreeBSD.org> * Remove redundant C++ toolchain check The C++ toolchain is sanity checked when C++14 conformance is checked; there's no reason why we need to check whether or not the C++ toolchain works again. Signed-off-by: Enji Cooper <ngie@FreeBSD.org> --------- Signed-off-by: Enji Cooper <ngie@FreeBSD.org>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR addresses some build bugs and adds some build enhancements:
auto_array(bugfix).atf-*-apimanpages (enhancement).-std=C++14with ATF (enhancement).WCOREDUMP(..)macro on platforms which support it via autotools (bugfix).__attribute__((nonnull))support, similar to__attribute__((unused)), et al (enhancement).AC_PROG_CPP(bugfix).