From ff30da3ebb107421372e2a1a6d37f7c189667f2b Mon Sep 17 00:00:00 2001 From: Martin Whitaker Date: Wed, 10 Jan 2018 09:03:20 +0000 Subject: Simplify run_() helper function and minimise direct calls to system(). --- lib/MGA/DrakISO/BuildISO.pm | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) (limited to 'lib/MGA/DrakISO/BuildISO.pm') diff --git a/lib/MGA/DrakISO/BuildISO.pm b/lib/MGA/DrakISO/BuildISO.pm index b3ca5d2..8e90c55 100644 --- a/lib/MGA/DrakISO/BuildISO.pm +++ b/lib/MGA/DrakISO/BuildISO.pm @@ -114,7 +114,7 @@ sub build_iso_image { chomp($xorriso_version); my $date_file = $build_dir . '/DATE.txt'; - run_('date', '>', $date_file); + run_("date > $date_file"); run_('xorrisofs', if_($::verbose < 2, '-quiet'), @@ -149,11 +149,10 @@ sub build_iso_image { '/DATE.txt=' . $date_file, ) or die "ERROR: unable to run xorrisofs\n"; - my $filename = basename($dest); - run_('mgaiso-addmd5', '>', '/dev/null', '2>', '/dev/null', $dest); - run_({ chdir => $build_dir }, 'md5sum', '>', $dest . '.md5', $filename); - run_({ chdir => $build_dir }, 'sha1sum', '>', $dest . '.sha1', $filename); - run_({ chdir => $build_dir }, 'sha512sum', '>', $dest . '.sha512', $filename); + run_("mgaiso-addmd5 $dest > /dev/null 2> /dev/null"); + run_("md5sum $dest > $dest.md5"); + run_("sha1sum $dest > $dest.sha1"); + run_("sha512sum $dest > $dest.sha512"); if (my $suffix = $build->get_set_suffix) { if (my ($prefix, $ext) = $dest =~ /(.*)(\.[^.]+)$/) { my $link = $prefix . $suffix . $ext; -- cgit v1.2.1