manage flatpak remotes w/ `services.flatpak.remotes`

[TH3-S4LM0N]

Nothing exploded on my system so it's probably fine

[GermanBread]

Looks great! However it should also remove remotes NOT declared in the config

[TH3-S4LM0N]

However it should also remove remotes NOT declared in the config

How do you suggest to do that?

for i in "$(flatpak remotes --columns=name)"; do
   flatpak remote-delete $i
done

would delete all remotes and re-add them which seems inefficient

[GermanBread]

You can try diffing the remotes declared in the config vs what flatpak remotes --columns=name,url returns.

Take a look at how the module does diffing in the "uninstall" part

---

My brain also considered the possibility of declaring remotes like this (would factor out formatting inconsistencies between the flatpak cli and malformed strings in the user's nixos config)

services.flatpak.remotes = [
  {
    name = "foo";
    url = "https://example.com/foo.flatpakref";
  };
];

You'd then have to reconstruct the string somehow

[GermanBread]

Hi, just to let you know, I'm still interested in getting this merged.

Edit: I could merge this PR as-is, but I would have to resolve the review above myself.

[GermanBread]

I implemented this feature myself