Opened 2 years ago
Closed 2 years ago
#66340 closed defect (fixed)
cmake 3.24.3 fails on 10.7, 10.8 and 10.9
Reported by: | catap (Kirill A. Korinsky) | Owned by: | catap (Kirill A. Korinsky) |
---|---|---|---|
Priority: | High | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | chillin-, RobK88 | |
Port: | cmake |
Description (last modified by catap (Kirill A. Korinsky))
It fails to build with the first error:
/opt/local/var/macports/build/_opt_bblocal_var_buildworker_ports_build_ports_devel_cmake/cmake/work/cmake-3.24.3/Source/cmWindowsRegistry.cxx:768:36: error: no matching constructor for initialization of 'std::string' (aka 'basic_string<char, char_traits<char>, allocator<char> >') return std::vector<std::string>{ std::string{ expression } }; ^ ~~~~~~~~~~~~~~
Thus, this failed build broke build of all cmake-based ports by buildbots on this platforms.
See:
- https://build.macports.org/builders/ports-10.9_x86_64-builder/builds/207496/steps/install-port/logs/stdio
- https://build.macports.org/builders/ports-10.8_x86_64-builder/builds/109789/steps/install-port/logs/stdio
- https://build.macports.org/builders/ports-10.7_x86_64-builder/builds/119854/steps/install-port/logs/stdio
Attachments (1)
Change History (11)
comment:1 Changed 2 years ago by catap (Kirill A. Korinsky)
Description: | modified (diff) |
---|---|
Priority: | Normal → High |
comment:2 Changed 2 years ago by michaelld (Michael Dickens)
comment:3 Changed 2 years ago by catap (Kirill A. Korinsky)
comment:4 follow-up: 8 Changed 2 years ago by chillin-
Failed for me, too on
Macmini5,2
OS X 10.8.5
Xcode 5.1.1
comment:5 Changed 2 years ago by chillin-
Cc: | chillin- added |
---|
comment:6 Changed 2 years ago by catap (Kirill A. Korinsky)
I'm wake up and on it. Will open a PR with fix in 30-60, just need a few cup of coffee :)
comment:7 Changed 2 years ago by catap (Kirill A. Korinsky)
Owner: | changed from michaelld to catap |
---|
comment:8 Changed 2 years ago by catap (Kirill A. Korinsky)
Replying to chillin-:
Failed for me, too on
Here the fix: https://github.com/macports/macports-ports/pull/16798
You may enjoy it right now, or wait until it is merged :)
comment:9 Changed 2 years ago by RobK88
Cc: | RobK88 added |
---|
comment:10 Changed 2 years ago by catap (Kirill A. Korinsky)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Thanks for the heads up. Guessing I need to bump the C++ standard ... thinking the prior was at 2011 & maybe the new is 2014 ... will look into this shortly.