[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug #52746] An extra trailing slash is added to ZFS root pool and it's
From: |
Jozsef Fejes |
Subject: |
[bug #52746] An extra trailing slash is added to ZFS root pool and it's not bootable |
Date: |
Tue, 26 Dec 2017 16:31:25 -0500 (EST) |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.24 Safari/537.36 |
Follow-up Comment #1, bug #52746 (project grub):
Some more context:
Grub apparently passes this (invalid) value to initramfs, which is processed
here under Debian:
https://anonscm.debian.org/cgit/pkg-zfsonlinux/zfs.git/tree/contrib/initramfs/scripts/zfs#n770
And this is apparently passed verbatim to zfsonlinux, which dies here:
https://github.com/zfsonlinux/zfs/search?utf8=%E2%9C%93&q=%22trailing+slash+in+name%22&type=
I think it sounds sane that ZOL has this validation. One might argue that the
first link is a bit confusing since their format string includes the slash
unconditionally, so at least their documentation should be fixed. I'm not sure
why this file is under a Debian repo, probably other distros have a different
file with similar functionality?
I'm pretty sure life would be great if update-grub produced a valid name, ie.
without the trailing slash. That should hopefully not break any existing
setup.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/bugs/?52746>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/