Opened 5 years ago

Closed 5 years ago

#59099 closed defect (fixed)

fcron @3.2.1: Undefined symbols: _strndup

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: kurthindenburg (Kurt Hindenburg)
Priority: Normal Milestone:
Component: ports Version:
Keywords: tiger leopard snowleopard Cc:
Port: fcron

Description

fcron fails to build on Snow Leopard because it uses strndup. Using the legacy support port group might help.

Change History (6)

comment:1 Changed 5 years ago by Kurt Hindenburg <kurt.hindenburg@…>

Resolution: fixed
Status: assignedclosed

In 02f7cbb80978ff4b9496f0846a58a3003c019352/macports-ports (master):

fcron: add legacy support to attempt to fix builds

fixes #59099

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

Resolution: fixed
Status: closedreopened

This does not appear to be enough. It looks like it's not using our LDFLAGS.

comment:3 Changed 5 years ago by Kurt Hindenburg <kurt.hindenburg@…>

In dde212cfacc5acddf8fde5409ae447c21c9fb5a8/macports-ports (master):

fcron: adjust Makefile to actually use LDFLAGS instead of just LIBS

I'm not entirely sure how this every worked; only legacy build were
failing. Force re-building to see if this breaks anything.
Tested locally on 10.13.6 and 10.15.1

See #59099

comment:4 Changed 5 years ago by kurthindenburg (Kurt Hindenburg)

I'm waiting on the > 10.9 builds to start - everything else seems to work

https://ports.macports.org/port/fcron/builds

comment:5 in reply to:  3 Changed 5 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to Kurt Hindenburg <kurt.hindenburg@…>:

I'm not entirely sure how this every worked

It was getting the library path from LIBRARY_PATH. (Only very old versions of clang don't support that.) As for the rest of our LDFLAGS, it wasn't using them, so it wasn't built with headerpad_max_install_names and it wasn't built for the right architectures. With your change, now it should be.

comment:6 Changed 5 years ago by kurthindenburg (Kurt Hindenburg)

Resolution: fixed
Status: reopenedclosed

builds look good

Note: See TracTickets for help on using tickets.