search

jellyfin / jellyfin-tizen

Jellyfin Samsung TV Client
https://jellyfin.org
1k stars 79 forks source link

Issue with Building on Windows #154

Closed Simuman closed 2 years ago

Simuman commented 2 years ago

Followed instructions as far as Build Jellyfin Web, as soon as I run the command "npm ci --no-audit" get multiple errors in Powershell (see below).

Have just clean installed NPM, NodeJS and Tizen on this Windows 10 system running on I7 7700K CPU 32GB RAM and GTX 1080ti for this build. Versions of apps shown below.

NPM Ver = 9.1.1

NodeJS x18.12.1

Tizen Studio Version 5.0 Build Time 2022-04-18 14:12

Screenshot 2022-11-16 233945 2022-11-16T10_25_59_748Z-debug-0.log

dmitrylyzo commented 2 years ago

jellyfin/jellyfin-web@57cd4f91f22a6b631dffb93c92727617b810f16d Node.JS 18.12.1 (NPM 8.19.2) :heavy_check_mark: Node.JS 18.12.1 (NPM 9.1.1) :heavy_check_mark: Both in PowerShell.

ps1

Not sure why the output differs (assets, entrypoint). The prepare script is (almost) deprecated. Run npm run build:production.

Simuman commented 2 years ago

Thx for quick reply.

Unfortunately ran into another error after running the command you advised which ran ok. Then tried running the next part of the instructions as per Github guide and got this error. Checked the folder and quite correctly Index.html does not exist.

PS: No www folder created.

Next error 2022-11-17 025613 2022-11-16T13_49_38_834Z-debug-0.log

dmitrylyzo commented 2 years ago

You forgot /dist in the JELLYFIN_WEB_DIR.

Simuman commented 2 years ago

Thanks I managed to pick that up. Now getting error running the line;

tizen build-web -e "." -e gulpfile.js -e README.md -e "node_modules/" -e "package*.json" -e "yarn.lock"

Had to put the .\ in front for it to pickup the command correctly, and now get this.

Screenshot 2022-11-17 043848

Then kicks me out and my antivirus pops open telling me of Malicious activity and quarantines a .js file.

Man this is getting so frustrating.

dmitrylyzo commented 2 years ago

tizen.bat should be used. And

All tools (yarn, node, tizen) are assumed to be added to the PATH environment variable.

Or use a full path.

Simuman commented 2 years ago

Hi dmitrylyzo,

Thanks for all the help. Finally managed to build the file correctly, as was missing some important syntax not shown on the guide. Very happy now that I've got it install on my TV. Hopefully there will be a installation file for Samsung TV's soon, so many more self hosters will be able to implement this app as it's great.