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

Pull the complete version string of Erlang #916

Merged
merged 1 commit into from
Dec 1, 2016
Merged

Pull the complete version string of Erlang #916

merged 1 commit into from
Dec 1, 2016

Conversation

tas50
Copy link
Contributor

@tas50 tas50 commented Dec 1, 2016

Pull the complete version string of Erlang

Improve the shellout with help of StackOverflow and srenatus. This requires less fragile work in Ruby by formatting the data in erlang properly.

# "18"
# "7.3"
# "2.10"
# 19.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sample output now is surely 19.1,8.1,2.11 ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

true

Improve the shellout with help of StackOverflow and srenatus. This
requires less fragile work in Ruby by formatting the data in erlang
properly.

Signed-off-by: Tim Smith <tsmith@chef.io>
@tas50
Copy link
Contributor Author

tas50 commented Dec 1, 2016

@thommay Example output fixed

@tas50 tas50 merged commit 378f894 into master Dec 1, 2016
@thommay thommay added Type: Enhancement Adds new functionality. and removed Enhancement labels Jan 24, 2017
@chef chef locked and limited conversation to collaborators Nov 16, 2017
@tas50 tas50 deleted the erlang branch January 30, 2018 00:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Enhancement Adds new functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants