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

getJavaVersion has dangling \r #4

Open
brunerd opened this issue Mar 25, 2015 · 1 comment
Open

getJavaVersion has dangling \r #4

brunerd opened this issue Mar 25, 2015 · 1 comment

Comments

@brunerd
Copy link

brunerd commented Mar 25, 2015

There's a \r at the end of currVersBase from the getJavaVersion, so the compare fails and will bug you in Self Service mode

Added a sed item to to kill \r: -e "s/"$'\r'"//g"

So it is now:
currVersBase=$( curl -sfA "${UAGENT}" "${URL}" 2>/dev/null | awk -F'[>|<]' '/Version/{print}' | sed -e 's/Version //;s/ Update /./' -e "s/"$'\r'"//g")

@mm2270
Copy link
Owner

mm2270 commented Apr 4, 2015

Hi brunerd.

Thanks. I had seen that extra carriage return myself, and, at one point had fixed it (with a perl snippet, but sed works just as well), but my bad, I forgot to add that update into the script posted here. I'll be sure to update it with the correct code.

I'm working on a new version of the script with more features so a lot of my focus has been on that script, which is probably why I forgot to add it in to this version. At least that's my story and I'm sticking to it :)

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