-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
jellyfin-web does not build on master #176288
Comments
Yeah, I suspect the right solution here would be a fix that covers all the affected packages, rather than doing it on a case-by-case basis. |
Due to "jellyfin-web does not build on master" See: `https://github.com/NixOS/nixpkgs/issues/176288`
Excuse my above commit referencing this issue. I wrongly escaped the URL referencing this issue. |
is it known what caused this issue? |
Yes:
|
jellyfin-web now builds on master, since the NPM update was reverted in #177002. If upstream does not remove the dependency on the home directory, I think it might be best to open a new common issue instead of one for each affected package, so I'm closing this issue. Feel free to re-open if you feel otherwise. |
Due to "jellyfin-web does not build on master" See: `https://github.com/NixOS/nixpkgs/issues/176288`
Due to "jellyfin-web does not build on master" See: `https://github.com/NixOS/nixpkgs/issues/176288`
Steps To Reproduce
nix-build -A jellyfin
at commit 0fe729dBuild log
Additional context
Same root cause as #176127, see that issue for more details.
As a workaround change this line:
nixpkgs/pkgs/servers/jellyfin/web.nix
Line 59 in 0fe729d
to
HOME="$(mktemp -d)" npm run build:production
This seems to be impacting a few packages. I am not sure if I should open a pull-request with my workaround, or wait for a fix for npm.
Notify maintainers
@nyanloutre @minijackson @purcell @jojosch
Metadata
Please run
nix-shell -p nix-info --run "nix-info -m"
and paste the result.The text was updated successfully, but these errors were encountered: