Issue #14386 has been updated by k0kubun (Takashi Kokubun).
Status changed from Open to Closed
Applied in changeset r62025.
For rosenfeld's suggestion, open3 would be a good place to have the functionality.
----------------------------------------
Feature #14386: Add option to let Kernel.#system raise error instead of returning false
https://bugs.ruby-lang.org/issues/14386#change-69765
* Author: k0kubun (Takashi Kokubun)
* Status: Closed
* Priority: Normal
* Assignee:
* Target version:
----------------------------------------
I sometimes write code like:
~~~ ruby
system('git pull origin master') || raise('Failed to execute: git pull origin master')
system('bundle check || bundle install') || raise('Failed to execute: bundle check || bundle install')
~~~
Using rake, we can simplify the above code to the following one, but there's no way to do that outside Rakefile. (Note that I just want to do the same thing as "bash -e", the error message is actually not important.)
~~~ ruby
sh 'git pull origin master'
sh 'bundle check || bundle install'
~~~
If we add the following option, we can simplify such code even when we're not on a rake task. I'm not sure whether it's a good name or not though.
~~~ ruby
system 'git pull origin master', assert_status: true
system 'bundle check || bundle install', assert_status: true
# => RuntimeError: Command failed with status (1): bundle check || bundle install
~~~
--
https://bugs.ruby-lang.org/
Unsubscribe: <mailto:ruby-core-request / ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>