solenv/bin/modules/installer/strip.pm |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit d70b2154a6ba68b676c24e7512121a882a7ef963
Author:     Stephan Bergmann <[email protected]>
AuthorDate: Wed Apr 24 14:35:19 2019 +0200
Commit:     Stephan Bergmann <[email protected]>
CommitDate: Wed Apr 24 16:32:56 2019 +0200

    Proper shell quoting
    
    Had found
    
    > sh: -c: line 0: syntax error near unexpected token `('
    > sh: -c: line 0: `file 
/Users/stephan/Software/lo/core/instdir/LibreOffice.app/Contents/Frameworks/LibreOfficePython.framework/Versions/3.5/lib/python3.5/site-packages/setuptools/script
 (dev).tmpl'
    
    in workdir/CustomTarget/instsetoo_native/install/install.phony.log on macOS.
    
    Change-Id: I4b01b5f0a292424a60558cdb8b431a3d8dbdf1d4
    Reviewed-on: https://gerrit.libreoffice.org/71234
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <[email protected]>

diff --git a/solenv/bin/modules/installer/strip.pm 
b/solenv/bin/modules/installer/strip.pm
index 9a8de5bbbb6d..2074973828ef 100644
--- a/solenv/bin/modules/installer/strip.pm
+++ b/solenv/bin/modules/installer/strip.pm
@@ -42,7 +42,8 @@ sub _need_to_strip
 
     # Check using the "file" command
 
-    open (FILE, "file $filename |");
+    $filename =~ s/'/'\\''/g;
+    open (FILE, "file '$filename' |");
     my $fileoutput = <FILE>;
     close (FILE);
 
_______________________________________________
Libreoffice-commits mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to