[PATCH v1] automation: allow to rerun build script

Olaf Hering posted 1 patch 1 year ago
Failed in applying to current master (apply log)
automation/scripts/build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH v1] automation: allow to rerun build script
Posted by Olaf Hering 1 year ago
Calling build twice in the same environment will fail because the
directory 'binaries' was already created before. Use mkdir -p to ignore
an existing directory and move on to the actual build.

Signed-off-by: Olaf Hering <olaf@aepfle.de>
---
 automation/scripts/build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/automation/scripts/build b/automation/scripts/build
index 197d085f3e..9085cba352 100755
--- a/automation/scripts/build
+++ b/automation/scripts/build
@@ -36,7 +36,7 @@ fi
 cp xen/.config xen-config
 
 # Directory for the artefacts to be dumped into
-mkdir binaries
+mkdir -p binaries
 
 if [[ "${CPPCHECK}" == "y" ]] && [[ "${HYPERVISOR_ONLY}" == "y" ]]; then
     # Cppcheck analysis invokes Xen-only build.
Re: [PATCH v1] automation: allow to rerun build script
Posted by Stefano Stabellini 1 year ago
On Wed, 17 May 2023, Olaf Hering wrote:
> Calling build twice in the same environment will fail because the
> directory 'binaries' was already created before. Use mkdir -p to ignore
> an existing directory and move on to the actual build.
> 
> Signed-off-by: Olaf Hering <olaf@aepfle.de>

Acked-by: Stefano Stabellini <sstabellini@kernel.org>


> ---
>  automation/scripts/build | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/automation/scripts/build b/automation/scripts/build
> index 197d085f3e..9085cba352 100755
> --- a/automation/scripts/build
> +++ b/automation/scripts/build
> @@ -36,7 +36,7 @@ fi
>  cp xen/.config xen-config
>  
>  # Directory for the artefacts to be dumped into
> -mkdir binaries
> +mkdir -p binaries
>  
>  if [[ "${CPPCHECK}" == "y" ]] && [[ "${HYPERVISOR_ONLY}" == "y" ]]; then
>      # Cppcheck analysis invokes Xen-only build.
>