Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44035 )
Change subject: sconfig: Switch to getopt ......................................................................
Patch Set 2:
(1 comment)
Patch Set 2: Code-Review+1
Do we need to include getopt.c for macos / BSD / Windows?
Isn't getopt just part of libc?
https://review.coreboot.org/c/coreboot/+/44035/2/util/sconfig/main.c File util/sconfig/main.c:
https://review.coreboot.org/c/coreboot/+/44035/2/util/sconfig/main.c@1314 PS2, Line 1314: printf(" -c | --output_c : Output static.c file (required)\n"); : printf(" -r | --output_h : Header static.h file (required)\n"); : printf(" -m | --mainboard_devtree : Mainboard devicetree file (required)\n"); : printf(" -o | --override_devtree : Override devicetree file (optional)\n");
These are *paths* to files, right?
Correct, point taken 😄