https://bugs.kde.org/show_bug.cgi?id=391396

Michael Pyne <mp...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Version Fixed In|                            |18.04
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/kde
                   |                            |src-build/742d92ac938f9e30f
                   |                            |8ccbdbcdea8a268be30aca0
             Status|UNCONFIRMED                 |RESOLVED

--- Comment #1 from Michael Pyne <mp...@kde.org> ---
Git commit 742d92ac938f9e30f8ccbdbcdea8a268be30aca0 by Michael Pyne.
Committed on 06/03/2018 at 04:33.
Pushed by mpyne into branch 'master'.

Don't shortcut build output for non-CMake build systems.

There is code to avoid using the TTY code that updates the status of the
build in the current line.  This code prevents using the fancy TTY stuff
if progress output is not supported, but doesn't actually ever output
the end result of the build (success or failure), which wasn't the
intent.

In the other situations we don't want to clutter the line output with a
redundant status.  Instead of duplicating the "progress output
supported" check later to output the status (or not), I just took out
the check for whether progress output is supported entirely, since the
progress callback can work even with build systems w/out support.
FIXED-IN:18.04

M  +4    -12   modules/ksb/BuildSystem.pm
M  +0    -5    modules/ksb/BuildSystem/KDE4.pm

https://commits.kde.org/kdesrc-build/742d92ac938f9e30f8ccbdbcdea8a268be30aca0

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to