~elementary-apps/elementaryos/+git/distinst:restore

Last commit made on 2021-12-23
Get this branch:
git clone -b restore https://git.launchpad.net/~elementary-apps/elementaryos/+git/distinst

Branch merges

Branch information

Recent commits

7968335... by Michael Aaron Murphy <email address hidden>

fix: Alternative way of restoring accounts service files

307adc6... by Michael Aaron Murphy <email address hidden>

fix(restore): Backup and restore user icons

985759e... by Michael Aaron Murphy <email address hidden>

fix(restore): Ensure restored user has valid AccountsService file

50e83a4... by Brock <email address hidden>

fix: Slice by grapheme index rather than bytes

3eb6ece... by David Hewitt <email address hidden>

fix(vapi): Fix compilation with Vala 0.54

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.

9ec2291... by David Hewitt <email address hidden>

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.

0f7f8c6... by Jeremy Soller

Revert "feat: Activate ext4 casefolding support by default"

This reverts commit 0129d7fc7539d1831d07fd72a700a24497fa9074.

7690b7c... by Brock <email address hidden>

Only check REQUIRED_SECTORS if required

6d458ae... by Brock <email address hidden>

WIth some humility this time.

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.

642687d... by Brock <email address hidden>

Handle disk sector size correctly

Closes #271