[Buildroot] [PATCH v3 2/9] testing/infra/builder: split configure() from build()

[Thomas Petazzoni]

Hello,

On Sun, 01 Apr 2018 18:32:01 -0300, Ricardo Martincoski wrote:

> It is possible to do that.
> 
> The next patch on the same series uses .update() so the local "env" is changed.
>         env = {"PATH": os.environ["PATH"]}
>         env.update(make_extra_env)
> 
> Note: I did not tested the code below.
> 
> In order to always have a clean environment, we could do in the constructor:
>         self.env = {"PATH": os.environ["PATH"]}
> And change the methods to:
>         env = self.env.copy()
>         env.update(make_extra_env)
> 
> Or we assume we will need only PATH from env in the future, and do in the
> constructor:
>         self.env_path = os.environ["PATH"]
> And change the methods to:
>         env = self.make_extra_env.copy()
>         env.update({"PATH": self.env_path})
> 
> The access to os.environ should be inexpensive (I did not measured it), so I am
> not sure it needs to be done. I am not against it either.
> At first glance, it seems just more code to me.
> Maybe I am missing something.

Hum, indeed. Let's see what it gives with the next patches, and we can
always adjust that kind of details later.

Thanks for your feedback!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com