Skip to content

Fix VERSION case error#64

Merged
sparklyballs merged 1 commit intomasterfrom
zaggash-variable_case
Oct 17, 2016
Merged

Fix VERSION case error#64
sparklyballs merged 1 commit intomasterfrom
zaggash-variable_case

Conversation

@zaggash
Copy link
Copy Markdown
Contributor

@zaggash zaggash commented Oct 17, 2016

linuxserver.io

Edit file https://github.com/linuxserver/docker-plex/blob/master/root/etc/cont-init.d/50-plex-update

Thanks, team linuxserver.io

@sparklyballs sparklyballs merged commit 9bbf1b3 into master Oct 17, 2016
@sparklyballs sparklyballs deleted the zaggash-variable_case branch October 17, 2016 19:10
@ljm42
Copy link
Copy Markdown
Contributor

ljm42 commented Oct 18, 2016

Is this going to cause issues when passing in a specific version such as "1.1.4.2757-24ffd60", which contains lowercase letters?

@sparklyballs
Copy link
Copy Markdown
Contributor

no, it trims uppercase to lowercase.

@ljm42
Copy link
Copy Markdown
Contributor

ljm42 commented Oct 18, 2016

wow, I read that completely backwards :) looks good, thanks

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

Successfully merging this pull request may close these issues.

Automatic updates failing

3 participants