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

rsync': undefined method success?' for nil:NilClass #34

Closed
edwinvdgraaf opened this issue Jun 22, 2015 · 5 comments
Closed

rsync': undefined method success?' for nil:NilClass #34

edwinvdgraaf opened this issue Jun 22, 2015 · 5 comments

Comments

@edwinvdgraaf
Copy link
Contributor

On line: https://github.com/DigitPaint/roger/blob/master/lib/roger/release/finalizers/rsync.rb#L68

CodeClimate coverage
https://codeclimate.com/github/DigitPaint/roger/coverage/5587d85dd86bab7e2601da4d

How is the $CHILD_STATUS set?

@flurin
Copy link
Contributor

flurin commented Jun 22, 2015

Hummm. That's supposed to be a magic global from ruby. I think the rubocop autocorrector did something to this

On 22 jun. 2015, at 13:45, Edwin van der Graaf notifications@github.com wrote:

On line: https://github.com/DigitPaint/roger/blob/master/lib/roger/release/finalizers/rsync.rb#L68

CodeClimate coverage
https://codeclimate.com/github/DigitPaint/roger/coverage/5587d85dd86bab7e2601da4d

How is the $CHILD_STATUS set?


Reply to this email directly or view it on GitHub.

@edwinvdgraaf
Copy link
Contributor Author

Aye.

306a84b#diff-d12e35b68c9b91f475a6ab3ad7d3b30aL55

-        unless $?.success?
-          raise RuntimeError, "Rsync failed.\noutput:\n #{output}"
-        end
+        fail "Rsync failed.\noutput:\n #{output}" unless $CHILD_STATUS.success?

@flurin
Copy link
Contributor

flurin commented Jun 24, 2015

What ruby version does this happen? I tested it in ruby-1.9.3, ruby-2.2.0 and there doesn't seem to be an issue

@flurin
Copy link
Contributor

flurin commented Jun 24, 2015

Can you see if the tests in PR #37 branch also break for you?

@flurin
Copy link
Contributor

flurin commented Jun 24, 2015

Fixed in #37

@flurin flurin closed this as completed Jun 24, 2015
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