Skip to content

Commit 6a1bc09

Browse files
committed
fix: only verify dylib install_name for macOS shared library builds
1 parent 8ed4017 commit 6a1bc09

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

scripts/build_xz_with_cmake.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -198,9 +198,10 @@ def fix_windows_lib_names(install_dir):
198198
print(f"[WARNING] Source file {source_lib} not found")
199199
return False
200200

201-
def verify_and_fix_dylib_install_name(install_dir):
201+
def verify_and_fix_dylib_install_name(install_dir, runtime_link="static"):
202202
"""Verify and fix macOS dylib install_name to use @rpath"""
203-
if platform.system() != "Darwin":
203+
# Only relevant for macOS shared library builds
204+
if platform.system() != "Darwin" or runtime_link != "shared":
204205
return True
205206

206207
dylib_path = os.path.join(install_dir, 'lib', 'liblzma.dylib')
@@ -366,7 +367,7 @@ def main():
366367
build_cmake(build_dir) and
367368
install_cmake(build_dir, install_dir) and
368369
fix_windows_lib_names(install_dir) and
369-
verify_and_fix_dylib_install_name(install_dir) and
370+
verify_and_fix_dylib_install_name(install_dir, runtime_link) and
370371
verify_build(install_dir, runtime_link)
371372
)
372373

0 commit comments

Comments
 (0)