[coreboot-gerrit] Patch merged into coreboot/master: c253717 libpaylod: fix lpgcc logic statement

gerrit at coreboot.org gerrit at coreboot.org
Tue Dec 30 19:12:14 CET 2014


the following patch was just integrated into master:
commit c2537175ecf06d1834e8ce5f5b08816269ad3393
Author: Vadim Bendebury <vbendeb at chromium.org>
Date:   Mon May 19 15:43:47 2014 -0700

    libpaylod: fix lpgcc logic statement
    
    The -z "${V}" sure must have meant to be -n "${V}", but come to think
    of it, this check is not necessary, as the following check will
    succeed if and only if V is set to 1.
    
    BUG=none
    TEST=verified that adding V=1 to the environment causes the lpgcc
         debug statements to show up in the output.
    
    Original-Change-Id: I1eb43ef49aeb4f16aef4fbee3a1037e853f9b40f
    Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/200501
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-by: Marc Jones <marc.jones at se-eng.com>
    (cherry picked from commit 7d69a292b1dc90e68e539e329f019098f8af5007)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: I63785fd9fc88b95d50ecced1f4f74a76ca68089c
    Reviewed-on: http://review.coreboot.org/7912
    Reviewed-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
    Reviewed-by: David Hendricks <dhendrix at chromium.org>
    Reviewed-by: Alexandru Gagniuc <mr.nuke.me at gmail.com>
    Tested-by: build bot (Jenkins)


See http://review.coreboot.org/7912 for details.

-gerrit



More information about the coreboot-gerrit mailing list