Skip to content

Update cxxopts to v3.3.1 (GCC-15 compatibility) - #2084

Merged
jordimas merged 1 commit into
OpenNMT:masterfrom
bekard:cxxopts-3.3.1
Aug 16, 2026
Merged

Update cxxopts to v3.3.1 (GCC-15 compatibility)#2084
jordimas merged 1 commit into
OpenNMT:masterfrom
bekard:cxxopts-3.3.1

Conversation

@bekard

@bekard bekard commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

When I was building it with g++-15 and got an error:

In file included from /home/ant/prog/temp/ctranslate/CTranslate2/cli/translator.cc:4:
/home/ant/prog/temp/ctranslate/CTranslate2/cli/../third_party/cxxopts/include/cxxopts.hpp:85:5: error: ‘uint8_t’ does not name a type
   85 |     uint8_t major, minor, patch;

it's because

GCC-15 no longer includes implicitly.
https://gcc.gnu.org/gcc-15/porting_to.html

PR with fix in cxxopts: jarro2783/cxxopts#438

@bekard

bekard commented Aug 16, 2026

Copy link
Copy Markdown
Contributor Author

Failed check looks unrelated to PR's changes. Please restart the failed job.

@jordimas jordimas changed the title Update cxxopts to v3.3.1 Update cxxopts to v3.3.1 (g++-15 compatibility) Aug 16, 2026
@jordimas jordimas changed the title Update cxxopts to v3.3.1 (g++-15 compatibility) Update cxxopts to v3.3.1 (GCC-15 compatibility) Aug 16, 2026
@jordimas

Copy link
Copy Markdown
Collaborator

Thanks so much!

@jordimas
jordimas merged commit 6419e4c into OpenNMT:master Aug 16, 2026
64 of 66 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants