Opened 11 months ago

Closed 11 months ago

Last modified 10 months ago

#67784 closed defect (fixed)

R-s2 @1.1.4: error: use of undeclared identifier 'MAP_ANONYMOUS'

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: barracuda156
Priority: Normal Milestone:
Component: ports Version: 2.8.1
Keywords: lion mountainlion mavericks yosemite Cc:
Port: R-s2

Description

R-s2 @1.1.4 failed to build on the 10.7–10.10 buildbot workers two months ago. Logs have expired so I've scheduled new builds. There is a backlog of builds so it may take a day for it to get to those builds.

Change History (8)

comment:1 in reply to:  description ; Changed 11 months ago by barracuda156

Replying to ryandesign:

R-s2 @1.1.4 failed to build on the 10.7–10.10 buildbot workers two months ago. Logs have expired so I've scheduled new builds. There is a backlog of builds so it may take a day for it to get to those builds.

But it was building on 10.6 Intel? I think all those should be using the same Clang, after we blacklisted Xcode ones in R PG, so no idea, honestly, why it would fail.

For the record, s2 is a tricky thing: I failed to fix it on Rosetta, for example, though it builds now on PPC natively. There is a discussion with upstream: https://github.com/google/s2geometry/issues/315

comment:2 in reply to:  1 Changed 11 months ago by ryandesign (Ryan Carsten Schmidt)

Summary: R-s2 @1.1.4: Build failureR-s2 @1.1.4: error: use of undeclared identifier 'MAP_ANONYMOUS'

Replying to barracuda156:

Replying to ryandesign:

R-s2 @1.1.4 failed to build on the 10.7–10.10 buildbot workers two months ago. Logs have expired so I've scheduled new builds. There is a backlog of builds so it may take a day for it to get to those builds.

But it was building on 10.6 Intel?

Yes. The 10.7–10.10 builds ran and all failed due to:

absl/debugging/internal/examine_stack.cc:58:34: error: use of undeclared identifier 'MAP_ANONYMOUS'
                   MAP_PRIVATE | MAP_ANONYMOUS, -1, 0);
                                 ^
1 error generated.

so that's a simple fix of:

#ifndef MAP_ANONYMOUS
#define MAP_ANONYMOUS MAP_ANON
#endif

Or the legacysupport portgroup can be included which includes that fix.

I'm not sure why the build succeeded on 10.6.

comment:3 Changed 11 months ago by ryandesign (Ryan Carsten Schmidt)

comment:4 Changed 11 months ago by ryandesign (Ryan Carsten Schmidt)

I'm testing a fix now and will commit shortly.

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

Resolution: fixed
Status: assignedclosed

In 6102b5d9c70d7c21c9d20f79ce9ff65fc2b77772/macports-ports (master):

R-s2: Define MAP_ANONYMOUS for OS X <= 10.10

Closes: #67784

comment:6 in reply to:  4 Changed 11 months ago by barracuda156

Replying to ryandesign:

I'm testing a fix now and will commit shortly.

Thank you! Great!

comment:7 Changed 11 months ago by ryandesign (Ryan Carsten Schmidt)

It should look familiar to you! The patch is for R-s2's bundled copy of abseil, and you wrote a similar patch for abseil in https://github.com/macports/macports-ports/pull/15471.

comment:8 in reply to:  7 Changed 10 months ago by barracuda156

Replying to ryandesign:

It should look familiar to you! The patch is for R-s2's bundled copy of abseil, and you wrote a similar patch for abseil in https://github.com/macports/macports-ports/pull/15471.

Yes indeed.

Note: See TracTickets for help on using tickets.