Project

General

Profile

Actions

Bug #15035

closed

Windows make install failures

Added by MSP-Greg (Greg L) over 5 years ago. Updated over 5 years ago.

Status:
Closed
Target version:
-
ruby -v:
ruby 2.6.0dev (2018-08-28 trunk 64578) [x64-mingw32]
[ruby-core:88699]

Description

@hsbt (Hiroshi SHIBATA)

Windows has been failing on install, seems due to a combination of RubyGems changes.

ruby-loco is passing with the patch, and the ruby/ruby mswin is completing make install (but failing on some recent spec changes). See GH PR 1942 (https://github.com/ruby/ruby/pull/1942).

Patch attached.

Thanks, Greg


Files

make_install_rg.patch (1.81 KB) make_install_rg.patch MSP-Greg (Greg L), 08/28/2018 12:38 AM
Actions #1

Updated by hsbt (Hiroshi SHIBATA) over 5 years ago

  • Status changed from Open to Closed

Applied in changeset trunk|r64582.


Fixed test failures in mswin environment at r64555.

[ruby-core:88699][Bug #15035]
This patch was provided by MSP-Greg.

Updated by MSP-Greg (Greg L) over 5 years ago

@hsbt (Hiroshi SHIBATA)

Thanks for applying the patch. I should have been clearer, as the patch fixes two issues, both involving RubyGems, but one of the patched files (tool/rbinstall.rb) is patched for the parameter passed to a RubyGems method. That fix is what removes the error at line 30801 of the Appveyor log, https://ci.appveyor.com/project/ruby/ruby/build/9490/job/awckr1if158iu9b3#L3081

In the original PR (https://github.com/ruby/ruby/pull/1942), I had some typos that I didn't see testing locally on mingw, once fixed, Travis passed.

With the patches from the PR, ruby-loco passed at ruby 2.6.0dev (2018-08-28 trunk 64581) [x64-mingw32], and mswin passed on my fork at https://ci.appveyor.com/project/MSP-Greg/ruby/build/8, but I'm not sure what rev/commit that was at, as the mswin build script does not show the revision number.

I think mswin may be locking on test-all from some of the other recent commits. ruby-loco just passed again at r64582.

I realize the spec patch is probably better left for @Eregon (Benoit Daloze) to review.

Thanks, Greg

Updated by hsbt (Hiroshi SHIBATA) over 5 years ago

@MSP-Greg (Greg L)

Thanks for your patches. I will merge a patch of tool/rbinstall.rb after confirming with my windows environment.

Updated by hsbt (Hiroshi SHIBATA) over 5 years ago

  • Assignee set to hsbt (Hiroshi SHIBATA)
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0