From: Vignesh R Date: Thu, 25 Jun 2020 19:03:01 +0530 reverting below commit as we add Parse/Yapp/Driver.pm as a bundled one. https://github.com/samba-team/samba/commit/af47a77673f58cc6436798b250c90be0e325d20e pidl/wscript: configure should insist on Parse::Yapp::Driver following 83ffe67, you get a build error if you lack a system perl Parse::Yapp. Let's make it a configure failure instead. Signed-off-by: Douglas Bagnall Reviewed-by: Andrew Bartlett Autobuild-User(master): Andrew Bartlett Autobuild-Date(master): Sun Feb 2 10:09:11 UTC 2020 on sn-devel-184 --- pidl/wscript | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/pidl/wscript b/pidl/wscript index 01f8f5e..350af54 100644 --- a/pidl/wscript +++ b/pidl/wscript @@ -1,6 +1,6 @@ #!/usr/bin/env python -from waflib import Logs, Errors +from waflib import Logs # This function checks if a perl module is installed on the system. def check_system_perl_module(conf, module, version=None): @@ -21,14 +21,10 @@ def options(opt): def configure(conf): # Check if perl(Parse::Yapp::Driver) is available. - if not check_system_perl_module(conf, - "Parse::Yapp::Driver", - 1.05): - raise Errors.WafError('perl module "Parse::Yapp::Driver" not found') + check_system_perl_module(conf, "Parse::Yapp::Driver", 1.05) # yapp is used for building the parser - if not conf.find_program('yapp', var='YAPP'): - raise Errors.WafError('yapp not found') + conf.find_program('yapp', var='YAPP') def build(bld): -- 2.25.1