Opened 5 months ago

Closed 5 months ago

#70273 closed defect (fixed)

py-re2 dependents now require c++17, portfile needs to be updated

Reported by: ned-deily (Ned Deily) Owned by: ryandesign (Ryan Carsten Schmidt)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: py-re2

Description

The py-re2 ports depend on re2 which depends on abseil which now requires c++17 (https://github.com/macports/macports-ports/pull/23963). The current py-re2 patches for c++14:

[...]
:debug:patch Executing proc-post-org.macports.patch-patch-0
:info:patch --->  Patching setup.py: s|-std=c++11|-std=c++14|g
:debug:patch Executing reinplace: /usr/bin/sed s|-std=c++11|-std=c++14|g </opt/macports/var/macports/build/_opt_macports_var_macports_sources_rsync.macports.org_macports_release_tarballs_ports_python_py-re2/py311-re2/work/pyre2-1.0.7/setup.py >@file13
[...]
:info:build In file included from _re2.cc:37:
:info:build In file included from /opt/macports/include/re2/re2.h:224:
:info:build In file included from /opt/macports/include/absl/types/optional.h:39:
:info:build /opt/macports/include/absl/utility/utility.h:82:12: error: no member named 'in_place_t' in namespace 'std'
:info:build using std::in_place_t;
:info:build       ~~~~~^
:info:build /opt/macports/include/absl/utility/utility.h:83:12: error: no member named 'in_place' in namespace 'std'
[...]

Suggest:

    compiler.cxx_standard   2017

       reinplace "s|-std=c\+\+11|-std=c++17|g" ${worksrcpath}/setup.py


Change History (1)

comment:1 Changed 5 months ago by ryandesign (Ryan Carsten Schmidt)

Owner: set to ryandesign
Resolution: fixed
Status: newclosed

In 6f0a64768ade63ee1263e186740bdb27d93c24a8/macports-ports (master):

py-re2: re2 requires C++17 now

Closes: #70273

Note: See TracTickets for help on using tickets.