Skip to content

Commit

Permalink
Merge pull request #133 from eregon/truffleruby-23.1
Browse files Browse the repository at this point in the history
Updates for TruffleRuby 23.1
  • Loading branch information
byroot authored Sep 19, 2023
2 parents 0b9901e + 63ec825 commit dfb4e35
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ jobs:
strategy:
fail-fast: false
matrix:
ruby: [ '3.2', '3.1', '3.0', '2.7', '2.6', '2.5', 'jruby', 'truffleruby', 'truffleruby+graalvm-22.3.1' ]
ruby: [ '3.2', '3.1', '3.0', '2.7', '2.6', '2.5', 'jruby', 'truffleruby', 'truffleruby+graalvm' ]
runs-on: ubuntu-latest
steps:
- name: Checkout
Expand All @@ -32,8 +32,8 @@ jobs:
- name: Set TRUFFLERUBYOPT
run: echo "TRUFFLERUBYOPT=--jvm --polyglot" >> $GITHUB_ENV
if: startsWith(matrix.ruby, 'truffleruby+graalvm')
- name: Install GraalVM js component
run: if ! gu list | grep '^js '; then gu install js; fi
- name: Install GraalJS
run: truffleruby-polyglot-get js
if: startsWith(matrix.ruby, 'truffleruby+graalvm')

- name: Run test
Expand Down
3 changes: 1 addition & 2 deletions lib/execjs/graaljs_runtime.rb
Original file line number Diff line number Diff line change
Expand Up @@ -136,8 +136,7 @@ def available?

unless Polyglot.languages.include? "js"
warn "The language 'js' is not available, you likely need to `export TRUFFLERUBYOPT='--jvm --polyglot'`", uplevel: 0 if $VERBOSE
warn "You also need to install the 'js' component with 'gu install js' on GraalVM 22.2+", uplevel: 0 if $VERBOSE
warn "Note that you need TruffleRuby+GraalVM and not just the TruffleRuby standalone to use #{self.class}", uplevel: 0 if $VERBOSE
warn "You also need to install the 'js' component, see https://github.com/oracle/truffleruby/blob/master/doc/user/polyglot.md#installing-other-languages", uplevel: 0 if $VERBOSE
return @available = false
end

Expand Down

0 comments on commit dfb4e35

Please sign in to comment.