After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 701177 - scripts: improve git-update.sh status message
scripts: improve git-update.sh status message
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other Linux
: Normal enhancement
: 1.1.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-05-29 08:27 UTC by Krzysztof Konopko
Modified: 2013-05-30 22:42 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (1.38 KB, patch)
2013-05-29 08:27 UTC, Krzysztof Konopko
committed Details | Review

Description Krzysztof Konopko 2013-05-29 08:27:55 UTC
Created attachment 245525 [details] [review]
Proposed patch

By default when the script is about to exit (normally or due to an error), it checks whether $ERROR_LOG file exists.  If the log file exists, the script prints a "Failures: " message prefix and dumps the log file to the output.

Apparently the log file is always created and if the update/build is successful, the script finishes with a bit misleading "Failures: " message.

An improvement provided with this change lets the log file to be created as needed, i.e. if there's an error message to be printed.  If the file doesn't exists, the script prints a "Update done" message which clearly indicates success.
Comment 1 Tim-Philipp Müller 2013-05-30 22:42:13 UTC
Thanks for the patch! Pushed into master:


commit 527423fcd9a5b74b8bc3dfbae75871b9c0f87f7c
Author: Krzysztof Konopko <krzysztof.konopko@gmail.com>
Date:   Tue May 28 23:34:54 2013 +0100

    scripts: improve git-update.sh status message
    
    By default when the script is about to exit (normally or due to an error),
    it checks whether $ERROR_LOG file exists.  If the log file exists, the
    script prints a "Failures: " message prefix and dumps the log file to the
    output.
    
    Apparently the log file is always created and if the update/build is
    successful, the script finishes with a bit misleading "Failures: " message.
    
    An improvement provided with this change lets the log file to be created as
    needed, i.e. if there's an error message to be printed.  If the file
    doesn't exists, the script prints a "Update done" message which clearly
    indicates success.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=701177