On Tue, Nov 07, 2023 at 11:41:05PM +0200, Francis Laniel wrote: > Adds new file cli_hush_2021.c, it is a copy of Busybox hush file as it was of > time to commit 37460f5da.
I'm not sure where best to say this in the series so I'll try here. I think we should name the hush.c file being imported from busybox as "cli_hush_upstream.c" and the u-boot glue file around it as "cli_hush_modern.c". Then instead of "CLI_HUSH_2021" etc, "CLI_HUSH_MODERN" and then in due time, we drop CLI_HUSH_OLD and rename CLI_HUSH_MODERN to CLI_HUSH. > This commit modifies Busybox hush to not compile some part specific to Busybox > and adds some code needed by U-Boot. > The modifications consists mainly on adding code #if(n)def guards. This patch is also where we should make the one line removal of "# undef CONFIG_FEATURE_SH_STANDALONE" from the upstream file, for CI. -- Tom
signature.asc
Description: PGP signature