From b796893524878797a791b46fd955609489f3f895 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Thu, 16 Jan 2014 15:47:42 +0100 Subject: [PATCH] [SIGNED OFF] Bug 11571 - FIX regression introduced by bug 7934 Bug 7934 breaks the -f option introduced in bug 9161. While the generation of the command tmpl_process3.pl, a space is missing if the -x option is given. The tmpl_process3.pl is called like: /home/koha/src/misc/translator/tmpl_process3.pl -q update -i /home/koha/src/koha-tmpl/intranet-tmpl/prog/en/ -s /home/koha/src/misc/translator/po/fr-FR-i-staff-t-prog-v-3006000.po -r -x 'help'-f pay.tt Revised test plan: 1) cd ./misc/translator 2) put a warn at LangInstaller.pm line 375. 3) time ./translate update fr-FR -f pay.tt -- note the execution time and the output. The options in the command contain "-x 'help'-f pay.tt" The -f param is not passed to the script. The execution time is strangely long. 5) git reset --hard origin/master 6) apply this patch 7) put a warn at LangInstaller.pm line 375. 8) time ./translate update fr-FR -f pay.tt -- verify the output and the execution time is now corrected. Signed-off-by: Mark Tompsett --- misc/translator/LangInstaller.pm | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/misc/translator/LangInstaller.pm b/misc/translator/LangInstaller.pm index 50ba308..158ca22 100644 --- a/misc/translator/LangInstaller.pm +++ b/misc/translator/LangInstaller.pm @@ -349,7 +349,7 @@ sub install_tmpl { "-s $self->{path_po}/$self->{lang}$trans->{suffix} -r $excludes" . ( @$files - ? '-f ' . join ' -f ', @$files + ? ' -f ' . join ' -f ', @$files : '' ) } @@ -378,7 +378,7 @@ sub update_tmpl { "-s $self->{path_po}/$self->{lang}$trans->{suffix} -r $excludes" . ( @$files - ? '-f ' . join ' -f ', @$files + ? ' -f ' . join ' -f ', @$files : '' ) } @@ -417,7 +417,7 @@ sub create_tmpl { "-s $self->{path_po}/$self->{lang}$trans->{suffix} -r $excludes" . ( @$files - ? '-f ' . join ' -f ', @$files + ? ' -f ' . join ' -f ', @$files : '' ) } -- 1.7.9.5