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

default options use a invalid url #44

Open
Fjox opened this issue Oct 22, 2024 · 4 comments
Open

default options use a invalid url #44

Fjox opened this issue Oct 22, 2024 · 4 comments

Comments

@Fjox
Copy link

Fjox commented Oct 22, 2024

Build option not provided, using default one: Jellyfin
You can change it by passing option name as second argument for this script.
Tag URL not provided, using latest available version:
You can change it by passing tag URL as third argument for this script.


        Thanks to https://github.com/jeppevinkel for providing the pre-packaged jellyfin-tizen builds!
        These builds can be found at https://github.com/jeppevinkel/jellyfin-tizen-builds

        Using Jellyfin Tizen Build Jellyfin.wgt
        from release:


Attempting to connect to Samsung TV at IP address x.x.x.x
* Server is not running. Start it now on port 26099 *
* Server has started successfully *
connecting to x.x.x.x:26101 ...
connected to x.x.x.x:26101
Attempting to get the TV name...
Found TV name: <redacted>
Attempting to install jellyfin-tizen-builds Jellyfin.wgt from release:
https://github.com/jeppevinkel/jellyfin-tizen-builds/releases/download//Jellyfin.wgt
There is no package with named Jellyfin.wgt.
[Fail] Fail install. There is the problem about package.
Total time: 00:00:00.880

as you can see its trying to grab a url with // wich is invalid

@Georift
Copy link
Owner

Georift commented Dec 16, 2024

Hmm that's interesting, I just ran the container and didn't encounter this. I wonder if you're still having the issue?

It is a result coming from a network call, so we should probably add a check there to warn that we were unable to get a version string from the network.

@Georift
Copy link
Owner

Georift commented Dec 16, 2024

I've added a few more checks, I'll merge this shortly so hopefully in future we'll know whats causing this issue for us.

@Georift
Copy link
Owner

Georift commented Dec 28, 2024

@Fjox I've merged this now, I wonder if you could try again with the latest build, sha-dcc4e93 or latest.

@Fjox
Copy link
Author

Fjox commented Dec 28, 2024

@Fjox I've merged this now, I wonder if you could try again with the latest build, sha-dcc4e93 or latest.

I don't have another Samsung TV to try this on and I really don't want to do anything that could break my current installation...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants