From 523258e675e881d6f8efc031e3885d0197c8aad0 Mon Sep 17 00:00:00 2001
From: Fridrich Štrba <fridrich.strba@bluewin.ch>
Date: Wed, 3 Jul 2013 11:18:41 +0200
Subject: Fix Windows signing and timestamping

Change-Id: Ife8774c9a6157e8bb943d1ba8ec32f560c8281c4
---
 postprocess/CustomTarget_signing.mk | 9 +++++----
 postprocess/signing/signing.pl      | 6 +++---
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/postprocess/CustomTarget_signing.mk b/postprocess/CustomTarget_signing.mk
index a9c4a45cfb91..e44c32461a68 100644
--- a/postprocess/CustomTarget_signing.mk
+++ b/postprocess/CustomTarget_signing.mk
@@ -26,11 +26,12 @@ $(call gb_CustomTarget_get_workdir,postprocess/signing)/signing.done : \
 	$(call gb_Output_announce,$(subst $(WORKDIR)/,,$@),$(true),PRL,2)
 ifeq ($(COM),MSC)
 ifneq ($(ENABLE_DBGUTIL),TRUE)
-	$(PERL) $< -e $(SRCDIR)/postprocess/signing/no_signing.txt
+	$(PERL) $(SRCDIR)/postprocess/signing/signing.pl \
+			-e $(SRCDIR)/postprocess/signing/no_signing.txt
 			-l $(subst .done,_log.txt,$@) \
-			-f $(PFXFILE) \
-			-p $(PFXPASSWORD) \
-			-t $(TIMESTAMPURL) \
+			$(if $(PFXFILE),-f $(PFXFILE)) \
+			$(if $(PFXPASSWORD),-p $(PFXPASSWORD)) \
+			$(if $($(TIMESTAMPURL),-t $(TIMESTAMPURL)) \
 			$(OUTDIR)/bin/*.dll $(OUTDIR)/bin/*.exe \
 	&& touch $@
 else
diff --git a/postprocess/signing/signing.pl b/postprocess/signing/signing.pl
index b937f0e634bb..686e56afd6d6 100644
--- a/postprocess/signing/signing.pl
+++ b/postprocess/signing/signing.pl
@@ -147,9 +147,9 @@ sub sign_files      #09.07.2007 10:36
     }
     $signtool .= " -v" if ($opt_verbose);
     $commandline_base = $signtool;
-    $commandline_base .= " -f $opt_pfxfile" if ($opt_pfxfile != "");
-    $commandline_base .= " -p $opt_pass" if ($opt_pass != "");
-    $commandline_base .= " -t $opt_timestamp_url" if ($opt_timestamp_url != "");
+    $commandline_base .= " -f $opt_pfxfile" if ($opt_pfxfile ne "");
+    $commandline_base .= " -p $opt_pass" if ($opt_pass ne "");
+    $commandline_base .= " -t $opt_timestamp_url" if ($opt_timestamp_url ne "");
 
     # Here switch between:
     # one command line for muliple files (all doesn't work, too much) / for each file one command line
-- 
cgit