[Bug 695290] Re: 10_lupin case problem with ntfs UUIDs

[Colin Watson]

The testing done in bug 610898 should be sufficient to cover this.
Marking verification-done.

** Tags added: verification-done
** Tags removed: verification-needed

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to grub2 in Ubuntu.
https://bugs.launchpad.net/bugs/695290

Title:
  10_lupin case problem with ntfs UUIDs

Status in “grub2” package in Ubuntu:
  Fix Released
Status in “grub2” source package in Lucid:
  Fix Committed
Status in “grub2” source package in Maverick:
  Fix Committed

Bug description:
  1) Ubuntu 10.10
  2) lupin-support        0.32

  Normally, when running update-grub, 10_lupin should add entries to
  grub.cfg with the root=UUID=x parameter.

  On my system, this fails and the script falls back to using device
  nodes (/dev/sda?) - which prevented the system from booting correctly,
  because the device order gets mixed on every boot (sometimes it's sda,
  sometimes sdb).

  I found out the problem is on line 53 in /etc/grub.d/10_lupin:
  host_device_uuid="`${grub_probe} --device "${host_device}" --target=fs_uuid 2> /dev/null`" || true

  this returns the UUID in lower case, and on the next line this gets compared to /dev/disk/by-uuid/*:
  test -e "/dev/disk/by-uuid/${host_device_uuid}"

  But the device node in /dev/disk/by-uuid is in uppercase.

  Workaround is to change line 53 to:
  host_device_uuid="`${grub_probe} --device "${host_device}" --target=fs_uuid 2> /dev/null | tr '[:lower:]' '[:upper:]'`" || true

  But I guess this is not correct (the conversion to uppercase should
  only happen on NTFS hosts) - so a more proper fix would be necessary.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/grub2/+bug/695290/+subscriptions