srbala/os-autoinst-distri-fedora
1
0
Fork 0
mirror of https://pagure.io/fedora-qa/os-autoinst-distri-fedora.git synced 2024-11-25 23:33:07 +00:00
Code Issues Releases Activity

Fix copy_host_file for files containing quotes (I hope)

Browse Source 
`man printf` says \" is treated as a quote, but not \'. So
let's have the command use double quotes to wrap the format,
and escape any double quotes in the text. Hope this works.
This commit is contained in:
Adam Williamson 2017-01-15 10:52:14 -08:00
parent 3e44c7b99f
commit e848ce768e
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/fedorabase.pm
View File

@ -88,7 +88,9 @@ sub clone_host_file {
while (<$fh>) { while (<$fh>) {
$text .= $_; $text .= $_;
} }
assert_script_run "printf '$text' > $file"; # escape any ' characters in the text...
$text =~ s/"/\\"/g;
assert_script_run "printf \"$text\" > $file";
# for debugging... # for debugging...
assert_script_run "cat $file"; assert_script_run "cat $file";
} }