Opened 2 years ago

Closed 2 years ago

#64629 closed update (fixed)

protozero: Update to 1.7.1

Reported by: SiggyF Owned by: frankdean
Priority: Normal Milestone:
Component: ports Version: 2.7.1
Keywords: Cc:
Port: protozero

Description

This ports needs an update to 1.7.1 to get working with m1 mac, as described in https://github.com/mapbox/protozero/issues/107. There is also another issue, which I could fix by editing the cmake settings. The problem is that by default -werror is used combined with -std=c++11.

/opt/local/include/google/protobuf/parse_context.h:365:3: error: use of the 'nodiscard' attribute is a C++17 extension [-Werror,-Wc++17-extensions]
  PROTOBUF_NODISCARD const char* AppendString(const char* ptr,

If I set WERROR:BOOL=OFF the build completes with clang++ . Another option would be to change the -std=c++11 to -std=c++17, but I could not find where to put that in the cmake settings.

Change History (3)

comment:1 Changed 2 years ago by ryandesign (Ryan Carsten Schmidt)

Owner: set to frankdean
Status: newassigned
Summary: Update needed for mac m1 maxprotozero: Update to 1.7.1

We should disable the use of -Werror. We should not unnecessarily force the use of newer language standards.

comment:2 Changed 2 years ago by frankdean

I've submitted PR #13937 to update the version and disable treating warnings as errors - https://github.com/macports/macports-ports/pull/13937

comment:3 Changed 2 years ago by frankdean <frankdean@…>

Resolution: fixed
Status: assignedclosed

In e5b3dc46573625816e04fff1a8639ab47bf0875a/macports-ports (master):

protozero: Update to 1.7.1

Closes: #64629

Note: See TracTickets for help on using tickets.