From c46aaf1102da5e8ba53e9f89f2f756fe8dce83c5 Mon Sep 17 00:00:00 2001 From: Andreas Schneider Date: Fri, 28 Feb 2014 15:45:35 +0100 Subject: [PATCH] Revert "pidl: Make perl(Parse:Yapp:Driver) installation optional." This reverts commit f0030c619bcf22dbe9eed54b2c0dc9c61f564838. Reviewed-by: Andreas Schneider Reviewed-by: Stefan Metzmacher --- pidl/lib/wscript_build | 6 +----- pidl/wscript | 25 +------------------------ 2 files changed, 2 insertions(+), 29 deletions(-) diff --git a/pidl/lib/wscript_build b/pidl/lib/wscript_build index 09d26ebd6d8f..29632d4d99b8 100644 --- a/pidl/lib/wscript_build +++ b/pidl/lib/wscript_build @@ -28,10 +28,6 @@ bld.INSTALL_FILES(bld.env.VENDORDIR_PERL, Parse/Pidl/Samba3/ServerNDR.pm Parse/Pidl/Compat.pm Parse/Pidl/NDR.pm + Parse/Yapp/Driver.pm ''', flat=False) - -if not bld.CONFIG_SET('USING_SYSTEM_PARSE_YAPP_DRIVER'): - bld.INSTALL_FILES(bld.env.VENDORDIR_PERL, - 'Parse/Yapp/Driver.pm', - flat=False) diff --git a/pidl/wscript b/pidl/wscript index c7b72c400ff9..77abb019de9b 100644 --- a/pidl/wscript +++ b/pidl/wscript @@ -1,6 +1,6 @@ #!/usr/bin/env python -import os, sys, Logs, Options +import os, sys, Logs from samba_utils import MODE_755 def set_options(opt): @@ -20,9 +20,6 @@ def configure(conf): conf.find_program('yapp', var='YAPP') conf.find_program('pod2man', var='POD2MAN') - # Check for perl(Parse::Yapp::Driver) - check_system_perl_module(conf, "Parse::Yapp::Driver", 1.05) - def build(bld): bld.INSTALL_FILES('${BINDIR}', 'pidl', chmod=MODE_755) @@ -85,23 +82,3 @@ try this to avoid this message: $ touch ../pidl/lib/Parse/Pidl/IDL.pm ../pidl/lib/Parse/Pidl/Expr.pm ''') -def check_system_perl_module(conf, module, version=None): - bundle_name = module.replace('::', '_') - module_check = module - found = False - - # Create module string with version - if version: - module_check = module + ' ' + str(version) - - # Check if we have to bundle it. - if conf.LIB_MUST_BE_BUNDLED(bundle_name.lower()): - return False - - # Check for system perl module - if not conf.check_perl_module(module_check): - return False - - conf.define('USING_SYSTEM_%s' % bundle_name.upper(), 1) - - return True -- 2.34.1