Skip to content

Commit 41e8f14

Browse files
committed
Remove the Bash launcher for JRuby on Windows, always use jruby.exe
* The Bash launcher causes warnings, see ruby/setup-ruby#13 (comment) * Add tests when using an absolute path to jruby.
1 parent ba5351f commit 41e8f14

File tree

1 file changed

+14
-2
lines changed

1 file changed

+14
-2
lines changed

.github/workflows/build.yml

+14-2
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ jobs:
6565
asset_content_type: application/gzip
6666

6767
buildJRubyWindows:
68-
if: false
68+
if: true
6969
strategy:
7070
fail-fast: false
7171
matrix:
@@ -92,6 +92,8 @@ jobs:
9292
mkdir ~/.rubies
9393
tar xf jruby-dist-${{ matrix.jruby-version }}-bin.tar.gz -C ~/.rubies
9494
cd ~/.rubies/${{ matrix.ruby }}/bin
95+
# Remove the Bash launcher, always use jruby.exe
96+
rm jruby
9597
echo -en "@ECHO OFF\r\n@\"%~dp0jruby.exe\" %*\r\n" > ruby.bat
9698
- name: Install Bundler if needed
9799
shell: bash
@@ -108,6 +110,16 @@ jobs:
108110
- name: Bundler version
109111
run: ~/.rubies/${{ matrix.ruby }}/bin/ruby -S bundle --version
110112

113+
- name: Basic test (Bash)
114+
run: ~/.rubies/${{ matrix.ruby }}/bin/jruby --version
115+
shell: bash
116+
- name: OpenSSL test (Bash)
117+
run: ~/.rubies/${{ matrix.ruby }}/bin/jruby -ropen-uri -e 'puts open(%{https://rubygems.org/}) { |f| f.read(1024) }'
118+
shell: bash
119+
- name: Bundler version (Bash)
120+
run: ~/.rubies/${{ matrix.ruby }}/bin/jruby -S bundle --version
121+
shell: bash
122+
111123
- name: Create archive
112124
run: tar czf ${{ matrix.ruby }}-${{ matrix.os }}.tar.gz -C ~/.rubies ${{ matrix.ruby }}
113125
shell: bash
@@ -187,7 +199,7 @@ jobs:
187199
asset_content_type: application/gzip
188200

189201
buildRubinius:
190-
if: true
202+
if: false
191203
strategy:
192204
fail-fast: false
193205
matrix:

0 commit comments

Comments
 (0)