Skip to content

Commit 120623d

Browse files
authored
Merge pull request #712 from lanquemar/master
Fix packaging script when paths contain spaces
2 parents cee3b07 + 028e325 commit 120623d

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

Diff for: script/repackage-dylibs.rb

+3-3
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ def note(msg)
4545
TARGET_FRAMEWORKS_PATH = File.join(TARGET_BUILD_DIR, FRAMEWORKS_FOLDER_PATH)
4646

4747
def extract_link_dependencies(executable)
48-
deps = `otool -L #{executable}`
48+
deps = `otool -L "#{executable}"`
4949

5050
lines = deps.split("\n").map(&:strip)
5151
lines.shift
@@ -92,7 +92,7 @@ def repackage_dependency(dep)
9292
FileUtils.cp dep[:path], TARGET_FRAMEWORKS_PATH
9393
FileUtils.chmod "u=rw", packaged_path
9494

95-
out = `install_name_tool -change #{dep.path} "@rpath/#{dep.name}" #{dep.executable}`
95+
out = `install_name_tool -change "#{dep.path}" "@rpath/#{dep.name}" "#{dep.executable}"`
9696
if $? != 0
9797
err "install_name_tool failed with error #{$?}:\n#{out}"
9898
end
@@ -107,7 +107,7 @@ def repackage_dependency(dep)
107107

108108
def fix_install_id(dep)
109109
note "Fixing #{dep.name} install_name id…"
110-
out = `install_name_tool -id @rpath/#{dep.name} #{dep.executable}`
110+
out = `install_name_tool -id "@rpath/#{dep.name}" "#{dep.executable}"`
111111
if $? != 0
112112
err "install_name_tool failed with error #{$?}:\n#{out}"
113113
end

0 commit comments

Comments
 (0)