Re: [OE-core] more ptest pedantry: why conditionally include "run-ptest" in SRC_URI?

["Robert P. J. Day" <rpjday@crashcourse.ca>]

[-- Attachment #1: Type: text/plain, Size: 1339 bytes --]

On Mon, 12 Feb 2024, Ross Burton wrote:

> On 12 Feb 2024, at 06:39, Robert P. J. Day via lists.openembedded.org <rpjday=crashcourse.ca@lists.openembedded.org> wrote:
> >  more annoying nitpickery ... here:
> >
> > https://git.openembedded.org/openembedded-core/tree/meta/recipes-extended/xz/xz_5.4.5.bb#n28
> >
> > it's not clear why someone would go to the trouble of conditionally
> > including run-ptest in SRC_URI as i would have thought the only
> > benefit would be to avoid the triviality of copying run-ptest into
> > WORKDIR, but beyond that, i don't see what the point is, and that's
> > the only recipe i've run across that bothers to do that.
> >
> >  is there something more subtle happening here?
>
> No good reason, and I’d approve a patch that removed the
> conditionals on the grounds of cleaning up the recipe and ensuring
> that there’s less conditional paths through the recipe.

  additional note in that, if a recipe inherits ptext, then it's silly
to conditionally include "run-ptest" in SRC_URI. but it's trickier if
something else in SRC_URI is included conditionally:

https://git.openembedded.org/openembedded-core/tree/meta/recipes-devtools/flex/flex_2.6.4.bb#n18

without knowing what that patch does, it seems like you kind of have
to leave it as conditional. maybe other examples are more obvious.

rday