Skip to content

Commit

Permalink
Fix build for macOS (we better rely on @executable_path/libz3.dylib i…
Browse files Browse the repository at this point in the history
…n order for the plugin to work across all architectures of macOS)
  • Loading branch information
Arnaud Diederen committed Jan 24, 2023
1 parent 64496c5 commit a5ae0d7
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
3 changes: 2 additions & 1 deletion BUILDING.md
Original file line number Diff line number Diff line change
Expand Up @@ -47,5 +47,6 @@ On macOS:

* `/path/to/idasdk_pro82/bin/plugins/goomba*` -> `/path/to/ida82/ida.app/Contents/MacOS/plugins/`
* `/path/to/idasdk_pro82/bin/cfg/goomba.cfg` -> `/path/to/ida82/ida.app/Contents/MacOS/cfg/`
* `/path/to/idasdk_pro82/bin/libz3.*` -> `/path/to/ida82/ida.app/Contents/Frameworks/`
* `/path/to/idasdk_pro82/bin/libz3.*` -> `/path/to/ida82/ida.app/Contents/MacOS/`
* `/path/to/idasdk_pro82/bin/libz3.*` -> `/path/to/ida82/ida64.app/Contents/MacOS/`

4 changes: 4 additions & 0 deletions makefile
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,10 @@ ifeq ($(THIRD_PARTY),)
Z3_INCLUDE = $(Z3_BIN)../include/
endif

ifdef __MAC__
POSTACTION=install_name_tool -change libz3.dylib @executable_path/libz3.dylib $@
endif

ifdef __NT__
# link to the import library on Windows
STDLIBS += $(Z3_BIN)libz3.lib
Expand Down

0 comments on commit a5ae0d7

Please sign in to comment.