On 10/18/23 13:28, Marcin Juszkiewicz 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.
Project URL (to clone and build): https://projec...@dev.azure.com/projectmu/Uncrustify/_git/Uncrustify Command line options to pass to the binary: -c ${EDK2_ROOT}/.pytool/Plugin/UncrustifyCheck/uncrustify.cfg \ --replace \ --no-backup \ --if-changed \ You can specify the pathnames to reformat as operands, or you can additionally pass the "-F -" option, and then the pathnames will be read from stdin. Make sure you have committed or at least staged your changes, before running this command, because these options will update the files in-place. The only really important option is of course the "-c" one. Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#109748): https://edk2.groups.io/g/devel/message/109748 Mute This Topic: https://groups.io/mt/102035954/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
