Bug #7443: Method redefinition from Fiddle is causing a test failure (MinGW)

• Status: Closed
• Assignee: tenderlovemaking (Aaron Patterson)
• Target version: 2.0.0
• ruby -v: ruby 2.0.0dev (2012-11-27 trunk 37887) [i386-mingw32]

[luislavena (Luis Lavena)]

=begin

The modification introduced in r37881 associated with #5458 is causing a test failure on RubyInstaller CI:

1. Failure:
   test_modify_expand_memory_leak(Test_StringModifyExpand) [C:/Users/Worker/Jenkins/workspace/ruby-trunk-x64-build/test/-ext-/string/test_modify_expand.rb:7]:
   rb_str_modify_expand().
   <[true, ""]> expected but was
   <[true, "C:/Users/Worker/Jenkins/workspace/ruby-trunk-x64-build/build/.ext/x64-mingw32/fiddle.so: warning: method redefined; discarding old dlwrap\nC:/Users/Worker/Jenkins/workspace/ruby-trunk-x64-build/build/.ext/x64-mingw32/fiddle.so: warning: method redefined; discarding old dlwrap\n"]>

This happens on both x86 and x64:

http://ci.rubyinstaller.org/job/ruby-trunk-x86-test-all/374/console
http://ci.rubyinstaller.org/job/ruby-trunk-x64-test-all/276/console

Thank you
=end

[zzak (zzak _)]

For what it's worth, here's the issues I'm getting:
https://gist.github.com/4152220

[luislavena (Luis Lavena)]

=begin
zzak (Zachary Scott) wrote:

For what it's worth, here's the issues I'm getting:
https://gist.github.com/4152220

Thanks Zachary,

The issue is not a test from Fiddle that is failing, but Fiddle redefinition causing another test to fail.

test_modify_expand_memory_leak from test/-ext-/string/test_modify_expand.rb

I think you can run it independently doing:

make test-all TESTS="-ext-/string/test_modify_expand.rb"
=end

[zzak (zzak _)]

Luis, sorry to pollute your ticket, I was just discussing this with drbrain on irc. Ignore me :)

edit: that test passes ok for me on x86_64-linux

[zzak (zzak _)]

I should probably mention the failures I saw were due to missing libffi headers, carry on!

[luislavena (Luis Lavena)]

• % Done changed from 0 to 50

=begin
Thank you Aaron

This indeed improved the error, but now we are getting a different one:

http://ci.rubyinstaller.org/job/ruby-trunk-x64-test-all/286/console

1. Failure:
   test_modify_expand_memory_leak(Test_StringModifyExpand) [C:/Users/Worker/Jenkins/workspace/ruby-trunk-x64-build/test/-ext-/string/test_modify_expand.rb:7]:
   rb_str_modify_expand().
   <[true, ""]> expected but was
   <[true, "DL is deprecated, please use Fiddle\n"]>.

I don't understand why the child process of this particular tests will be loading DL (since is using a C extension).

Thank you.
=end

[usa (Usaku NAKAMURA)]

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

This issue was solved with changeset r38055.
Luis, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

• ChangeLog: mention to ML and ticket: [ruby-core:50194] [Bug #7443]