On Wed, Oct 18, 2023 at 12:28 PM Marcin Juszkiewicz <marcin.juszkiew...@linaro.org> wrote: > > W dniu 18.10.2023 o 13:23, Pedro Falcato pisze: > > On Wed, Oct 18, 2023 at 12:16 PM Marcin Juszkiewicz > > <marcin.juszkiew...@linaro.org> wrote: > >> > >> W dniu 18.10.2023 o 12:32, Nhi Pham pisze: > >>> Acked-by: Nhi Pham <n...@os.amperecomputing.com> > >>> > >>> Nit: I think you want to run uncrustify for Patch 3 as well :) > >> > >> Done, will check other changes too. > >> > >> I have a strong feeling that Qemu part of EDK2 needs a bit > >> bigger patch when it comes to formatting: > >> > >> Platform/Qemu/QemuOpenBoardPkg/Include/Library/QemuOpenFwCfgLib.h > >> | 7 +- > >> Platform/Qemu/QemuOpenBoardPkg/Library/PeiReportFvLib/PeiReportFvLib.c > >> | 33 +- > >> > >> Platform/Qemu/QemuOpenBoardPkg/Library/PlatformSecLib/Ia32/SecEntry.nasm > >> | 99 ++- > >> Platform/Qemu/QemuOpenBoardPkg/PlatformInitPei/Memory.c > >> | 108 +-- > >> Platform/Qemu/QemuOpenBoardPkg/PlatformInitPei/Pcie.c > >> | 30 +- > >> Platform/Qemu/QemuOpenBoardPkg/README.md > >> | 43 +- > > > > Something must be wrong with your config because QemuOpenBoardPkg is > > and was, AFAIK, all formatted using uncrustify. And if I run it > > locally, it seems to agree with me. > > EDK2 expects some random version of uncrustify. > > It is not part of BaseTools so I use upstream version. And it looks like > they format in different way using the same config file.
Right. But we used the correct uncrustify version, so things are well formatted. I don't see the point in formatting with upstream uncrustify, you're just going to end up misformatting everything. Whether the current sanctioned solution is any sane at all (it is not) is another matter, but I seriously have no stamina to discuss these kinds of changes anymore. -- Pedro -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#109733): https://edk2.groups.io/g/devel/message/109733 Mute This Topic: https://groups.io/mt/102035954/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
