On Sat, Jun 21, 2025 at 9:52 AM Andrew Hamilton <adham...@gmail.com> wrote:

> Correct a test failure in netboot_test and file_filter_test
> caused by an issue cleaning up the tmp directory created
> for netboot. Netboot creates a subdirectory in the tmp
> folder that causes the rmdir to fail - so cleanup the
> subdirectory first.
>
> Fixes: 1d59f39b5f1b ("tests/util/grub-shell: Remove the
> work directory on successful run and debug is not on")
>
> Signed-off-by: Andrew Hamilton <adham...@gmail.com>
>

Tested-by: Leo Sandoval <lsand...@redhat.com>

---
>  tests/util/grub-shell.in | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/tests/util/grub-shell.in b/tests/util/grub-shell.in
> index 8baa3667a..2a874a85d 100644
> --- a/tests/util/grub-shell.in
> +++ b/tests/util/grub-shell.in
> @@ -711,6 +711,9 @@ elif [ x$boot = xemu ]; then
>      test -n "$debug" || rm -rf "$rootdir"
>      test -n "$debug" || rm -f "$roottar"
>  fi
> +if [ x$boot = xnet ]; then
> +    test -n "$debug" || rm -rf "$work_directory/netdir"
> +fi
>  test -n "$debug" || rm -f "${isofile}"
>  test -n "$debug" || rm -rf "${rom_directory}"
>  test -n "$debug" || rm -f "${tmpfile}" "${cfgfile}" "${goutfile}"
> --
> 2.39.5
>
>
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/grub-devel
>
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to