diff options
| author | Alan Wu <XrXr@users.noreply.github.com> | 2022-04-04 14:10:29 -0400 |
|---|---|---|
| committer | Alan Wu <XrXr@users.noreply.github.com> | 2022-04-04 14:10:29 -0400 |
| commit | 20a633028eeba401b3148b33f53df8af1aaa5d19 (patch) | |
| tree | 9d54e1515b68bb3833dc254796edca17b9010086 /ext | |
| parent | ca1b131b3ee5c649959e8f8c856487f795e3b28f (diff) | |
| download | psych-20a633028eeba401b3148b33f53df8af1aaa5d19.tar.gz | |
Improve libyaml source downloading error messages
People trying to build CRuby by following the instructions in its
[README] have been running into [errors] due to missing `libyaml`
on their system. Let's try to present a better error message when
it happens.
[README]: https://github.com/ruby/ruby/tree/fb5aa31e2d20ea8e1425432672f4de4c8ca2c26b#how-to-compile-and-install
[errors]: https://github.com/ruby/psych/issues/552
Diffstat (limited to 'ext')
| -rw-r--r-- | ext/psych/extconf.rb | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/ext/psych/extconf.rb b/ext/psych/extconf.rb index 795a050..b9cd120 100644 --- a/ext/psych/extconf.rb +++ b/ext/psych/extconf.rb @@ -19,10 +19,16 @@ if yaml_source == true # search the latest libyaml source under $srcdir yaml_source = Dir.glob("#{$srcdir}/yaml{,-*}/").max_by {|n| File.basename(n).scan(/\d+/).map(&:to_i)} unless yaml_source - require_relative '../../tool/extlibs.rb' - extlibs = ExtLibs.new(cache_dir: File.expand_path("../../tmp/download_cache", $srcdir)) - unless extlibs.process_under($srcdir) - raise "failed to download libyaml source" + download_failure = "failed to download libyaml source" + begin + require_relative '../../tool/extlibs.rb' + extlibs = ExtLibs.new(cache_dir: File.expand_path("../../tmp/download_cache", $srcdir)) + unless extlibs.process_under($srcdir) + raise download_failure + end + rescue + # Implicitly captures Exception#cause. Newer rubies show it in the backtrace. + raise download_failure end yaml_source, = Dir.glob("#{$srcdir}/yaml-*/") raise "libyaml not found" unless yaml_source |