[coreboot-gerrit] Patch merged into coreboot/master: 0e53931 cbfstool: Clean up in preparation for adding new files

gerrit at coreboot.org gerrit at coreboot.org
Sat Apr 25 12:14:26 CEST 2015


the following patch was just integrated into master:
commit 0e53931fee0178c4f4ac4e2e6b355b103b5e8c42
Author: Sol Boucher <solb at chromium.org>
Date:   Thu Mar 5 15:38:03 2015 -0800

    cbfstool: Clean up in preparation for adding new files
    
    This enables more warnings on the cbfstool codebase and fixes the
    issues that surface as a result. A memory leak that used to occur
    when compressing files with lzma is also found and fixed.
    Finally, there are several fixes for the Makefile:
     - Its autodependencies used to be broken because the target for
       the .dependencies file was misnamed; this meant that Make
       didn't know how to rebuild the file, and so would silently
       skip the step of updating it before including it.
     - The ability to build to a custom output directory by defining
       the obj variable had bitrotted.
     - The default value of the obj variable was causing implicit
       rules not to apply when specifying a file as a target without
       providing a custom value for obj.
     - Add a distclean target for removing the .dependencies file.
    
    BUG=chromium:461875
    TEST=Build an image with cbfstool both before and after.
    BRANCH=None
    
    Change-Id: I951919d63443f2b053c2e67c1ac9872abc0a43ca
    Signed-off-by: Sol Boucher <solb at chromium.org>
    Original-Commit-Id: 49293443b4e565ca48d284e9a66f80c9c213975d
    Original-Change-Id: Ia7350c2c3306905984cfa711d5fc4631f0b43d5b
    Original-Signed-off-by: Sol Boucher <solb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/257340
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-by: Stefan Reinauer <reinauer at chromium.org>
    Reviewed-on: http://review.coreboot.org/9937
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list