The branch, master has been updated via ee8f98f pidl-waf: Only install Yapp::Driver if it is not available. via 9e591e3 pidl-waf: Check for system perl(Parse::Yapp::Driver). via 410ada4 pidl-waf: Add a function to check for a system perl module. via b7b0ca1 pidl-waf: Do not glob to install pidl modules. via 17244ef pidl-waf: Install pidl modules to the perl vendorlib directory. via 0bb6b01 pidl-waf: Remove unused variable pidl_src. via 1d95410 autobuild: Set perl vendorlib direcotry. via 48f0183 buildtools: Add perl vendorlib configure option. via 04685ff buildtools: Rename perl vendorarch configure option. via 87976b4 Revert "pidl: Install pidl files corrently." via c2d58c3 Revert "waf: Add option to specify perl vendor dir." via c46aaf1 Revert "pidl: Make perl(Parse:Yapp:Driver) installation optional." from d8e110e ctdb-vacuum: fix delete list counts in delete_marshall_traverse_first
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master - Log ----------------------------------------------------------------- commit ee8f98f26a3d6f02a0d10ceeac1194675d505246 Author: Andreas Schneider <a...@samba.org> Date: Fri Feb 28 16:00:54 2014 +0100 pidl-waf: Only install Yapp::Driver if it is not available. Signed-off-by: Andreas Schneider <a...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> Autobuild-User(master): Stefan Metzmacher <me...@samba.org> Autobuild-Date(master): Thu Mar 6 23:30:47 CET 2014 on sn-devel-104 commit 9e591e3d619d127c5ec5f6ac86326c84f59878fa Author: Andreas Schneider <a...@samba.org> Date: Fri Feb 28 15:59:45 2014 +0100 pidl-waf: Check for system perl(Parse::Yapp::Driver). Signed-off-by: Andreas Schneider <a...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> commit 410ada45b7041be6976fcfefadccbb99212a15a3 Author: Andreas Schneider <a...@samba.org> Date: Fri Feb 28 15:59:41 2014 +0100 pidl-waf: Add a function to check for a system perl module. Signed-off-by: Andreas Schneider <a...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> commit b7b0ca1ba9067362bb8a4b901f3a7f2f3bcbd6a1 Author: Andreas Schneider <a...@samba.org> Date: Fri Feb 28 15:55:46 2014 +0100 pidl-waf: Do not glob to install pidl modules. Signed-off-by: Andreas Schneider <a...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> commit 17244ef750f699b30ff7ff3a71264f4c76adad6d Author: Andreas Schneider <a...@samba.org> Date: Fri Feb 28 15:53:10 2014 +0100 pidl-waf: Install pidl modules to the perl vendorlib directory. Signed-off-by: Andreas Schneider <a...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> commit 0bb6b0102f6fa21e6c9f8010421386a660c5a5e0 Author: Andreas Schneider <a...@samba.org> Date: Fri Feb 28 15:54:09 2014 +0100 pidl-waf: Remove unused variable pidl_src. Signed-off-by: Andreas Schneider <a...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> commit 1d954103f582cd219c939606a43112f4374531ff Author: Andreas Schneider <a...@samba.org> Date: Wed Mar 5 16:27:15 2014 +0100 autobuild: Set perl vendorlib direcotry. Signed-off-by: Andreas Schneider <a...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> commit 48f0183dbeddd7bdf333a40fe0d3e1c7e7fe00f2 Author: Andreas Schneider <a...@samba.org> Date: Fri Feb 28 15:50:02 2014 +0100 buildtools: Add perl vendorlib configure option. After this patch has been pushed, we need to change autobuild to compile with this option or we will not be able to install pidl. Signed-off-by: Andreas Schneider <a...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> commit 04685ff4eed9535769d6a5feee7353f1796a4389 Author: Andreas Schneider <a...@samba.org> Date: Fri Feb 28 15:47:45 2014 +0100 buildtools: Rename perl vendorarch configure option. Signed-off-by: Andreas Schneider <a...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> commit 87976b4e45bd71b086e9201f0d0a563b9071ca4a Author: Andreas Schneider <a...@samba.org> Date: Fri Feb 28 15:46:01 2014 +0100 Revert "pidl: Install pidl files corrently." This reverts commit cee11cfa4544425099268cd32393169afb4bb107. Reviewed-by: Andreas Schneider <a...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> commit c2d58c3c2059ef3d43f876df547cbc7256235563 Author: Andreas Schneider <a...@samba.org> Date: Fri Feb 28 15:45:51 2014 +0100 Revert "waf: Add option to specify perl vendor dir." This reverts commit 9b200555fec2e33da9521db388d1839375aa8d83. Reviewed-by: Andreas Schneider <a...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> commit c46aaf1102da5e8ba53e9f89f2f756fe8dce83c5 Author: Andreas Schneider <a...@samba.org> Date: Fri Feb 28 15:45:35 2014 +0100 Revert "pidl: Make perl(Parse:Yapp:Driver) installation optional." This reverts commit f0030c619bcf22dbe9eed54b2c0dc9c61f564838. Reviewed-by: Andreas Schneider <a...@samba.org> Reviewed-by: Stefan Metzmacher <me...@samba.org> ----------------------------------------------------------------------- Summary of changes: buildtools/wafadmin/Tools/perl.py | 26 ++++++++++++------ pidl/lib/wscript_build | 4 +- pidl/wscript | 50 +++++++++++++++++++----------------- script/autobuild.py | 7 +++-- 4 files changed, 49 insertions(+), 38 deletions(-) Changeset truncated at 500 lines: diff --git a/buildtools/wafadmin/Tools/perl.py b/buildtools/wafadmin/Tools/perl.py index 90be8d8..8f13e28 100644 --- a/buildtools/wafadmin/Tools/perl.py +++ b/buildtools/wafadmin/Tools/perl.py @@ -98,19 +98,27 @@ def check_perl_ext_devel(conf): conf.env.EXTUTILS_TYPEMAP = read_out('print "$Config{privlib}/ExtUtils/typemap"') conf.env.perlext_PATTERN = '%s.' + read_out('print $Config{dlext}')[0] - if getattr(Options.options, 'perlarchdir', None): - conf.env.ARCHDIR_PERL = Options.options.perlarchdir + if getattr(Options.options, 'perl_vendorarch_dir', None): + conf.env.PERL_VENDORARCH_DIR = Options.options.perl_vendorarch_dir else: - conf.env.ARCHDIR_PERL = read_out('print $Config{sitearch}')[0] + conf.env.PERL_VENDORARCH_DIR = read_out('print $Config{vendorarch}')[0] - conf.env.VENDORDIR_PERL = Options.options.perlvendordir + if getattr(Options.options, 'perl_vendorlib_dir', None): + conf.env.PERL_VENDORLIB_DIR = Options.options.perl_vendorlib_dir + else: + conf.env.PERL_VENDORLIB_DIR = read_out('print $Config{vendorlib}')[0] def set_options(opt): opt.add_option("--with-perl-binary", type="string", dest="perlbinary", help = 'Specify alternate perl binary', default=None) - opt.add_option("--with-perl-archdir", type="string", dest="perlarchdir", help = 'Specify directory where to install arch specific files', default=None) - opt.add_option("--with-perl-vendordir", + opt.add_option("--with-perl-vendorarch", + type="string", + dest="perl_vendorarch_dir", + help = ('Specify directory where to install arch specific files'), + default=None) + + opt.add_option("--with-perl-vendorlib", type="string", - dest="perlvendordir", - help="Specify directory where to install perl vendor files", - default='${DATAROOTDIR}/perl5/vendor_perl') + dest="perl_vendorlib_dir", + help = ('Specify directory where to install vendor specific files'), + default=None) diff --git a/pidl/lib/wscript_build b/pidl/lib/wscript_build index 09d26eb..5023e07 100644 --- a/pidl/lib/wscript_build +++ b/pidl/lib/wscript_build @@ -1,7 +1,7 @@ #!/usr/bin/env python # install the pidl modules -bld.INSTALL_FILES(bld.env.VENDORDIR_PERL, +bld.INSTALL_FILES(bld.env.PERL_VENDORLIB_DIR, ''' Parse/Pidl.pm Parse/Pidl/Samba4.pm @@ -32,6 +32,6 @@ bld.INSTALL_FILES(bld.env.VENDORDIR_PERL, flat=False) if not bld.CONFIG_SET('USING_SYSTEM_PARSE_YAPP_DRIVER'): - bld.INSTALL_FILES(bld.env.VENDORDIR_PERL, + bld.INSTALL_FILES(bld.env.PERL_VENDORLIB_DIR, 'Parse/Yapp/Driver.pm', flat=False) diff --git a/pidl/wscript b/pidl/wscript index c7b72c4..4965870 100644 --- a/pidl/wscript +++ b/pidl/wscript @@ -1,8 +1,30 @@ #!/usr/bin/env python -import os, sys, Logs, Options +import os, sys, Logs from samba_utils import MODE_755 +# This function checks if a perl module is installed on the system. +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 + def set_options(opt): opt.tool_options('perl') @@ -10,6 +32,9 @@ def configure(conf): conf.check_tool('perl') conf.check_perl_ext_devel() + # Check if perl(Parse::Yapp::Driver) is available. + check_system_perl_module(conf, "Parse::Yapp::Driver", 1.05) + # we need a recent version of MakeMaker to get the right man page names if conf.CHECK_PERL_MANPAGE(): conf.env.PERLMAN1EXT = conf.CHECK_PERL_MANPAGE(section='1') @@ -20,9 +45,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 +107,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 diff --git a/script/autobuild.py b/script/autobuild.py index 4af70ee..4fd6ee4 100755 --- a/script/autobuild.py +++ b/script/autobuild.py @@ -45,7 +45,7 @@ tasks = { ("clean", "make clean", "text/plain") ], # We have 'test' before 'install' because, 'test' should work without 'install' - "samba" : [ ("configure", "./configure.developer ${PREFIX} --with-selftest-prefix=./bin/ab", "text/plain"), + "samba" : [ ("configure", "./configure.developer ${PREFIX} ${PERL_VENDOR_LIB} --with-selftest-prefix=./bin/ab", "text/plain"), ("make", "make -j", "text/plain"), ("test", "make test FAIL_IMMEDIATELY=1", "text/plain"), ("install", "make install", "text/plain"), @@ -67,7 +67,7 @@ tasks = { ("ctdb-header-ls", "ls ${PREFIX_DIR}/include/ctdb.h", "text/plain"), # build samba with cluster support against this ctdb: - ("samba-configure", "PYTHONPATH=${PYTHON_PREFIX}/site-packages:$PYTHONPATH PKG_CONFIG_PATH=$PKG_CONFIG_PATH:${PREFIX_DIR}/lib/pkgconfig ./configure.developer ${PREFIX} --with-selftest-prefix=./bin/ab --with-cluster-support --with-ctdb-dir=${PREFIX_DIR} --bundled-libraries=!tdb", "text/plain"), + ("samba-configure", "PYTHONPATH=${PYTHON_PREFIX}/site-packages:$PYTHONPATH PKG_CONFIG_PATH=$PKG_CONFIG_PATH:${PREFIX_DIR}/lib/pkgconfig ./configure.developer ${PREFIX} ${PERL_VENDOR_LIB} --with-selftest-prefix=./bin/ab --with-cluster-support --with-ctdb-dir=${PREFIX_DIR} --bundled-libraries=!tdb", "text/plain"), ("samba-make", "make", "text/plain"), ("samba-check", "./bin/smbd -b | grep CLUSTER_SUPPORT", "text/plain"), ("samba-install", "make install", "text/plain"), @@ -99,7 +99,7 @@ tasks = { ("ldb-make", "cd lib/ldb && make", "text/plain"), ("ldb-install", "cd lib/ldb && make install", "text/plain"), - ("configure", "PYTHONPATH=${PYTHON_PREFIX}/site-packages:$PYTHONPATH PKG_CONFIG_PATH=$PKG_CONFIG_PATH:${PREFIX_DIR}/lib/pkgconfig ./configure --bundled-libraries=!talloc,!tdb,!pytdb,!ntdb,!pyntdb,!ldb,!pyldb,!tevent,!pytevent --abi-check --enable-debug -C ${PREFIX}", "text/plain"), + ("configure", "PYTHONPATH=${PYTHON_PREFIX}/site-packages:$PYTHONPATH PKG_CONFIG_PATH=$PKG_CONFIG_PATH:${PREFIX_DIR}/lib/pkgconfig ./configure --bundled-libraries=!talloc,!tdb,!pytdb,!ntdb,!pyntdb,!ldb,!pyldb,!tevent,!pytevent --abi-check --enable-debug -C ${PREFIX} ${PERL_VENDOR_LIB}", "text/plain"), ("make", "make", "text/plain"), ("install", "make install", "text/plain"), ("dist", "make dist", "text/plain")], @@ -230,6 +230,7 @@ class builder(object): self.cmd = self.cmd.replace("${PYTHON_PREFIX}", get_python_lib(standard_lib=1, prefix=self.prefix)) self.cmd = self.cmd.replace("${PREFIX}", "--prefix=%s" % self.prefix) self.cmd = self.cmd.replace("${PREFIX_DIR}", "%s" % self.prefix) + self.cmd = self.cmd.replace("${PERL_VENDOR_LIB}", "--with-perl-vendorlib=%s/share/perl5" % self.prefix) # if self.output_mime_type == "text/x-subunit": # self.cmd += " | %s --immediate" % (os.path.join(os.path.dirname(__file__), "selftest/format-subunit")) print '%s: [%s] Running %s' % (self.name, self.stage, self.cmd) -- Samba Shared Repository