On Tue, Aug 13, 2024 at 06:14:40PM +0200, Thomas Schmitt wrote:
> Hi,
>
> thinking more i believe that the currently used mkdir option -p is
> inappropriate in  tests/grub_cmd_cryptomount.in .
>
> It hampers proper cleanup because the script cannot know how many
> directories in the path to TMPDIR were created and need to be removed.
> It is unusual because about all other tests in GRUB assume that non-empty
> TMPDIR or its default "/tmp" lead to an existing directory.
> (At least my local mktemp(1) refuses on non-existing components in
> the path.)
>
> If i read no objections, i will propose removal of -p in patch v2.

I am OK with this change but I would want to hear Glenn's and Vladimir's
opinion here.

Daniel

PS I have fixed Glenn's email address...

_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to