Using Vala 0.54 with this vapi file results in the following error:
```
distinst.vapi:683.9-683.27: error: [SimpleType] struct `Distinst.SectorResult' cannot have owned heap-allocated fields
public string error;
```
`SimpleType` should only be used for stack allocated structures, usually just when a builtin type gets typedef'ed to something in C. Since distinst doesn't manage the lifetime of this structure, we can remove the `SimpleType` and let Vala generate a destroy function for it to clean up the strings left on the heap.
As far as I can tell, this structure or the `from_str` method that returns it isn't used in either the elementary or Pop!_OS version of the installer, so the chance of regressions here should be low.
chore(debian): Drop grub-efi dependency for elementary
We've started building Ubuntu 22.04 based images and `grub-efi` no longer exists in Ubuntu `jammy`. It was always just a dummy package that pulled in `grub-efi-amd64` anyway. Dropping this here will stop installation failures due to that package not existing.
Should be the same outcome, but without running `boot.get_logical_block_size()` unless absolutely necessary.
I think this is kinda a hack. Personally I think `BlockDeviceExt::get_device_name()` should return `Result<String>`, because it has the ability to fail. This PR is trying to avoid calling `boot.get_logical_block_size()` here, because `get_logical_block_size()` calls `get_device_name()`, thus both can cause distinst to panic.