Skip to content

Commit

Permalink
Remove extra -headerpad linker argument
Browse files Browse the repository at this point in the history
According to the Clang 15 man page it needs a size argument, and I don't think it does anything we need that `-headerpad_max_install_names` doesn't do. And it is causing build failures for some people (see <https://www.biostars.org/p/9598655/>) because it doesn't have an argument.
  • Loading branch information
adamnovak committed Jul 15, 2024
1 parent d3f9dec commit 462dc6b
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -216,8 +216,9 @@ ifeq ($(shell uname -s),Darwin)
START_STATIC =
END_STATIC =

# We need to use special flags to let us rename libraries
LD_RENAMEABLE_FLAGS = -Wl,-headerpad -Wl,-headerpad_max_install_names
# We need to use special flags to let us rename libraries.
# See <https://github.com/Homebrew/legacy-homebrew/issues/20233>
LD_RENAMEABLE_FLAGS = -Wl,-headerpad_max_install_names
else
# We are not running on OS X
$(info OS is Linux)
Expand Down

1 comment on commit 462dc6b

@adamnovak
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vg CI tests complete for branch no-headerpad. View the full report here.

16 tests passed, 0 tests failed and 0 tests skipped in 17204 seconds

Please sign in to comment.