build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33509 )
Change subject: csb_patcher.sh: gets,checks,installs the coreboot and SeaBIOS patches ......................................................................
Patch Set 52:
(64 comments)
File csb_patcher.sh:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/1fb35ecd_3e769ab4 PS52, Line 26: # Asks a question '$1' and waits for Y/N user input, printing a decision - with '$2' message if Y. line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/465dc6d4_dfa4db8a PS52, Line 190: # Downloads a file '$1' from a link '$2' using the options '$3' and checks if this was successful. line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/3e6bfdd5_81f7268e PS52, Line 214: # Unzips the archive '$1', optional '$2' for -j path inside the archive, and then force removes it. line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/547620c9_48b5f0d6 PS52, Line 229: # Expands a file '$1' with zeroes using a dd to '$2' size - e.g. to a standard floppy size 1474560. line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/86a98ad4_982c38a0 PS52, Line 242: # Compares sha256sum of '$1.img' floppy image with '$2' and creates a hidden '.$1' file if matches. line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/7f6dddba_266e09b3 PS52, Line 250: printf "\n=== sha256sum should be:\n${bold}$floppy_verifier_sha256sum_correct${bend}\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/2c64091e_4fc3b6eb PS52, Line 252: printf "^^^ this is correct, ./floppies/${bold}$1.img${bend} is verified and could be added.\n\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/e7957589_3baa2920 PS52, Line 257: printf "${bold}^^^ ! MISMATCH for ./floppies/$1.img ! Check sha256sum manually: sha256sum ./floppies/$1.img${bend}\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/787ccaa2_59a9a8a2 PS52, Line 269: printf "\n${byellow}WARNING${bend}: ./floppies/${bold}$1.img${bend} - is a rolling release, its' SHA256 checksum" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/6ca4aa0a_0ffc3677 PS52, Line 270: printf "\n is changing constantly and not provided by $1 project, so not checked.\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/5732e569_567a5bb1 PS52, Line 278: # Downloads a collection of floppies, for the purpose of adding them later to a coreboot ROM image. line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/a2c3d0fe_2fce8ce7 PS52, Line 294: printf "\n${byellow}WARNING${bend}: cannot get ./floppies/${bold}kolibri.img${bend} without ${bold}7za${bend}\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/1f0b80d9_0def45af PS52, Line 301: floppy_verifier "freedos" "de2fb84dc2f132324549e89d58c6383497ec1af538c5b0ecde389fb748b9d537" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/e5e3f42a_4fa28c75 PS52, Line 304: printf "\n${byellow}WARNING${bend}: getting ./floppies/${bold}michalos.img${bend} could take a couple of minutes...\n\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/7259c131_d1101f73 PS52, Line 307: floppy_verifier "michalos" "8b62d508e30816b64694d99e58cfbeb8be8c4407423e3c3bdef8eefcb32f90ff" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/67157ccc_48091aa2 PS52, Line 315: floppy_verifier "fiwix" "9ec69b7b1a7554ff1aad784304c3926cad69e95abcac1f8a846a7f6251e25fd9" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/f1b61043_c2e43fdc PS52, Line 319: floppy_verifier "memtest" "d320f0ae56a1c65737e7bc207db22088da982f57236340abc9368860df26ea87" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/49cf3891_54d52fc7 PS52, Line 324: floppy_verifier "tatos" "2c66f884498a4fe7b469bc213aebfccd84a09fd7ace1b3b4b3e747e2392c35d1" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/8f21f140_b14f8dd5 PS52, Line 330: floppy_verifier "plop" "f170759966ec4efe60c8531158344a234d7543c8de81e54c8c1bb9d2d69eaf27" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/8065ef51_9b03e237 PS52, Line 334: floppy_verifier "floppybird" "249f77da69ba5d3295ed0c9180e4ffc646fb24f545630448fd55d5de2aec1455" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/07c5a3d9_3b48b3c8 PS52, Line 346: # Adds a './pci1002,$1.rom' to coreboot '$3' ROM using '$2' cbfstool, printing '$4'/'$5'/'$6' info. line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/aaee061c_cb8f715d PS52, Line 352: printf "\n${bgreen}NOTE${bend}: ./${bold}pci1002,$1.rom${bend} for ${bold}$5 : $6${bend} is already at your $3.\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/08cddba6_5c0290c2 PS52, Line 354: if yesno "\nAdd a ./${bold}pci1002,$1.rom${bend} for ${bold}$5 : $6${bend} to your $3 now? ${bold}~$4${bend}" ", adding..." ; then line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/23e06368_36967263 PS52, Line 358: printf "\n${bold} $2 $3 add -f ./pci1002,$1.rom -n pci1002,$1.rom -t optionrom${bend}\n\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/c7fcdbab_f9948b78 PS52, Line 373: # Adds a '$1.img' floppy image to coreboot '$3' ROM using '$2' cbfstool, printing '$4' size info. line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/7d67839f_f6e9966d PS52, Line 380: printf "\n${bgreen}NOTE${bend}: ./floppies/${bold}$1.img${bend} is already at your $3.\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/e97960b0_5a1a2a66 PS52, Line 383: printf "\n${byellow}WARNING${bend}: ./floppies/${bold}$1.img${bend} - is proprietary: all its' source code is closed !" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/48ab7ac0_00697df6 PS52, Line 384: printf "\n Add it only if you really need it and trust the author of $1 project.\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/dce5ff1d_92c5e95d PS52, Line 387: if yesno "\nAdd a ./floppies/${bold}$1.img${bend} to your $3 now? ${bold}~$4${bend}" ", adding..." ; then line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/f72d9c81_85d4dd4f PS52, Line 388: "$2" "$3" add -f "./floppies/$1.img" -n "floppyimg/$1.lzma" -t raw -c lzma line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/e261d7a5_73c19c66 PS52, Line 391: printf "\n${bold} $2 $3 add -f ./floppies/$1.img -n floppyimg/$1.lzma -t raw -c lzma${bend}\n\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/25fdff94_9f8b5a27 PS52, Line 396: printf "\n${byellow}WARNING${bend}: there was a SHA256 mismatch for ./floppies/${bold}$1.img${bend} -" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/87369051_2eec4a99 PS52, Line 397: printf "\n check sha256sum manually: sha256sum ./floppies/${bold}$1.img${bend}\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/2cc98383_5b2c100b PS52, Line 412: # Adds a set of AtomBIOS/floppies ('$1') to coreboot '$3' ROM copied to '$4', using '$2' cbfstool. line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/7e79a422_4c803915 PS52, Line 429: csb_patcher "atombios" "33886" "4" "6e7c015" "fc32365abe8506a673d7a61dc16bd54561d1993304d49aa473b14a6c3d1d6201" "$1" "AMD " line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/e1e2bb19_f299f061 PS52, Line 430: atombios_adder "990b" "$2" "$4" "62K" "Lenovo G505S with A10-5750M" "iGPU HD-8650G" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/40b2401c_2d2a5a23 PS52, Line 431: atombios_adder "6663" "$2" "$4" "33K" "Lenovo G505S with A10-5750M" "dGPU HD-8570M" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/389107a2_54ccc727 PS52, Line 432: atombios_adder "6665" "$2" "$4" "32K" "Lenovo G505S with A10-5750M" "dGPU R5-M230" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/a579e6aa_5f43644e PS52, Line 433: atombios_adder "9830" "$2" "$4" "59K" "ASUS AM1I-A with Athlon-5370" "iGPU HD-8400 / R3-Series" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/0fa701f4_ca301247 PS52, Line 434: atombios_adder "990c" "$2" "$4" "62K" "ASUS A88XM-E with A10-6700" "iGPU HD-8670D" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/3271088d_427bdd09 PS52, Line 464: # Finds the files of '$1' extension, saves the result to '$2' log and prints a message with '$3'. line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/066b9242_6433e924 PS52, Line 469: printf "\n${byellow}WARNING${bend}: found ${bold}*.$1${bend} files at these locations :\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/abab4b9e_f213e4d1 PS52, Line 540: printf "\n\n${byellow}WARNING${bend}: sorry, I don't know how to configure a board of ${bold}$1${bend}\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/8033212e_e5cdc2fb PS52, Line 676: printf "\n${byellow}WARNING${bend}: not sure if I am inside the coreboot directory," line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/064cb7e8_0ad2d3e8 PS52, Line 677: printf "\n trying to add a ${bold}$1${bend} set from here - could fail.\n\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/73fbc9b9_e9d6fc5e PS52, Line 680: printf "\n${byellow}WARNING${bend}: found a '.$1' hidden file at the current directory," line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/128510f7_1da84f2b PS52, Line 732: printf "\n${bold}To use it, it should be copied to '.config' of your ./coreboot directory.${bend}\n\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/ddfa42e9_51f8e93f PS52, Line 734: printf "\n${byellow}WARNING${bend}: copying it to ./.config will overwrite your current '.config'\n\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/15341fc2_2fb1e7ef PS52, Line 749: printf "\n${bold} cp ./configs/$csb_patcher_config_name ./.config${bend}\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/4050dc04_a6023cfa PS52, Line 751: printf "\n${byellow}WARNING${bend}: important README at the beginning of ./$1.diff\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/adf8cf83_defd88ab PS52, Line 757: printf "${bold}^^^ ! MISMATCH for a $7$1 patch ! Check sha256sum manually: sha256sum ./$1.diff${bend}\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/cebe13bc_c4d9efbc PS52, Line 773: csb_patcher "dgpu" "39873" "11" "2dcdeec" "a7c72231cf29eccfcabdf286d7cdf3cff097d302b2a981d45d9fda34357bb95c" "$1" "G505S " line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/88609115_1ec73737 PS52, Line 778: csb_patcher "atombios" "33886" "4" "6e7c015" "fc32365abe8506a673d7a61dc16bd54561d1993304d49aa473b14a6c3d1d6201" "$1" "AMD " line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/807a97fe_1449014e PS52, Line 783: csb_patcher "irq" "48427" "9" "1bf54c4" "c5f7dd4acd75c6075a692ef7f33fcffdf65ad2dbef8eb538f10174f39d976adc" "$1" "AMD good " line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/a64669a1_5004499e PS52, Line 788: csb_patcher "seabios" "32351" "15" "c4fc787" "b15163a492215c3155e3029415bf07a4514d6c02dd393ad09148d2d4adc17657" "$1" "" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/77f36754_58b59631 PS52, Line 793: csb_patcher "cfgsb" "44638" "1" "fc98bfa" "dae0ff00c5b3aa69d1a205b0e53f3a6c11437e24439bf5b936594506a3ed9d44" "$1" "for-configs " line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/e1b733c1_b4b82fe4 PS52, Line 798: csb_patcher "config.lenovo_g505s" "32352" "37" "a90256b" "ae7a0f2097ec71b215c421df695fc5adc021636240810b0b0a139ae41f975272" "$1" "" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/9a1decaa_ee8f0c44 PS52, Line 803: csb_patcher "config.asus_am1i-a" "33800" "24" "7aa38bc" "f0522955c9394e58e9e10f1d7dabcc5f8c643e7af548cefe5bc4856cf754a9e9" "$1" "" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/018f1d0f_6a711c08 PS52, Line 808: csb_patcher "config.asus_a88xm-e" "39900" "11" "00eb676" "ab89fcd3c01e657f36c5845af0921d080209c06ce3077635bf53a56408ddb18d" "$1" "" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/b22fd094_5ab96ffd PS52, Line 815: printf "${bold}===============${bend} ${bgreen}USAGE${bend} ${bold}================${bend}\n\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/d546559d_7ae5cc59 PS52, Line 818: printf "${bold}./csb_patcher.sh${bend} ${byellow}help${bend} ${bold}|${bend} ${byellow}usage${bend}\n" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/b658c9c2_4d0558cb PS52, Line 871: csb_finder "orig" "$csb_patcher_log" "applied correctly although at slightly different lines, can ignore it." line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/3aaa0dad_692f8b18 PS52, Line 872: csb_finder "rej" "$csb_patcher_log" "perhaps failed to apply, could result in a broken build! ${bred}Please report${bend}" line over 96 characters
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-131824): https://review.coreboot.org/c/coreboot/+/33509/comment/840bfc03_4c3c0cce PS52, Line 894: cbfs_mass_adder "$1" "./build/cbfstool" "./build/coreboot.rom" "./build/coreflop.rom" line over 96 characters