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

node.js version check doesn't work #18

Closed
alexanderdean opened this issue Mar 17, 2014 · 5 comments
Closed

node.js version check doesn't work #18

alexanderdean opened this issue Mar 17, 2014 · 5 comments
Assignees
Labels

Comments

@alexanderdean
Copy link
Member

TASK: [nodejs | Node.js | Checking installed version of node.js] **************
failed: [127.0.0.1] => {"changed": true, "cmd": "/usr/bin/test "$(node -v 2> /dev/null)" = v0.10.25 ", "delta": "0:00:00.003899", "end": "2014-03-17 11:08:12.214885", "item": "", "rc": 1, "start": "2014-03-17 11:08:12.210986"}
...ignoring

@alexanderdean
Copy link
Member Author

One for Fred

@fblundun
Copy link
Contributor

I thought this was intentional - the idea being that if the check fails then node is fetched and installed?

@alexanderdean
Copy link
Member Author

The task literally fails - meaning crash not non-zero result code. There's something non-robust in the check syntax, it needs playing around with on the command-line

@fblundun
Copy link
Contributor

It looks like tasks are reported as failed whenever the result code is non-zero - for example,

shell: /bin/false

So I've added a failed_when command so that the task is only reported as failed if the result code is > 1.

@alexanderdean
Copy link
Member Author

Nice, thanks Fred...

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

3 participants