Bug #6092

tests of psych reports many errors and failures when external encoding is not unicode

Added by Usaku NAKAMURA about 3 years ago. Updated about 3 years ago.

[ruby-core:42967]
Status:Closed
Priority:Normal
Assignee:Aaron Patterson
ruby -v:ruby 2.0.0dev (2012-02-27 trunk 34828) [x64-mswin64_100] Backport:

Description

after r34722, tests of psych reports many errors and failures, such as;

test_parse_file_exception(Psych::TestException) [C:/Users/usa/ruby/test/psych/test_exception.rb:63]:
[Psych::SyntaxError] exception expected, not
Class:
Message: <"YAML file must be UTF-8, UTF-16LE, or UTF-16BE, not Windows-31J">
---Backtrace---
C:/Users/usa/ruby/.ext/common/psych.rb:192:in parse'
C:/Users/usa/ruby/.ext/common/psych.rb:192:in
parse_stream'
C:/Users/usa/ruby/.ext/common/psych.rb:150:in parse'
C:/Users/usa/ruby/.ext/common/psych.rb:160:in
block in parse_file'
C:/Users/usa/ruby/.ext/common/psych.rb:159:in open'
C:/Users/usa/ruby/.ext/common/psych.rb:159:in
parse_file'

C:/Users/usa/ruby/test/psych/test_exception.rb:64:in `block in test_parse_file_exception'

and

test_scalar_tag(Psych::TestIOTainted):
ArgumentError: YAML file must be UTF-8, UTF-16LE, or UTF-16BE, not Windows-31J
C:/Users/usa/ruby/test/psych/test_tainted.rb:125:in parse'
C:/Users/usa/ruby/test/psych/test_tainted.rb:125:in
block in assert_taintedness'
C:/Users/usa/ruby/test/psych/test_tainted.rb:124:in open'
C:/Users/usa/ruby/test/psych/test_tainted.rb:124:in
assert_taintedness'
C:/Users/usa/ruby/test/psych/test_tainted.rb:72:in `test_scalar_tag'

, etc.

History

#1 Updated by Jon Forums about 3 years ago

I also see this on my [i386-mingw] and i686-linux systems.

A quick workaround for mingw is chcp 65001 in cmd.exe before starting sh to run make test-all, and for Arch, set LOCALE="en_US.utf8" (not en_US.iso88591) in /etc/rc.conf

#2 Updated by Aaron Patterson about 3 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 0 to 100

I fixed this in r34839, but it seems redmine isn't closing the ticket automatically.

Also available in: Atom PDF