681538a20c
build/release/roms: handle errors inside subshell
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 19:35:36 +01:00
a9bd54423c
build/release/roms: split logic into functions
...
main() on top
top-down logic
79 chars or less, per line
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 19:33:29 +01:00
2983309006
build/release/roms: use tabs for indentation
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 19:15:50 +01:00
fff5fa53ff
build/release/src: 79 chars or less per code line
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 19:03:04 +01:00
1cdf1c7cf0
build/release/src: handle errors in subshells
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 18:59:59 +01:00
16f878e882
build/release/src: split logic into functions
...
main() on top
top-down logic
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 18:52:33 +01:00
4e2ee58ac5
build/ich9utils: simplify, fix error handling
...
errors weren't being handled inside a subshell
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 18:32:52 +01:00
93ec91e862
build/memtest86plus: use tabs for indentation
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 18:28:58 +01:00
4b80f250fb
build/clean/crossgcc: better code style
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 18:28:17 +01:00
187d5fa418
build/descriptors: simplify and fix error handling
...
main() on top
some parts of the script weren't erroring properly
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 18:26:09 +01:00
a05be16998
build/grub: fix inconsistent indentation
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 18:13:44 +01:00
02919c47ce
build/grub: implement error handling
...
it uses a subshell, so errors weren't observed
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 18:10:41 +01:00
5bab3bbc33
build/grub: introduce main(), split it up
...
easier to read
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 18:06:15 +01:00
277e1df0af
build/cbutils: remove unnecessary directory check
...
it will already fail if the coreboot download did.
if the coreboot download succeeds, the directory exists.
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 18:01:02 +01:00
ed9eb4624c
build/cbutils: rename function for clarity
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 18:00:31 +01:00
b12dced470
build/cbutil: avoid frivilous use of subshells
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 17:57:15 +01:00
355a45b435
build/cbutils: top-down coding style, main on top
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 17:53:03 +01:00
9f58d4e481
build/cbutils: 79 chars or less per line
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 16:13:02 +01:00
691f266441
build/cbutils: use tabs for indendation
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-20 16:12:25 +01:00
3cbcfce9d1
gitclone: add my copyright for recent changes
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 13:19:38 +01:00
01a2ab3756
use env in shell scripts
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 13:17:28 +01:00
1e8f2cc170
gitclone: only rm the old directory at the end
...
this way, it will only be deleted after the
new git clone and patching worked successfully
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 13:03:28 +01:00
3da8d20cd6
gitclone: stricter error handling
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 13:01:12 +01:00
e804849486
gitclone: minor cleanup
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 12:56:42 +01:00
fd2ca12e9e
gitclone: split logic out of main()
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 12:55:34 +01:00
08ad9eb15f
download/coreboot: minor cleanup
2023-05-18 12:37:55 +01:00
8d9570b6f7
gitclone: cleaner coding style
...
main() on top
top-down logic
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 12:34:47 +01:00
4ac0bc8d3e
blobutil/download: minor code cleanup
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 11:41:56 +01:00
9fb489ac3e
modify: clean up duplicated code
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 11:24:08 +01:00
f7f3aef17e
modify: cleaner coding style
...
main() on top
top-down logic
reduced indentation
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 11:14:49 +01:00
34df727c98
build: cleaner coding style
...
main() on top
top-down logic
reduced indentation
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 10:49:47 +01:00
1a062bb628
build: reduce code to less than 80 chars per line
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 10:21:54 +01:00
a212a5bec8
blobutil: exit 1 if a called script fails
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 10:11:32 +01:00
e62215718c
blobutil: cleaner coding style
...
reduced indentation
main() on top
top-down logic
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 10:09:57 +01:00
c08e3258cb
.gitcheck: exit 1 if unsupported argument given
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 09:55:40 +01:00
c51225577b
.gitcheck: use subshells where appropriate
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 09:49:26 +01:00
dd8fb524df
.gitcheck: re-add redirection to /dev/null
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 09:47:44 +01:00
82c4d7b280
.gitcheck clean: clean coreboot directories too
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 09:44:11 +01:00
0f3c3ca600
.gitcheck: reduce indentation level for loop
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 09:42:06 +01:00
ecd7f1d11e
.gitcheck: move logic out of main()
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 09:41:00 +01:00
829bc02bf2
.gitcheck: *actually* check coreboot directories
2023-05-18 09:38:20 +01:00
52bc07bc84
.gitcheck: improved coding style
...
main() on top
top-down order of logic
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 09:35:26 +01:00
83235fb96b
.gitcheck: check argv when running gitcheck-clean
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 09:25:25 +01:00
6ce77652c6
.gitcheck: actually *run* gitcheck-clean
...
Run() is called, but without argument.
This patch fixes that.
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 09:23:23 +01:00
8782bff8ef
download: code cleanup
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 09:18:42 +01:00
a232f9c575
download: check for non-existent script in loop
...
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 09:16:50 +01:00
b4f1804e48
download script: bugfix: gitcheck clean didn't run
...
the "placeholder" git credentials were not being
wiped, which sometimes overwrites the user's git
credentials permanently, when working on lbmk
(permanently, until manually reset by the user)
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 09:12:27 +01:00
62c88dfb6e
download script: improved coding style
...
introduce main()
cleaned up the if/else block
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-18 09:11:59 +01:00
5b59490928
util/spkmodem_recv: Use pledge but only on OpenBSD
...
It will only be used on OpenBSD. Other operating
systems will behave in the same way.
Pledge is feature specific to OpenBSD that
restricts system operations, for security:
https://man.openbsd.org/pledge.2
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-16 23:11:35 +01:00
25241ae222
util/spkmodem_recv: Add -u flag (no line buffer)
...
printf outputs to stdout, which is line buffered
by default.
Adding a -u option to disable buffering.
Exit when a non-support flag is given, but adhere
to current behaviour when no flag is given.
Signed-off-by: Leah Rowe <leah@libreboot.org>
2023-05-16 23:11:35 +01:00
Leah Rowe