Skip to content

Commit

Permalink
Use clean installs from MPN (#298)
Browse files Browse the repository at this point in the history
  • Loading branch information
steverep authored Feb 15, 2024
1 parent bbc760d commit 4858809
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/aiohttp.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ jobs:
key: ubuntu-latest-node-${{ hashFiles('vendor/llhttp/**/package-lock.json') }}
restore-keys: ubuntu-latest-node-
- name: Install llhttp dependencies
run: npm install --ignore-scripts
run: npm ci --ignore-scripts
working-directory: vendor/llhttp
- name: Build llhttp
run: make
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ jobs:
${{ runner.os }}-node-
- name: Install dependencies
run: npm install --ignore-scripts
run: npm ci --ignore-scripts

- name: Build libllhttp.a
shell: bash
Expand Down Expand Up @@ -88,7 +88,7 @@ jobs:
${{ runner.os }}-node-
- name: Install dependencies
run: npm install --ignore-scripts
run: npm ci --ignore-scripts

# Custom script, because progress looks not good in CI
- name: Run tests
Expand All @@ -112,7 +112,7 @@ jobs:
${{ runner.os }}-node-
- name: Install dependencies
run: npm install --ignore-scripts
run: npm ci --ignore-scripts

- name: Run lint command
run: npm run lint
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -397,7 +397,7 @@ With this flag this check is disabled.
Make sure you have [Node.js](https://nodejs.org/), npm and npx installed. Then under project directory run:
```sh
npm install
npm ci
make
```

Expand Down Expand Up @@ -451,7 +451,7 @@ _Note that using the git repo directly (e.g., via a git repo url and tag) will n

1. Ensure that `Clang` and `make` are in your system path.
2. Using Git Bash, clone the repo to your preferred location.
3. Cd into the cloned directory and run `npm install`
3. Cd into the cloned directory and run `npm ci`
5. Run `make`
6. Your `repo/build` directory should now have `libllhttp.a` and `libllhttp.so` static and dynamic libraries.
7. When building your executable, you can link to these libraries. Make sure to set the build folder as an include path when building so you can reference the declarations in `repo/build/llhttp.h`.
Expand Down

0 comments on commit 4858809

Please sign in to comment.