diff options
author | Carlo Cabrera <30379873+carlocab@users.noreply.github.com> | 2022-02-19 22:03:16 +0800 |
---|---|---|
committer | Carlo Cabrera <30379873+carlocab@users.noreply.github.com> | 2022-02-19 22:03:16 +0800 |
commit | eacf1747c63848b8492f3201cc071095910c7289 (patch) | |
tree | 43d02a9fd1e21633b2484fd128a1e9c195be42f3 | |
parent | Update readme (diff) | |
download | libkeccak-eacf1747c63848b8492f3201cc071095910c7289.tar.gz libkeccak-eacf1747c63848b8492f3201cc071095910c7289.tar.bz2 libkeccak-eacf1747c63848b8492f3201cc071095910c7289.tar.xz |
Set install name in `LIBFLAGS`
My previous PR (#16) deliberately set the install name at install time
instead of at build time, since this is the correct time to determine
the library's install name.
However, if you prefer to do this during build time instead, then there
is no need to call `install_name_tool`. We can pass the appropriate
flags to the linker instead.
-rw-r--r-- | Makefile | 1 | ||||
-rw-r--r-- | linux.mk | 2 | ||||
-rw-r--r-- | macos.mk | 5 |
3 files changed, 2 insertions, 6 deletions
@@ -139,7 +139,6 @@ $(OBJ): $(HDR) libkeccak.$(LIBEXT): $(OBJ) $(CC) $(LIBFLAGS) -o $@ $(OBJ) $(LDFLAGS) - $(FIX_INSTALL_NAME) libkeccak.a: $(OBJ) -rm -f -- $@ @@ -3,5 +3,3 @@ LIBFLAGS = -shared -Wl,-soname,libkeccak.$(LIBEXT).$(LIB_MAJOR) LIBMAJOREXT = $(LIBEXT).$(LIB_MAJOR) LIBMINOREXT = $(LIBEXT).$(LIB_VERSION) - -FIX_INSTALL_NAME = : @@ -1,7 +1,6 @@ LIBEXT = dylib -LIBFLAGS = -dynamiclib -Wl,-compatibility_version,$(LIB_MAJOR) -Wl,-current_version,$(LIB_VERSION) +LIBFLAGS = -dynamiclib -Wl,-install_name,"$(PREFIX)/lib/libkeccak.$(LIBMAJOREXT)" \ + -Wl,-compatibility_version,$(LIB_MAJOR) -Wl,-current_version,$(LIB_VERSION) LIBMAJOREXT = $(LIB_MAJOR).$(LIBEXT) LIBMINOREXT = $(LIB_VERSION).$(LIBEXT) - -FIX_INSTALL_NAME = install_name_tool -id "$(PREFIX)/lib/libkeccak.$(LIBMAJOREXT)" "libkeccak.$(LIBEXT)" |