[Fuego] [PATCH 1/2] kernel_build: add per job build folder by default

Bird, Timothy Tim.Bird at sony.com
Wed Nov 22 06:39:27 UTC 2017


Applied and pushed.  Thanks!
 -- Tim

> -----Original Message-----
> From: fuego-bounces at lists.linuxfoundation.org [mailto:fuego-
> bounces at lists.linuxfoundation.org] On Behalf Of Daniel Sangorrin
> Sent: Tuesday, November 21, 2017 6:18 PM
> To: fuego at lists.linuxfoundation.org
> Subject: [Fuego] [PATCH 1/2] kernel_build: add per job build folder by
> default
> 
> This patch fixes a conflict that occurs when trying to
> build different kernel_build specs, by cloning and
> building the kernel on a per-spec build folder.
> The patch sets the PER_JOB_BUILD variable to "true"
> by default. For that reason, there is no need to
> set it on the spec file.
> 
> There is a small inefficiency in this approach: specs
> that use the same kernel git repository but different
> branches or config files will still have separate
> build folders allocated for them. This means they
> will not be able share the cost (time) of cloning
> the git repository. Note that this cost is
> somehow reduced by the fact that we use the git
> parameters depth=1 and branch=<branchname>.
> For now, I think we can live with this inefficiency
> and leave it for future work.
> 
> Signed-off-by: Daniel Sangorrin <daniel.sangorrin at toshiba.co.jp>
> ---
>  engine/tests/Functional.kernel_build/fuego_test.sh | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/engine/tests/Functional.kernel_build/fuego_test.sh
> b/engine/tests/Functional.kernel_build/fuego_test.sh
> index 414ad32..daa7d24 100755
> --- a/engine/tests/Functional.kernel_build/fuego_test.sh
> +++ b/engine/tests/Functional.kernel_build/fuego_test.sh
> @@ -1,3 +1,5 @@
> +FUNCTIONAL_KERNEL_BUILD_PER_JOB_BUILD="true"
> +
>  function test_pre_check {
>      echo "Doing a pre_check"
>      # FIXTHIS: if making uImage, check for mkimage
> --
> 2.7.4
> 
> 
> _______________________________________________
> Fuego mailing list
> Fuego at lists.linuxfoundation.org
> https://lists.linuxfoundation.org/mailman/listinfo/fuego


More information about the Fuego mailing list