Define progress output option for build systems.

This actually is already present, but is checked for by looking for
cmake buildsystems directly, whereas this determination should really
belong to the build system (now that such an entity is defined).

Note that it's not supported with l10n from what I understand, even
though it uses CMake... though this is just me porting the old behavior,
it may work fine now.
wilder
Michael Pyne 14 years ago
parent 29ca10d2d1
commit 7ee1a46c4d
  1. 15
      kdesrc-build

@ -2011,6 +2011,13 @@ HOME
return 1;
}
# Returns true if the buildsystem will give percentage-completion updates on its output.
# Such percentage updates will be searched for to update the kdesrc-build status.
sub isProgressOutputSupported
{
return 0;
}
# Returns true if the module should have make install run in order to be
# used, or false if installation is not required or possible.
sub needsInstalled
@ -2285,6 +2292,12 @@ HOME
return 'KDE';
}
sub isProgressOutputSupported
{
return 1;
}
sub configureInternal
{
my $self = assert_isa(shift, 'KDEBuildSystem');
@ -4501,7 +4514,7 @@ sub run_make_command
# 1. Not using CMake (i.e. Qt)
# 2. If we're not printing to a terminal.
# 3. When we're debugging (we'd interfere with debugging output).
if ((!module_uses_cmake($module)) or not -t STDERR or debugging)
if (!$module->buildSystem()->isProgressOutputSupported() || ! -t STDERR || debugging())
{
return log_command($module, $filename, $argRef);
}

Loading…
Cancel
Save