Commit 3816a97
doc: add detail for how to update llhttp
Add detail on how to update llhttp in deps to the
guide for maintaining http.
Signed-off-by: Michael Dawson <mdawson@devrus.com>
PR-URL: #43028
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent 8e54c19 commit 3816a97
1 file changed
Lines changed: 28 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -74,9 +74,35 @@ to align them with the APIs built for the client. | |||
| 74 | 74 | The low-level implementation of the | |
| 75 | 75 | [HTTP](https://nodejs.org/docs/latest/api/http.html) | |
| 76 | 76 | and [HTTPS](https://nodejs.org/docs/latest/api/https.html) APIs | |
| 77 | - are maintained in the [llttp](https://github.com/nodejs/llhttp) | ||
| 77 | + are maintained in the [llhttp](https://github.com/nodejs/llhttp) | ||
| 78 | 78 | repository. Updates are pulled into Node.js under | |
| 79 | - [deps/llhttp](https://github.com/nodejs/node/tree/HEAD/deps/llhttp) | ||
| 79 | + [deps/llhttp](https://github.com/nodejs/node/tree/HEAD/deps/llhttp). | ||
| 80 | + | ||
| 81 | + In order to update Node.js with a new version of llhttp: | ||
| 82 | + | ||
| 83 | + * check out the tagged release that you want to update to (a release | ||
| 84 | + should be created in the llhttp repo before updating Node.js). | ||
| 85 | + * run `npm install` in the directory that you checked out llhttp. | ||
| 86 | + * run `make release` in the directory that you checked out llhttp. | ||
| 87 | + * copy the contents of the `release` directory from the directory you | ||
| 88 | + checked llhttp out to | ||
| 89 | + [deps/llhttp](https://github.com/nodejs/node/tree/HEAD/deps/llhttp) | ||
| 90 | + | ||
| 91 | + It should look like the following: | ||
| 92 | + | ||
| 93 | + ```console | ||
| 94 | + ├── CMakeLists.txt | ||
| 95 | + ├── common.gypi | ||
| 96 | + ├── include | ||
| 97 | + │ └── llhttp.h | ||
| 98 | + ├── LICENSE-MIT | ||
| 99 | + ├── llhttp.gyp | ||
| 100 | + ├── README.md | ||
| 101 | + └── src | ||
| 102 | + ├── api.c | ||
| 103 | + ├── http.c | ||
| 104 | + └── llhttp.c | ||
| 105 | + ``` | ||
| 80 | 106 | ||
| 81 | 107 | The low-level implementation is made available in the Node.js API through | |
| 82 | 108 | JavaScript code in the [lib](https://github.com/nodejs/node/tree/HEAD/lib) | |
0 commit comments