Re: [PATCH v2 00/14] Assorted Cleanups For prepare-kernel.sh

[Richard Weinberger <richard@sigma-star.at>]

Am Dienstag, 25. Juni 2024, 17:58:01 CEST schrieb 'Jan Kiszka' via upstream:
> On 25.06.24 17:46, Jan Kiszka wrote:
> > On 24.06.24 22:33, Richard Weinberger wrote:
> >> This series contains the following improvements and cleanups.
> >>
> >> - Implement --reverse to undo the effects of the script
> >> - Emit changes that work on any architecture, renders --arch= optional
> >> - Switch from /bin/bash to /bin/sh
> >> - Make shellcheck happy up to warning level, info level is still a lot of work
> >> - Convert the whole script to tabs
> >> - Remove dead and in vain code
> >> - Various fixes
> >>
> >> Changes since v1[0]:
> >> - Rebased to current next branch
> >> - Reorganized patch series
> >> - Worked in feedback from Jan and Florian
> >>
> >> Richard Weinberger (14):
> >>   prepare-kernel.sh: Convert to tabs
> >>   prepare-kernel.sh: Add reverse mode
> >>   prepare-kernel.sh: Emit architecture agnostic changes
> >>   prepare-kernel.sh: Improve kernel tree check
> >>   prepare-kernel.sh: Disable SC2115 warning around $linux_tree
> >>   prepare-kernel.sh: Remove --filterkvers= and --filterarch=
> >>   prepare-kernel.sh: Remove interactivity
> >>   prepare-kernel.sh: Don't depend on bash
> >>   prepare-kernel.sh: Remove --forcelink
> >>   prepare-kernel.sh: Fix dovetail check
> >>   prepare-kernel.sh: Define command line variables
> >>   prepare-kernel.sh: Remove dead code
> >>   prepare-kernel.sh: Don't allow uninitialized variables
> >>   prepare-kernel.sh: Remove kernel version check
> >>
> >>  scripts/Kconfig.frag      |   2 +-
> >>  scripts/prepare-kernel.sh | 587 +++++++++++++++++---------------------
> >>  2 files changed, 268 insertions(+), 321 deletions(-)
> >>
> >> [0]: https://lore.kernel.org/xenomai/20240525115806.12101-1-richard@nod.at/
> >>
> > 
> > Thanks, series with smaller adjustments in next now. The final diff is
> > limited to whitespaces, though.
> > 
> 
> Does not look good yet, at least regarding backward compatibility (we
> didn't change our CI rules, so...):
> 
> https://source.denx.de/Xenomai/xenomai/-/pipelines/21335

I'm checking right now what is causing this.

Thanks,
//richard

-- 
​​​​​sigma star gmbh | Eduard-Bodem-Gasse 6, 6020 Innsbruck, AUT
UID/VAT Nr: ATU 66964118 | FN: 374287y