Your one-stop-cake-shop for everything Freshly Baked has to offer

fix(kavita): move libraries to /var/lib/libraries #107

merged opened by a.starrysky.fyi targeting main from private/minion/push-wznllvxvuqmz

Previously we had our libraries in a subdirectory of /var/lib/copyparty

While that's an OK idea in theory, in practice it'd require us to make every directory on the way to /var/lib/copyparty/data/groups/library +rx to kavita. That's something we're unwilling to do. The true fix is to move this out so we don't have to meddle with permissions except those which are directly on this directory

Labels
merge-when-green
requested-reviewers
approved

None yet.

tested-working
rejected

None yet.

assignee

None yet.

Participants 1
AT URI
at://did:plc:uuyqs6y3pwtbteet4swt5i5y/sh.tangled.repo.pull/3m4td45yypx22
+11 -1
Diff #4
+11 -1
packetmix/systems/teal/copyparty.nix
··· 115 115 }; 116 116 }; 117 117 "/groups/library" = { 118 - path = "/var/lib/copyparty/data/groups/library"; 118 + path = "/var/lib/libraries"; 119 119 120 120 access = { 121 121 A = [ ··· 140 140 builtins.attrValues 141 141 (map (mount: mount.path)) 142 142 (map (lib.removePrefix "/var/lib/")) 143 + (builtins.filter (name: !(builtins.elem name [ "libraries" ]))) # For shared directories we can't afford for systemd to change permissions on us... 143 144 (lib.concatStringsSep " ") 144 145 ]); 145 146 }; ··· 293 294 294 295 clicks.storage.impermanence.persist.directories = [ 295 296 "/var/lib/copyparty" 297 + { 298 + directory = "/var/lib/libraries"; 299 + mode = "0770"; 300 + group = "copyparty+kavita"; 301 + defaultPerms = { 302 + mode = "0770"; 303 + group = "copyparty+kavita"; 304 + }; 305 + } 296 306 ]; 297 307 }; 298 308 }

History

5 rounds 0 comments
sign up or login to add to the discussion
1 commit
expand
fix(kavita): move libraries to /var/lib/libraries
expand 0 comments
pull request successfully merged
1 commit
expand
fix(kavita): move libraries to /var/lib/libraries
expand 0 comments
1 commit
expand
fix(kavita): move libraries to /var/lib/libraries
expand 0 comments
1 commit
expand
fix(kavita): move libraries to /var/lib/libraries
expand 0 comments
1 commit
expand
fix(kavita): move libraries to /var/lib/libraries
expand 0 comments