[coreboot-gerrit] Patch merged into coreboot/master: 83cbeda nyan*: Fix unexpected symbol (CR) when converting DOS-formatted BCT config.

gerrit at coreboot.org gerrit at coreboot.org
Mon Dec 15 20:11:44 CET 2014


the following patch was just integrated into master:
commit 83cbeda88dbfdd9717cf8124cc863abffb5f516b
Author: Neil Chen <neilc at nvidia.com>
Date:   Wed Apr 2 15:38:09 2014 +0800

    nyan*: Fix unexpected symbol (CR) when converting DOS-formatted BCT config.
    
    There are some unexpected symbol at the end of each line in the
    generated .inc file when the config file is in DOS format (CR+LF).
    Modify cfg2inc to support DOS format cfg file by removing carriage return symbols from the end of each line.
    
    BUG=chrome-os-partner:27614
    TEST=sudo cfg2inc.sh XXX.cfg # make a expected inc file
    
    BRANCH=nyan
    Signed-off-by: Neil Chen <neilc at nvidia.com>
    
    Original-Change-Id: I68b0f4b3805fcb5a6b633653c95afbafcb880a93
    Original-Reviewed-on: https://chromium-review.googlesource.com/192697
    Original-Tested-by: Neil Chen <neilc at nvidia.com>
    Original-Reviewed-by: Hung-Te Lin <hungte at chromium.org>
    Original-Commit-Queue: Neil Chen <neilc at nvidia.com>
    (cherry picked from commit 38e90ab0d9110d3ede39c70e27961b833813a7d4)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: I30737600fa8ac12a45ad0fbc6086a624993794e7
    Reviewed-on: http://review.coreboot.org/7741
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
    Tested-by: build bot (Jenkins)


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

-gerrit



More information about the coreboot-gerrit mailing list