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

broken now with new site layout #58

Closed
sjorge opened this issue Jun 24, 2016 · 14 comments
Closed

broken now with new site layout #58

sjorge opened this issue Jun 24, 2016 · 14 comments
Labels

Comments

@sjorge
Copy link

sjorge commented Jun 24, 2016

The new site layout seems to have broken everything, not really unexpected though.

@Mawuena
Copy link

Mawuena commented Jun 24, 2016

Hello,

Thank you @sjorge for posting this.
You beat me to it.

I'm not sure MrWorf will be able to help without information from the Plex team about the new URL.
I've posted something on the Plex Forum about it (https://forums.plex.tv/discussion/222567/plex-media-server-update-automation-download-page-url-change#latest)

So if we can put pressure on them on their side to get an answer, it'd be perfect.
Unless @mrworf have a solution/idea on how to resolve this ?

Sincerely,

Mawuena

@mrworf
Copy link
Owner

mrworf commented Jun 24, 2016

Yes, they've made it technically easier, but it's a major change from previously, so it will take me a day or two (especially since today is midsommar celebration in Sweden, which is as big as Thanksgiving in the US if not bigger)

@Mawuena
Copy link

Mawuena commented Jun 24, 2016

You rock @mrworf
Thank you very much !!!!

@Mawuena
Copy link

Mawuena commented Jun 24, 2016

And happy midsommar celebration.
Never heard of it but glad I'll sleep a little less dumb tonight :)

@mrworf
Copy link
Owner

mrworf commented Jun 24, 2016

@Mawuena @sjorge Pull and try it now, had to move away from RELEASE scheme, but the new setup wiil eventually support ARM and others. Close the issue if it solved your problems.

@mrworf mrworf added the bug label Jun 24, 2016
@Tommatheussen
Copy link
Contributor

Tommatheussen commented Jun 24, 2016

It sort of works, I was able to install the new version on my Ubuntu 14.04 32 bit, I think it can use some tweaks to the DISTRO, BUILD and DISTRO_INSTALL options.

I had to specify DISTRO, BUILD and DISTRO_INSTALL for it too work (since 64 bit is default, and I needed 32 bit).

I tried with a couple options:

  • Only BUILD, downloaded 64 bit, so it's ignored
  • BUILD and DISTRO, downloaded correct version, but tried to execute sudo /path/to/file (so now dpkg -i)
  • 3rd time, with all options specified, that worked.

Anyways, not complaining, just giving the feedback for improvements ;)
Very nice job man, been using your script for quite a while now, never had any real trouble.

Keep it up! :)

@Mawuena
Copy link

Mawuena commented Jun 24, 2016

Am at work and will only be able to test it tonight (Fedora 23 Server, 64 bits)
I may be luckier than @Tommatheussen

However ever since the beginning the line where it says to install it through the command "REDHAT_INSTALL="yum -y install". Had to change it into "dnf -y install" since yum has been deprecated

Anyway, will give mine (feedback) in few hours
Awesome work @mrworf, really !

@glberen
Copy link

glberen commented Jun 24, 2016

It works for me now after updating. Ubuntu 14.04. Thank you!

@sjorge
Copy link
Author

sjorge commented Jun 24, 2016

@mrworf looks good now! Using ubuntu 16.04 64-bit

@grahamrb
Copy link

Also tested on Ubuntu 16.04 64-bit, working great! Thanks for maintaining!

@Mawuena
Copy link

Mawuena commented Jun 24, 2016

Confirmed to be working on Fedora 23 Server, 64 Bit !

@sjorge
Copy link
Author

sjorge commented Jun 24, 2016

Seems to be working for everyone, marking as closed

@sjorge sjorge closed this as completed Jun 24, 2016
@mrworf
Copy link
Owner

mrworf commented Jun 25, 2016

For everyone who followed this thread, latest version posted will autodetect 32/64 bit now and also has an option -l to list possible options. Also improved error messages to avoid confusion. @Tommatheussen

@Tommatheussen
Copy link
Contributor

@mrworf Thank you, I just tested it by removing the build, distro and distro_install keys and it worked perfectly :)

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

No branches or pull requests

6 participants