[SCM] Samba Shared Repository - branch master updated

Matthieu Patou mat at samba.org
Mon Oct 18 13:51:01 MDT 2010


The branch, master has been updated
       via  902e728 selftest: use full path for selftest.pl instead of relative paths
       via  b1a13e5 selftest: correct the import of Time::HighRes
      from  8556602 ldb:"ldb_schema_attribute_by_name_internal" - switch back to 32bit counters

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 902e7288a738c0580c466ae4c5c503d604c66e6e
Author: Matthieu Patou <mat at matws.net>
Date:   Mon Oct 18 22:23:00 2010 +0400

    selftest: use full path for selftest.pl instead of relative paths
    
    FindBin on solaris8 with perl 5.04 has problem to get the path of the
    called script if the script is called with a relative path.
    
    Autobuild-User: Matthieu Patou <mat at samba.org>
    Autobuild-Date: Mon Oct 18 19:50:03 UTC 2010 on sn-devel-104

commit b1a13e5e1453b6f776e0a35f293f4060c77fe59a
Author: Matthieu Patou <mat at matws.net>
Date:   Mon Oct 18 20:21:12 2010 +0400

    selftest: correct the import of Time::HighRes
    
    The "use" is always evaluated first even if enclosed into a if. So
    on plateform without Time::Hires (ie. sun8 in the build farm). This
    leads to an error.
    
    Instead we use the synthax require Module + Module->import("function")

-----------------------------------------------------------------------

Summary of changes:
 selftest/selftest.pl     |    5 +++--
 source4/selftest/wscript |    3 ++-
 2 files changed, 5 insertions(+), 3 deletions(-)


Changeset truncated at 500 lines:

diff --git a/selftest/selftest.pl b/selftest/selftest.pl
index 08ee52c..7431bed 100755
--- a/selftest/selftest.pl
+++ b/selftest/selftest.pl
@@ -135,9 +135,10 @@ use SocketWrapper;
 
 eval {
 require Time::HiRes;
+Time::HiRes->import("time");
 };
-unless ($@) {
-	use Time::HiRes qw(time);
+if ($@) {
+	print "You don't have Time::Hires installed !\n";
 }
 
 my $opt_help = 0;
diff --git a/source4/selftest/wscript b/source4/selftest/wscript
index 249db5e..2f4e436 100644
--- a/source4/selftest/wscript
+++ b/source4/selftest/wscript
@@ -148,7 +148,8 @@ def cmd_testonly(opt):
     if os.path.exists(st_done):
         os.unlink(st_done)
 
-    cmd = '(${PERL} ../selftest/selftest.pl --prefix=${SELFTEST_PREFIX} --builddir=. --srcdir=. --exclude=./selftest/skip --testlist="${PYTHON} ./selftest/tests.py|" ${OPTIONS} --socket-wrapper ${TESTS} && touch ${SELFTEST_PREFIX}/st_done) | ${FILTER_OPTIONS} | tee ${SELFTEST_PREFIX}/subunit'
+    # We use the full path rather than relative path because it cause problems on some plateforms (ie. solaris 8).
+    cmd = '(${PERL} %s/../selftest/selftest.pl --prefix=${SELFTEST_PREFIX} --builddir=. --srcdir=. --exclude=./selftest/skip --testlist="${PYTHON} ./selftest/tests.py|" ${OPTIONS} --socket-wrapper ${TESTS} && touch ${SELFTEST_PREFIX}/st_done) | ${FILTER_OPTIONS} | tee ${SELFTEST_PREFIX}/subunit' % (os.getcwd())
     if os.environ.get('RUN_FROM_BUILD_FARM') is None and not Options.options.FILTERED_SUBUNIT:
         cmd += ' | ${FORMAT_TEST_OUTPUT}'
     cmd = EXPAND_VARIABLES(opt, cmd)


-- 
Samba Shared Repository


More information about the samba-cvs mailing list