Skip to content

Commit

Permalink
Install jco via npm instead of git submodule
Browse files Browse the repository at this point in the history
  • Loading branch information
kateinoigakukun committed Sep 4, 2024
1 parent 0b031b2 commit f3e6dd4
Show file tree
Hide file tree
Showing 8 changed files with 2,829 additions and 7,345 deletions.
1 change: 0 additions & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,6 @@ jobs:
chmod +x ./build-exec
# wait for docker exec to be ready
timeout 10 bash -c 'until ./build-exec bundle config set --local without check; do sleep 1; done'
- run: ./build-exec rustup target add wasm32-wasip1 # For jco build
- run: ./build-exec ./bin/setup
- run: ./build-exec bundle exec rake compile
- name: Pre-release configuration
Expand Down
3 changes: 0 additions & 3 deletions .gitmodules

This file was deleted.

2 changes: 0 additions & 2 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,6 @@ This document describes development setup and pointers for diving into this proj
$ git clone https://github.com/ruby/ruby.wasm --recursive
$ cd ruby.wasm
$ ./bin/setup
# Just for building vendor/jco (will be removed soon)
$ rustup target add wasm32-wasi
# Compile extension library
$ bundle exec rake compile
$ rake --tasks
Expand Down
7 changes: 0 additions & 7 deletions bin/setup
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,4 @@ root="$(cd "$(dirname "$0")/.." && pwd)"

env BUNDLE_GEMFILE="$root/Gemfile" bundle install

# Build vendored jco if Rust toolchain is available and submodule is checked out
if command -v rustc && [ -f vendor/jco/package.json ]; then
npm -C vendor/jco install
npm -C vendor/jco run build
else
echo "Rust toolchain not found or jco submodule not checked out, skipping jco build"
fi
npm ci
Loading

0 comments on commit f3e6dd4

Please sign in to comment.