Skip to content
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

Closed
newAM opened this issue Jun 4, 2022 · 5 comments
Closed

jellyfin-web does not build on master #176288

newAM opened this issue Jun 4, 2022 · 5 comments

Comments

@newAM
Copy link
Member

newAM commented Jun 4, 2022

Steps To Reproduce

nix-build -A jellyfin at commit 0fe729d

Build log

these 2 derivations will be built:
  /nix/store/0r26521cd4ihaw9ia1q320819i7rzkya-jellyfin-web-10.7.7.drv
  /nix/store/k1lv8acvrbaqmms5dx7ihqw8nay95sv8-jellyfin-10.7.7.drv
building '/nix/store/0r26521cd4ihaw9ia1q320819i7rzkya-jellyfin-web-10.7.7.drv'...
unpacking sources
unpacking source archive /nix/store/4n28kmji75gd671zm8v6xg8ywghv3s2s-source
source root is source
patching sources
configuring
no configure script, doing nothing
building

error: builder for '/nix/store/0r26521cd4ihaw9ia1q320819i7rzkya-jellyfin-web-10.7.7.drv' failed with exit code 243;
       last 8 log lines:
       > unpacking sources
       > unpacking source archive /nix/store/4n28kmji75gd671zm8v6xg8ywghv3s2s-source
       > source root is source
       > patching sources
       > configuring
       > no configure script, doing nothing
       > building
       >
       For full logs, run 'nix log /nix/store/0r26521cd4ihaw9ia1q320819i7rzkya-jellyfin-web-10.7.7.drv'.
error: 1 dependencies of derivation '/nix/store/k1lv8acvrbaqmms5dx7ihqw8nay95sv8-jellyfin-10.7.7.drv' failed to build

Additional context

Same root cause as #176127, see that issue for more details.

As a workaround change this line:

npm run build:production

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.

[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.43, NixOS, 22.05 (Quokka)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.1`
 - channels(alex): `""`
 - channels(root): `"nixos-21.11, nixos-unstable, nixos-unstable-small"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`
@purcell
Copy link
Member

purcell commented Jun 5, 2022

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.

soupglasses added a commit to soupglasses/phenix that referenced this issue Jun 5, 2022
Due to "jellyfin-web does not build on master"
See: `NixOS/nixpkgs#176288
@soupglasses
Copy link
Member

Excuse my above commit referencing this issue. I wrongly escaped the URL referencing this issue.

@Kranzes
Copy link
Member

Kranzes commented Jun 7, 2022

is it known what caused this issue?

@newAM
Copy link
Member Author

newAM commented Jun 8, 2022

is it known what caused this issue?

Yes:

@minijackson
Copy link
Member

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.

soupglasses added a commit to soupglasses/phenix that referenced this issue Jun 13, 2022
Due to "jellyfin-web does not build on master"
See: `NixOS/nixpkgs#176288
soupglasses added a commit to soupglasses/phenix that referenced this issue Jun 13, 2022
Due to "jellyfin-web does not build on master"
See: `NixOS/nixpkgs#176288
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants