jellyfin / jellyfin-tizen

Jellyfin Samsung TV Client
https://jellyfin.org
861 stars 69 forks source link

Windows Build error #233

Closed aaronmee closed 1 month ago

aaronmee commented 7 months ago

Hi, I want to build jellyfin-web on windows for my samsung smart tv. But I receive an error when trying to build jellyfin-web using "npm run build:production".

I have node.js 20.10.0 LTS installed, npm is up to date. At the end of the build process I get this line: webpack 5.70.0 compiled with 1 error and 1 warning in 38751 ms The error is this: ERROR in ./components/images/blurhash.worker.ts \ Module build failed (from ../node_modules/worker-loader/dist/cjs.js):

I followed the build on windows guide exactly, but sadly it didn't work.

Any help is appreciated!

Regards

dmitrylyzo commented 7 months ago

NodeJS 20.10.0 + NPM 10.2.4 jeff-1

Make sure that the modules are installed correctly.

aaronmee commented 7 months ago

npm: 10.2.4 Node.js: v20.10.0

Commands:

Output: grafik

Omitting npx update-browserslist-db@latest results in the same

aaronmee commented 7 months ago

Update:

I resolved the issue with installing yarn: npm install --global yarn

It's working now!

dmitrylyzo commented 7 months ago

Update:

I resolved the issue with installing yarn: npm install --global yarn

It's working now!

Weird. Yarn is deprecated in jellyfin-web. AFAIK, the set command doesn't work in PowerShell. There is a different syntax for setting an environment variable (see collapsed blocks in the Wiki).

aaronmee commented 7 months ago

Weird. Yarn is deprecated in jellyfin-web. AFAIK, the set command doesn't work in PowerShell. There is a different syntax for setting an environment variable (see collapsed blocks in the Wiki).

I'm using $env:ENV= to set environment variables, as described in the collapsed blocks. I see that yarn isn't used anymore. But in an older youtube tutorial I saw that it once was a dependency, so I installed it. And then it suddenly worked. Apart from installing yarn, I didn't do anything differently than before.

aaronmee commented 1 month ago

Closing this