Re: [Buildroot] [PATCH 07/18 v3] support/scripts: move merged-usr errors message into check-merged-usr.sh

["Yann E. MORIN via buildroot" <buildroot@buildroot.org>]

Romain, All,

On 2025-11-05 11:52 +0100, Romain Naour spake thusly:
> Le 01/09/2025 à 11:01, Yann E. MORIN via buildroot a écrit :
[--SNIP--]
> > +	@$(call MESSAGE,"Sanity check in overlays $(call qstrip,$(BR2_ROOTFS_OVERLAY))")
> > +	support/scripts/check-merged-usr.sh \
> > +		--type overlay \
> > +		$(call qstrip,$(BR2_ROOTFS_OVERLAY))
> BR2_ROOTFS_OVERLAY can have a list of overlay directory.
> We had a foreach loop to check all of them.

Exactly. Now they are passed as arguments to the check-merged-usr.sh
script, so that...

[--SNIP--]
> > +for root; do
> I'm not sure to follow here.
> Where root is defined?

... they now iterated over by this for-loop.

When there is no 'in' part to a for loop, it iterates over the
positional parameters:

    $ cat foo
    #!/bin/sh
    for i; do printf 'i=%s\n' "${i}"; done

    # ./foo a b c d
    i=a
    i=b
    i=c
    i=d

So, those two lines are equivalent:

    for i; do
    for i in "${@}"; do

See section 2.9.4.2 of the Open Group Base Specifications Issue 8:
    https://pubs.opengroup.org/onlinepubs/9799919799/utilities/V3_chap02.html#tag_19_09_04

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot