Skip to content

Commit

Permalink
Rewrite JS transformer and scope hoisting in Rust (#6230)
Browse files Browse the repository at this point in the history
  • Loading branch information
devongovett committed May 10, 2021
1 parent fdbcbd6 commit ba57b65
Show file tree
Hide file tree
Showing 238 changed files with 13,366 additions and 8,308 deletions.
File renamed without changes.
56 changes: 28 additions & 28 deletions .github/workflows/nightly-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,17 +19,17 @@ jobs:
toolchain: stable
profile: minimal
override: true
- name: Link native packages
run: node scripts/link-native.js
- uses: bahmutov/[email protected]
- name: Build native packages
run: yarn build-native-release
- name: Strip debug symbols # https://github.com/rust-lang/rust/issues/46034
if: ${{ matrix.os == 'macos-latest' }}
run: strip -x native-packages/*/*.node # Must use -x on macOS. This produces larger results on linux.
run: strip -x packages/*/*/*.node # Must use -x on macOS. This produces larger results on linux.
- name: Upload artifacts
uses: actions/upload-artifact@v2
with:
name: bindings-${{ matrix.os }}
path: native-packages/*/*.node
path: packages/*/*/*.node
- name: Smoke test
run: node -e "require('@parcel/fs-search')"

Expand All @@ -48,18 +48,18 @@ jobs:
toolchain: stable
profile: minimal
override: true
- name: Link native packages
run: node scripts/link-native.js
- uses: bahmutov/[email protected]
- name: Build native packages
run: yarn build-native-release
- name: Strip debug symbols # https://github.com/rust-lang/rust/issues/46034
run: strip native-packages/*/*.node
run: strip packages/*/*/*.node
- name: Upload artifacts
uses: actions/upload-artifact@v2
with:
name: bindings-linux-gnu-x64
path: native-packages/*/*.node
path: packages/*/*/*.node
- name: debug
run: ls -l native-packages/*/*.node
run: ls -l packages/*/*/*.node
- name: Smoke test
run: node -e 'require("@parcel/fs-search")'

Expand Down Expand Up @@ -89,20 +89,20 @@ jobs:
run: |
sudo apt-get update
sudo apt-get install gcc-arm-linux-gnueabihf gcc-aarch64-linux-gnu g++-aarch64-linux-gnu -y
- name: Link native packages
run: node scripts/link-native.js
- uses: bahmutov/[email protected]
- name: Build native packages
run: yarn build-native-release
env:
RUST_TARGET: ${{ matrix.target }}
- name: Strip debug symbols # https://github.com/rust-lang/rust/issues/46034
run: ${{ matrix.strip }} native-packages/*/*.node
run: ${{ matrix.strip }} packages/*/*/*.node
- name: Upload artifacts
uses: actions/upload-artifact@v2
with:
name: bindings-${{ matrix.target }}
path: native-packages/*/*.node
path: packages/*/*/*.node
- name: debug
run: ls -l native-packages/*/*.node
run: ls -l packages/*/*/*.node
- name: Configure binfmt-support
run: docker run --rm --privileged multiarch/qemu-user-static:register --reset
- name: Smoke test
Expand Down Expand Up @@ -143,20 +143,20 @@ jobs:
curl -O https://musl.cc/aarch64-linux-musl-cross.tgz
tar xzf aarch64-linux-musl-cross.tgz
cp -R aarch64-linux-musl-cross/* /usr
- name: Link native packages
run: node scripts/link-native.js
- uses: bahmutov/[email protected]
- name: Build native packages
run: yarn build-native-release
env:
RUST_TARGET: ${{ matrix.target }}
- name: Strip debug symbols # https://github.com/rust-lang/rust/issues/46034
run: ${{ matrix.strip }} native-packages/*/*.node
run: ${{ matrix.strip }} packages/*/*/*.node
- name: Upload artifacts
uses: actions/upload-artifact@v2
with:
name: bindings-linux-musl
path: native-packages/*/*.node
path: packages/*/*/*.node
- name: debug
run: ls -l native-packages/*/*.node
run: ls -l packages/*/*/*.node
- name: Smoke test
if: ${{ matrix.target == 'x86_64-unknown-linux-musl' }}
run: node -e 'require("@parcel/fs-search")'
Expand All @@ -173,20 +173,20 @@ jobs:
profile: minimal
override: true
target: aarch64-apple-darwin
- name: Link native packages
run: node scripts/link-native.js
- uses: bahmutov/[email protected]
- name: Build native packages
run: yarn build-native-release
env:
RUST_TARGET: aarch64-apple-darwin
- name: Strip debug symbols
run: strip -x native-packages/*/*.node
run: strip -x packages/*/*/*.node
- name: Upload artifacts
uses: actions/upload-artifact@v2
with:
name: bindings-apple-aarch64
path: native-packages/*/*.node
path: packages/*/*/*.node
- name: debug
run: ls -l native-packages/*/*.node
run: ls -l packages/*/*/*.node

build-and-release:
runs-on: ubuntu-latest
Expand All @@ -198,17 +198,17 @@ jobs:
- build-apple-silicon
steps:
- uses: actions/checkout@v1
- name: Link native packages
run: node scripts/link-native.js
- uses: bahmutov/[email protected]
- name: Build native packages
run: yarn build-native-release
- name: Download artifacts
uses: actions/download-artifact@v2
with:
path: artifacts
- name: Move artifacts
run: for d in artifacts/*/*; do cp $d/*.node native-packages/$(basename $d); done
run: for d in artifacts/*/*/*; do cp $d/*.node packages/$(basename $(dirname $d))/$(basename $d); done
- name: Debug
run: ls -l native-packages/*/*
run: ls -l packages/*/*/*.node
- run: echo //registry.npmjs.org/:_authToken=${NPM_TOKEN} > .npmrc
env:
NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
Expand Down
56 changes: 28 additions & 28 deletions .github/workflows/tag-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,17 +18,17 @@ jobs:
toolchain: stable
profile: minimal
override: true
- name: Link native packages
run: node scripts/link-native.js
- uses: bahmutov/[email protected]
- name: Build native packages
run: yarn build-native-release
- name: Strip debug symbols # https://github.com/rust-lang/rust/issues/46034
if: ${{ matrix.os == 'macos-latest' }}
run: strip -x native-packages/*/*.node # Must use -x on macOS. This produces larger results on linux.
run: strip -x packages/*/*/*.node # Must use -x on macOS. This produces larger results on linux.
- name: Upload artifacts
uses: actions/upload-artifact@v2
with:
name: bindings-${{ matrix.os }}
path: native-packages/*/*.node
path: packages/*/*/*.node
- name: Smoke test
run: node -e "require('@parcel/fs-search')"

Expand All @@ -47,18 +47,18 @@ jobs:
toolchain: stable
profile: minimal
override: true
- name: Link native packages
run: node scripts/link-native.js
- uses: bahmutov/[email protected]
- name: Build native packages
run: yarn build-native-release
- name: Strip debug symbols # https://github.com/rust-lang/rust/issues/46034
run: strip native-packages/*/*.node
run: strip packages/*/*/*.node
- name: Upload artifacts
uses: actions/upload-artifact@v2
with:
name: bindings-linux-gnu-x64
path: native-packages/*/*.node
path: packages/*/*/*.node
- name: debug
run: ls -l native-packages/*/*.node
run: ls -l packages/*/*/*.node
- name: Smoke test
run: node -e 'require("@parcel/fs-search")'

Expand Down Expand Up @@ -88,20 +88,20 @@ jobs:
run: |
sudo apt-get update
sudo apt-get install gcc-arm-linux-gnueabihf gcc-aarch64-linux-gnu g++-aarch64-linux-gnu -y
- name: Link native packages
run: node scripts/link-native.js
- uses: bahmutov/[email protected]
- name: Build native packages
run: yarn build-native-release
env:
RUST_TARGET: ${{ matrix.target }}
- name: Strip debug symbols # https://github.com/rust-lang/rust/issues/46034
run: ${{ matrix.strip }} native-packages/*/*.node
run: ${{ matrix.strip }} packages/*/*/*.node
- name: Upload artifacts
uses: actions/upload-artifact@v2
with:
name: bindings-${{ matrix.target }}
path: native-packages/*/*.node
path: packages/*/*/*.node
- name: debug
run: ls -l native-packages/*/*.node
run: ls -l packages/*/*/*.node
- name: Configure binfmt-support
run: docker run --rm --privileged multiarch/qemu-user-static:register --reset
- name: Smoke test
Expand Down Expand Up @@ -142,20 +142,20 @@ jobs:
curl -O https://musl.cc/aarch64-linux-musl-cross.tgz
tar xzf aarch64-linux-musl-cross.tgz
cp -R aarch64-linux-musl-cross/* /usr
- name: Link native packages
run: node scripts/link-native.js
- uses: bahmutov/[email protected]
- name: Build native packages
run: yarn build-native-release
env:
RUST_TARGET: ${{ matrix.target }}
- name: Strip debug symbols # https://github.com/rust-lang/rust/issues/46034
run: ${{ matrix.strip }} native-packages/*/*.node
run: ${{ matrix.strip }} packages/*/*/*.node
- name: Upload artifacts
uses: actions/upload-artifact@v2
with:
name: bindings-linux-musl
path: native-packages/*/*.node
path: packages/*/*/*.node
- name: debug
run: ls -l native-packages/*/*.node
run: ls -l packages/*/*/*.node
- name: Smoke test
if: ${{ matrix.target == 'x86_64-unknown-linux-musl' }}
run: node -e 'require("@parcel/fs-search")'
Expand All @@ -172,20 +172,20 @@ jobs:
profile: minimal
override: true
target: aarch64-apple-darwin
- name: Link native packages
run: node scripts/link-native.js
- uses: bahmutov/[email protected]
- name: Build native packages
run: yarn build-native-release
env:
RUST_TARGET: aarch64-apple-darwin
- name: Strip debug symbols
run: strip -x native-packages/*/*.node
run: strip -x packages/*/*/*.node
- name: Upload artifacts
uses: actions/upload-artifact@v2
with:
name: bindings-apple-aarch64
path: native-packages/*/*.node
path: packages/*/*/*.node
- name: debug
run: ls -l native-packages/*/*.node
run: ls -l packages/*/*/*.node

build-and-release:
runs-on: ubuntu-latest
Expand All @@ -197,17 +197,17 @@ jobs:
- build-apple-silicon
steps:
- uses: actions/checkout@v1
- name: Link native packages
run: node scripts/link-native.js
- uses: bahmutov/[email protected]
- name: Build native packages
run: yarn build-native-release
- name: Download artifacts
uses: actions/download-artifact@v2
with:
path: artifacts
- name: Move artifacts
run: for d in artifacts/*/*; do cp $d/*.node native-packages/$(basename $d); done
run: for d in artifacts/*/*/*; do cp $d/*.node packages/$(basename $(dirname $d))/$(basename $d); done
- name: Debug
run: ls -l native-packages/*/*
run: ls -l packages/*/*/*.node
- run: echo //registry.npmjs.org/:_authToken=${NPM_TOKEN} > .npmrc
env:
NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
Expand Down
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -21,4 +21,3 @@ tmp
parcel-bundle-reports/
.verdaccio_storage/
sourcemap-info.json
packages/native-packages
1 change: 1 addition & 0 deletions .prettierrc
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
{
"files": [
"packages/packagers/js/src/prelude.js",
"packages/packagers/swc/src/dev-prelude.js",
"packages/shared/scope-hoisting/src/helpers.js"
],
"options": {
Expand Down
Loading

0 comments on commit ba57b65

Please sign in to comment.