[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdesdk/scripts
From:       Michael Pyne <mpyne () purinchu ! net>
Date:       2009-04-11 19:35:04
Message-ID: 1239478504.019319.15574.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 952395 by mpyne:

More run-tests tweaks to fix behavior noted by dfaure.


 M  +7 -4      kdesvn-build  


--- trunk/KDE/kdesdk/scripts/kdesvn-build #952394:952395
@@ -4748,6 +4748,11 @@
     {
         whisper "Enabling tests";
         push @commands, "-DKDE4_BUILD_TESTS:BOOL=ON";
+
+        # Also enable phonon tests.
+        if ($module =~ /^(kdesupport|phonon)$/) {
+            push @commands, "-DPHONON_BUILD_TESTS:BOOL=ON";
+        }
     }
 
     # For l10n-kde4, we must run cmake to search in the (fake) build dir.  There
@@ -5676,15 +5681,13 @@
     }
 
     # Note that we do not run safe_make, which should really be called
-    # safe_compile at this point.  We do parse make_options though.
+    # safe_compile at this point.
 
-    my @args = split(' ', get_option($module, 'make-options'));
-
     # Step 1: Ensure the tests are built, oh wait we already did that when we ran
     # CMake :)
 
     # Step 2: Run the tests.
-    my $result = log_command($module, 'test-results', [ 'make', @args, 'test' ]);
+    my $result = log_command($module, 'test-results', [ 'make', 'test' ]);
     if ($result != 0) {
         my $logdir = get_log_dir($module);
         warning "Some tests failed for y[$module], consult $logdir/test-results.log for info";
[prev in list] [next in list] [prev in thread] [next in thread]