Build docs: require GCC 13 and show the ccache CC/CXX configure flags - #239
Open
Darshan-upadhyay1110 wants to merge 1 commit into
Open
Build docs: require GCC 13 and show the ccache CC/CXX configure flags#239Darshan-upadhyay1110 wants to merge 1 commit into
Darshan-upadhyay1110 wants to merge 1 commit into
Conversation
The online code uses std::format (common/RecentFiles.cpp), and libstdc++ only provides the format header from GCC 13, while configure has no compiler version check, so a build with GCC 12 or older passes configure and then fails at compile time. State the GCC 13 or newer requirement on the build pages so nobody hits that wall. Per distribution: openSUSE gets a gcc13/gcc13-c++ install step and a configure line with CC="ccache gcc-13" CXX="ccache g++-13"; Ubuntu notes 24.04 or later ships GCC 13 by default and shows the same explicit flags as the fallback for older releases; Fedora notes 38 or later; Debian notes 13 or later, since Debian 12 has no gcc-13 package; Arch already satisfies the requirement. The shared build-online shortcode takes optional cc and cxx parameters so every distro section renders a configure line with the ccache prefix, and ccache was added to each package install list. Fixes #235. Fixes #125.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
The online code uses std::format (common/RecentFiles.cpp), and libstdc++ only provides the format header from GCC 13, while configure has no compiler version check, so a build with GCC 12 or older passes configure and then fails at compile time. State the GCC 13 or newer requirement on the build pages so nobody hits that wall.
Per distribution: openSUSE gets a gcc13/gcc13-c++ install step and a configure line with CC="ccache gcc-13" CXX="ccache g++-13"; Ubuntu notes 24.04 or later ships GCC 13 by default and shows the same explicit flags as the fallback for older releases; Fedora notes 38 or later; Debian notes 13 or later, since Debian 12 has no gcc-13 package; Arch already satisfies the requirement. The shared build-online shortcode takes optional cc and cxx parameters so every distro section renders a configure line with the ccache prefix, and ccache was added to each package install list.
Fixes #235. Fixes #125.