Hi all, Am Mittwoch, dem 10.08.2022 um 09:59 +0200 schrieb Jan Kiszka: > > - please make sure all patches have a commit message providing a > brief (or longer, in complex cases) reasoning for the change >
Thanks, will do. > - do not blindly copy file headers from existing files in to new one > but also update them as needed ;) > Thanks, fixed. > > And where in the patches are the external parser files you mentioned? > Didn't find them yet, but I might have been blind. > No, you're not. They are just implicitly mentioned in Makefile.am [1] by linking against libiniparser.a. What's missing (and will be fixed in a v2) is a mention in COMPILE.md. I went with iniparser [2], a small four-file library that links statically. Kind regards, Tobias [1] https://github.com/schtobia/efibootguard/blob/05ce60cb12f41629e3954124ef26c6594a128c81/Makefile.am#L138 [2] https://github.com/ndevilla/iniparser -- You received this message because you are subscribed to the Google Groups "EFI Boot Guard" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/efibootguard-dev/83cba4c6a11f73047794b9e28c7c295af5f71952.camel%40siemens.com.
