[bitbake-devel] [OE-core] $PWD in terminal.bbclass

[Richard Purdie]

On Tue, 2019-04-30 at 13:20 +0200, Enrico Scholz via Openembedded-core
wrote:
> Assigning PWD works like 'cd' and changes the actual path so that
> do_terminal() will be executed from the directory where bitbake has
> been started from.  But usually, the [dirs] directory (e.g. '${B}')
> is
> expected.
> 
> Is this really intented?  Here, this breaks kernel's '-c menuconfig'
> (at least).

Its not intended.

> A trivial fix would be something like
> 
> >    envdata.delVar("PS1")
> > +   envdata.delVar("PWD")
> 
> in terminal.bbclass.  But is there any reason to have PWD in
> ${BB_ORIGENV}
> at all?  imo, there is no reason to keep this variable and it should
> be removed in bitbake already.

I'm fine with the delVar. I'm nervous about "randomly" deleting bits of
the origenv, it would tell us where bitbake was executed from for
example and I think I'd prefer to leave it unaltered.


Cheers,

Richard