[Buildroot] [PATCH buildroot-test] scripts/autobuild-run: calculate dldir as an absolute path

[Atharva Lele]

On Sat, Sep 14, 2019 at 7:05 PM Thomas Petazzoni
<thomas.petazzoni at bootlin.com> wrote:
>
> Right now, self.dldir is a relative path, equal to
> "instance-X/dl". Then, the Buildroot build is executed with
> BR2_DL_DIR=self.dldir, which means that self.dldir is interpreted
> relatively to the Buildroot source directory.
>
> Due to this, we get the following directory organization:
>
>  instance-0
>   + buildroot
>     + instance-0
>       + dl
>   + instance.log
>   + output
>
> Instead of the excepted organization:
>
>  instance-0
>  + buildroot
>  + dl
>  + instance.log
>  + output
>
> We fix this issue (which was introduced when the Builder class was
> created) by making self.dldir an absolute path.
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni at bootlin.com>

I had not noticed this..

Thanks for fixing it!

Reviewed-by: Atharva Lele <itsatharva at gmail.com>

Regards,
Atharva Lele

> ---
>  scripts/autobuild-run | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/scripts/autobuild-run b/scripts/autobuild-run
> index ead81a0..7e3c7a0 100755
> --- a/scripts/autobuild-run
> +++ b/scripts/autobuild-run
> @@ -270,7 +270,7 @@ class Builder:
>          # frequently needed directories
>          self.idir = "instance-%d" % self.instance
>          self.srcdir = os.path.join(self.idir, "buildroot")
> -        self.dldir = os.path.join(self.idir, "dl")
> +        self.dldir = os.path.abspath(os.path.join(self.idir, "dl"))
>          # We need the absolute path to use with O=, because the relative
>          # path to the output directory here is not relative to the
>          # Buildroot sources, but to the location of the autobuilder
> --
> 2.21.0
>