New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[18.09] nixos/emby: use the dataDir option #48150
Conversation
cc @xeji |
@Vskilet btw it's nice to be able to tell if this is a backport by having the pr title include |
@GrahamcOfBorg build emby |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: emby Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: emby Partial log (click to expand)
|
@Vskilet Could you please stick to the contribution guidelines in your PRs and not add an extra space before colons in commit messages in the future? Also, please use For general amusement, we have a word for this in Germany: https://en.wikipedia.org/wiki/Plenken 😄 |
(cherry picked from commit a16c6d8)
(cherry picked from commit bd02b33)
(cherry picked from commit 6e8da1d)
(cherry picked from commit f7a2e20)
(cherry picked from commit 9ec17c6)
(cherry picked from commit c2e269e)
(cherry picked from commit bb06b5b)
(cherry picked from commit 7d43e2a)
(cherry picked from commit 751c647)
9ec2560
to
2c5f94b
Compare
Do we want/need to backport the version bump also? Naive question really, I did not follow previous work |
Yes and yes. I've tested in a vm with the 18.09 release and switching to this version hasn't caused any problems.
|
@GrahamcOfBorg build emby |
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: emby Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: emby Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: emby Partial log (click to expand)
|
Motivation for this change
In #48043 @joachifm suggest to add the pull request to the release. I added other commits to assure compatibility.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)