Skip to content

Commit

Permalink
fix(cli): ci template upgrade
Browse files Browse the repository at this point in the history
  • Loading branch information
Brooooooklyn committed Dec 9, 2022
1 parent f6ac154 commit bd1d3fe
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 20 deletions.
5 changes: 0 additions & 5 deletions cli/src/new/cargo-config.ts
Original file line number Diff line number Diff line change
@@ -1,13 +1,8 @@
export const createCargoConfig = (
enableLinuxArm7: boolean,
enableLinuxArm8Gnu: boolean,
enableLinuxArm8Musl: boolean,
) => {
const result: string[] = []
if (enableLinuxArm8Gnu) {
result.push(`[target.aarch64-unknown-linux-gnu]
linker = "aarch64-linux-gnu-gcc"`)
}
if (enableLinuxArm8Musl) {
result.push(`[target.aarch64-unknown-linux-musl]
linker = "aarch64-linux-musl-gcc"
Expand Down
13 changes: 6 additions & 7 deletions cli/src/new/ci-template.ts
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ jobs:
target: 'armv7-unknown-linux-gnueabihf'
setup: |
sudo apt-get update
sudo apt-get install gcc-arm-linux-gnueabihf g++-arm-linux-gnueabihf -y
sudo apt-get install gcc-arm-linux-gnueabihf -y
build: |
yarn build --target=armv7-unknown-linux-gnueabihf
arm-linux-gnueabihf-strip *.node
Expand Down Expand Up @@ -145,11 +145,10 @@ jobs:
target/
key: \${{ matrix.settings.target }}-cargo-\${{ matrix.settings.host }}
- name: Cache NPM dependencies
uses: actions/cache@v3
with:
path: .yarn/cache
key: npm-cache-build-\${{ matrix.settings.target }}-node@16
- uses: goto-bus-stop/setup-zig@v2
if: \${{ matrix.settings.target == 'armv7-unknown-linux-gnueabihf' }}
with:
version: 0.10.0
- name: Setup toolchain
run: \${{ matrix.settings.setup }}
Expand Down Expand Up @@ -211,7 +210,7 @@ jobs:
usesh: true
mem: 3000
prepare: |
pkg install -y curl node16 python2
pkg install -y -f curl node libnghttp2
curl -qL https://www.npmjs.com/install.sh | sh
npm install --location=global --ignore-scripts yarn
curl https://sh.rustup.rs -sSf --output rustup.sh
Expand Down
9 changes: 1 addition & 8 deletions cli/src/new/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -175,20 +175,13 @@ test('sum from native', (t) => {
join(process.cwd(), this.dirname!),
)

const enableLinuxArm8Gnu = this.targets!.includes(
'aarch64-unknown-linux-gnu',
)
const enableLinuxArm8Musl = this.targets!.includes(
'aarch64-unknown-linux-musl',
)
const enableLinuxArm7 = this.targets!.includes(
'armv7-unknown-linux-gnueabihf',
)
const cargoConfig = createCargoConfig(
enableLinuxArm7,
enableLinuxArm8Gnu,
enableLinuxArm8Musl,
)
const cargoConfig = createCargoConfig(enableLinuxArm7, enableLinuxArm8Musl)
if (cargoConfig.length) {
const configDir = join(process.cwd(), this.dirname!, '.cargo')
if (!this.dryRun) {
Expand Down

0 comments on commit bd1d3fe

Please sign in to comment.