Opened 14 years ago

Closed 13 years ago

#28656 closed defect (fixed)

zlib won't build correctly with clang (.so output instead of .dylib)

Reported by: Veence (Vincent) Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: clang Cc: landonf (Landon Fuller), ryandesign (Ryan Carsten Schmidt), bradskins@…, jeremyhu (Jeremy Huddleston Sequoia)
Port: zlib

Description

The Zlib configure script expect its compiler to contain the 'gcc' string, otherwise it branches to some default configuration that outputs a .so file (instead of .dylib). The following additional patch makes it work correctly with clang (llvm-gcc-4.2 works since it contains 'gcc').

Attachments (1)

patch-configure_clang.diff (321 bytes) - added by Veence (Vincent) 14 years ago.

Download all attachments as: .zip

Change History (6)

Changed 14 years ago by Veence (Vincent)

Attachment: patch-configure_clang.diff added

comment:1 Changed 14 years ago by Veence (Vincent)

Summary: zlib won't build with clangzlib won't build correctly with clang (.so output instead of .dylib)

comment:2 Changed 14 years ago by jmroot (Joshua Root)

Is there a reason we need to fix this in the port instead of just sending the patch upstream? We don't currently use clang to build zlib even on 10.7.

comment:3 Changed 14 years ago by Veence (Vincent)

We could send the patch upstream, of course, but I think it might be wise in the future to allow people to choose between llvm-gcc-4.2 which has, anyhow, a limited lifespan, and clang, which will probably be the only available compiler in the near future. This might at least be useful to maintainers who could spot compilation problems or discrepancies between llvm-gcc-4.2 and clang and report them to the Apple clang team.

comment:4 Changed 14 years ago by bradskins@…

Cc: bradskins@… added

Cc Me!

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

Cc: jeremyhu@… added
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.