$CHILD_STATUS is nil in some Ruby versions without require "English" about cocoapods-downloader HOT 5 CLOSED

@fabiopelosin are you sure .to_i is required? A big downside to that approach is that if the symbol is nil (eg. when $CHILD_STATUS does not exist as a variable) then .to_i will return zero and thus indicate success.

§ ruby -e "print nil.to_i == 0 ? 'yes' : 'no'"
yes
§ ruby -e 'print $FLUBBER.to_i == 0 ? "yes" : "no"'
yes

If you do need to force to integer for comparison you should probably check for nil first also. I know

from cocoapods-downloader.

@aufflick Thanks... I added the .to_i expecting to raise a no method error in case of nil... actually I've read about this behavior but it was completely lost in the inners of my mind.

I will fix it... thanks!

from cocoapods-downloader.

@fabiopelosin I don’t particularly like them, but can we just use the default Ruby globals instead? I.e. $? in this situation?

from cocoapods-downloader.

@alloy I was planning to go back to $? as long as we support Ruby 1.8.7 because it is just not worth it (working on another gem I have disabled this check on the cocoapods configuration of RuboCop exactly for this reason).

from cocoapods-downloader.

👍

from cocoapods-downloader.