Second version of the patch. As suggested by dazo I added printing of an
informational message if the sign module is not found. Also fixed a few
typos (signtool -> sign).
From a6321a47b0d4341700ad226a8b6c9abcba2052a9 Mon Sep 17 00:00:00 2001
From: =?utf-8?q?Samuli=20Sepp=C3=A4nen?= <sam...@openvpn.net>
Date: Fri, 12 Nov 2010 18:31:23 +0200
Subject: [PATCH] Removed hardcoded sign module dependency from win/build_all.py

Modified win/build_all.py so that build does not fail even if the optional sign
python module is not available.
---
 win/build_all.py |   10 ++++++++--
 1 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/win/build_all.py b/win/build_all.py
index 92d2bf4..709e060 100644
--- a/win/build_all.py
+++ b/win/build_all.py
@@ -1,7 +1,6 @@
 from config_all import main as config_all
 from build import main as build_openvpn
 from build_ddk import main as build_ddk
-from sign import main as sign
 from make_dist import main as make_dist
 
 def main(config):
@@ -9,7 +8,14 @@ def main(config):
     build_openvpn()
     build_ddk(config, 'tap', 'all')
     build_ddk(config, 'tapinstall', 'all')
-    sign(config, 'all')
+
+    # Sign the drivers only if "sign" module is available
+    try:
+       from sign import main as sign
+       sign(config, 'all')
+    except (ImportError):
+       print "Sign module not found: drivers won't get signed."
+
     make_dist(config)
 
 # if we are run directly, and not loaded as a module
-- 
1.6.3.3

Reply via email to