PREMIRRORS ordering seem wrong?

PREMIRRORS ordering seem wrong?

[Joakim Tjernlund]

I have set:
INHERIT += "own-mirrors"
PREMIRRORS:prepend = "my git pre mirror expr."

looking at:
 bitbake -e my-image | grep PREMIRRORS=
I see that "my git pre mirror expr." is last in PREMIRRORS variable, that seems wrong?

To me "my git pre mirror expr." should be before the entries added by own-mirrors

   Jocke

Re: [OE-core] PREMIRRORS ordering seem wrong?

[Richard Purdie]

On Wed, 2024-11-13 at 10:30 +0000, Joakim Tjernlund via
lists.openembedded.org wrote:
> I have set:
> INHERIT += "own-mirrors"
> PREMIRRORS:prepend = "my git pre mirror expr."
> 
> looking at:
>  bitbake -e my-image | grep PREMIRRORS=
> I see that "my git pre mirror expr." is last in PREMIRRORS variable,
> that seems wrong?
> 
> To me "my git pre mirror expr." should be before the entries added by
> own-mirrors

INHERIT in conf files is deferred and not immediately processed.

Cheers,

Richard

Re: [OE-core] PREMIRRORS ordering seem wrong?

[Joakim Tjernlund]

On Wed, 2024-11-13 at 12:49 +0000, Richard Purdie wrote:
> On Wed, 2024-11-13 at 10:30 +0000, Joakim Tjernlund via
> lists.openembedded.org wrote:
> > I have set:
> > INHERIT += "own-mirrors"
> > PREMIRRORS:prepend = "my git pre mirror expr."
> > 
> > looking at:
> >  bitbake -e my-image | grep PREMIRRORS=
> > I see that "my git pre mirror expr." is last in PREMIRRORS variable,
> > that seems wrong?
> > 
> > To me "my git pre mirror expr." should be before the entries added by
> > own-mirrors
> 
> INHERIT in conf files is deferred and not immediately processed.
>  

OK, that seems to support my findings as own-mirrors also does PREMIRRORS:prepend and that
puts my own PREMIRRORS:prepend = "my git pre mirror expr." last in PREMIRRORS

It that how it should be?

 Jocke

Re: [OE-core] PREMIRRORS ordering seem wrong?

[Richard Purdie]

On Wed, 2024-11-13 at 13:22 +0000, Joakim Tjernlund wrote:
> On Wed, 2024-11-13 at 12:49 +0000, Richard Purdie wrote:
> > On Wed, 2024-11-13 at 10:30 +0000, Joakim Tjernlund via
> > lists.openembedded.org wrote:
> > > I have set:
> > > INHERIT += "own-mirrors"
> > > PREMIRRORS:prepend = "my git pre mirror expr."
> > > 
> > > looking at:
> > >  bitbake -e my-image | grep PREMIRRORS=
> > > I see that "my git pre mirror expr." is last in PREMIRRORS
> > > variable,
> > > that seems wrong?
> > > 
> > > To me "my git pre mirror expr." should be before the entries
> > > added by
> > > own-mirrors
> > 
> > INHERIT in conf files is deferred and not immediately processed.
> >  
> 
> OK, that seems to support my findings as own-mirrors also does
> PREMIRRORS:prepend and that
> puts my own PREMIRRORS:prepend = "my git pre mirror expr." last in
> PREMIRRORS
> 
> It that how it should be?

INHERIT has always been handled at the end of "conf file" parsing so
that isn't a bug.

The fact own-mirrors uses prepend is also as it has been for a long
time. Things are behaving as expected, the only question would be if
own-mirrors should do something differently but that could break other
users :/

Cheers,

Richard