From 9be8263f2681f97c4e2bbf1fc661e120bce07728 Mon Sep 17 00:00:00 2001 From: syeopite Date: Sat, 2 Oct 2021 05:52:42 -0700 Subject: [PATCH] Use command literal for extraction --- scripts/fetch-player-dependencies.cr | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/scripts/fetch-player-dependencies.cr b/scripts/fetch-player-dependencies.cr index ba4c97aa..c141d80a 100644 --- a/scripts/fetch-player-dependencies.cr +++ b/scripts/fetch-player-dependencies.cr @@ -99,9 +99,8 @@ dependencies_to_install.each do |dep| # Unless we install an external dependency, crystal provides no way of extracting a tarball. # Thus we'll go ahead and call a system command. - args = Process.parse_arguments("-zxvf '#{download_path}/package.tgz' -C '#{download_path}'") - process = Process.new("tar", args: args) - process.wait.success? # => true + `tar -zxf '#{download_path}/package.tgz' -C '#{download_path}'"` + raise "Extraction for #{dep} failed" if !$?.success? # Would use File.rename in the following steps but for some reason it just doesn't work here. # Video.js itself is structured slightly differently