Skip to content

Export settings and integrate them with home-manager - #568

Closed
xaltsc wants to merge 18 commits into
andrewrabert:mainfrom
xaltsc:settings-export
Closed

Export settings and integrate them with home-manager#568
xaltsc wants to merge 18 commits into
andrewrabert:mainfrom
xaltsc:settings-export

Conversation

@xaltsc

@xaltsc xaltsc commented Jul 1, 2026

Copy link
Copy Markdown

This solves a part of #559.

This is blocked by #566 #528 #554 #527 #532 #515 and I think that's it.

This PR does three things:

  1. Update the rust code so that it can export a schema of the settings in jfn_config::SettingsData through a hidden cli option --generate-settings-schema
  2. Create a CI workflow to update those schemas on push. I had to make a weird decision and put this as a job in build-nix.yml so that it would run after the compilation. I'm not sure this is needed though, maybe cachix would catch on the result of the compilation needed for an ab initio nix run ., making the build useless.
  3. Handle the schema in nix as a base for the programs.jellyfin-desktop.settings.* options.

As usual, a lot of this needs rebasing, some values pertaining to the cache are placeholder values because #542 is not closed.

The three relevant commits are (by name, since their hash will change as PRs get merged):

  1. Add an option to export settings jellyfin/jellyfin-desktop@c88bde0
  2. flake: use the generated settings schema to produce options jellyfin/jellyfin-desktop@f6efed6
  3. flake+ci: add regen settings job to build-nix workflow jellyfin/jellyfin-desktop@297de8e

Status:

@xaltsc
xaltsc force-pushed the settings-export branch 4 times, most recently from 417d65f to 57e7fe2 Compare July 2, 2026 00:01
@xaltsc
xaltsc force-pushed the settings-export branch from 57e7fe2 to 8c254ae Compare July 2, 2026 00:32
@xaltsc xaltsc mentioned this pull request Jul 4, 2026
@andrewrabert

Copy link
Copy Markdown
Owner

@xaltsc please combine the nix PR's and I'll marge asap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants