build: no-longer allow GCC-10 in C++20 check #30228

fanquake commented at 9:48 AM on June 5, 2024: member

Reverts part of fa67f096bdea9db59dd20c470c9e32f3dac5be94, now that we require a minimum of GCC 11.

Code Coverage

For detailed information about the code coverage, see the test coverage report.

Reviews

See the guideline for information on the review process.

Type	Reviewers
ACK	maflcko, theuni

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

DrahtBot added the label Build system on Jun 5, 2024

maflcko commented at 9:56 AM on June 5, 2024: member

Is there any user facing difference? If so, it would be good to show it in a comment.

fanquake commented at 10:00 AM on June 5, 2024: member

Is there any user facing difference? If so, it would be good to show it in a comment.

./configure CC=gcc-10 CXX=g++-10
<snip>
configure: error: *** A compiler with support for C++20 language features is required.

maflcko commented at 10:09 AM on June 5, 2024: member

Otherwise it would continue to compile (and miscompile RenameOver)?

fanquake commented at 10:13 AM on June 5, 2024: member

Otherwise it would continue to compile (and miscompile RenameOver)?

I don't know where it would fail first, but I don't see the point of having it continue past configure in this case. There are multiple devs using clang-14 who've been confused by clang-14 failing to compile code, and unsure what the issue was. So fixing the case here for GCC seems more useful than random compile errors.

maflcko commented at 10:43 AM on June 5, 2024: member

I am asking, because this may need backport, if compilation succeeds. I assumed it would fail, which is why I left it as-is. However, if this compiles for users using g++10, then it seems dangerous. It would expose them to https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98985 and possibly other bugs.

fanquake commented at 10:53 AM on June 5, 2024: member

The first compile failure I see is in util/time.cpp:

util/time.cpp: In function ‘std::string FormatISO8601DateTime(int64_t)’:
util/time.cpp:50:24: error: ‘year_month_day’ in namespace ‘std::chrono’ does not name a type
   50 |     const std::chrono::year_month_day ymd{days};
      |                        ^~~~~~~~~~~~~~
util/time.cpp:51:24: error: ‘hh_mm_ss’ in namespace ‘std::chrono’ does not name a type
   51 |     const std::chrono::hh_mm_ss hms{secs - days};
      |                        ^~~~~~~~
util/time.cpp:52:63: error: ‘ymd’ was not declared in this scope
   52 |     return strprintf("%04i-%02u-%02uT%02i:%02i:%02iZ", signed{ymd.year()}, unsigned{ymd.month()}, unsigned{ymd.day()}, hms.hours().count(), hms.minutes().count(), hms.seconds().count());
      |                                                               ^~~
util/time.cpp:52:120: error: ‘hms’ was not declared in this scope
   52 |     return strprintf("%04i-%02u-%02uT%02i:%02i:%02iZ", signed{ymd.year()}, unsigned{ymd.month()}, unsigned{ymd.day()}, hms.hours().count(), hms.minutes().count(), hms.seconds().count());
      |                                                                                                                        ^~~
util/time.cpp: In function ‘std::string FormatISO8601Date(int64_t)’:
util/time.cpp:59:24: error: ‘year_month_day’ in namespace ‘std::chrono’ does not name a type
   59 |     const std::chrono::year_month_day ymd{days};
      |                        ^~~~~~~~~~~~~~
util/time.cpp:60:47: error: ‘ymd’ was not declared in this scope
   60 |     return strprintf("%04i-%02u-%02u", signed{ymd.year()}, unsigned{ymd.month()}, unsigned{ymd.day()});
      |                                               ^~~
make[2]: *** [Makefile:11756: util/libbitcoin_util_a-time.o] Error 1

maflcko commented at 11:29 AM on June 5, 2024: member

Thanks for checking. Can you re-try on 27.x, because https://github.com/bitcoin/bitcoin/commit/c3530254c926a5ab9b710512bbbd29e5cd5b10f7 is not in 27.x.

fanquake commented at 11:41 AM on June 5, 2024: member

Can you re-try on 27.x, because https://github.com/bitcoin/bitcoin/commit/c3530254c926a5ab9b710512bbbd29e5cd5b10f7 is not in 27.x.

27.x & fccd32efe6e2950b2c74fdec2ade54040ca32a2c compiles with gcc-10 (Ubuntu 10.5.0-4ubuntu2) 10.5.0.

maflcko added this to the milestone 28.0 on Jun 5, 2024

maflcko commented at 12:03 PM on June 5, 2024: member

Oh, 27.x remains on g++-10, so no backport needed. I was wrong. See https://github.com/bitcoin/bitcoin/blob/27.x/doc/dependencies.md

maflcko commented at 12:04 PM on June 5, 2024: member

utACK 232928b58a82e3f15307deba1ae921ae2960ccc8

Seems fine to fail early here, instead of running into a util/time compile failure later on.

theuni approved

theuni commented at 2:20 PM on June 5, 2024: member

utACK 232928b58a82e3f15307deba1ae921ae2960ccc8

fanquake merged this on Jun 6, 2024

fanquake closed this on Jun 6, 2024

fanquake deleted the branch on Jun 6, 2024

kwvg referenced this in commit 9f8d006421 on Feb 8, 2025

kwvg referenced this in commit cf44483ccf on Feb 8, 2025

kwvg referenced this in commit e5e91dfc38 on Feb 9, 2025

PastaPastaPasta referenced this in commit c84f5e128b on Feb 12, 2025

kwvg referenced this in commit ceedabb04b on Feb 12, 2025

PastaPastaPasta referenced this in commit d9704d3a01 on Feb 15, 2025

bitcoin locked this on Jun 6, 2025