Skip to content

Commit

Permalink
Revert "http-parser -> llhttp (#5364)"
Browse files Browse the repository at this point in the history
This reverts commit d379c6b.
  • Loading branch information
asvetlov committed Oct 27, 2021
1 parent 0fc598c commit c11a96a
Show file tree
Hide file tree
Showing 11 changed files with 282 additions and 302 deletions.
40 changes: 1 addition & 39 deletions .github/workflows/ci.yml
Expand Up @@ -71,41 +71,9 @@ jobs:
run: |
LC_ALL=C sort -c CONTRIBUTORS.txt
gen_llhttp:
name: Generate llhttp sources
needs: lint
runs-on: ubuntu-latest
timeout-minutes: 5
steps:
- name: Checkout
uses: actions/checkout@v2
with:
submodules: true
- name: Cache llhttp generated files
uses: actions/cache@v2
id: cache
with:
key: llhttp-${{ hashFiles('vendor/llhttp/package.json', 'vendor/llhttp/src/**/*') }}
path: vendor/llhttp/build
- name: Setup NodeJS
if: steps.cache.outputs.cache-hit != 'true'
uses: actions/setup-node@v2
with:
node-version: '14'
- name: Generate llhttp sources
if: steps.cache.outputs.cache-hit != 'true'
run: |
make generate-llhttp
- name: Upload llhttp generated files
uses: actions/upload-artifact@v2
with:
name: llhttp
path: vendor/llhttp/build
if-no-files-found: error

test:
name: Test
needs: gen_llhttp
needs: lint
strategy:
matrix:
pyver: [3.7, 3.8, 3.9, '3.10']
Expand Down Expand Up @@ -154,12 +122,6 @@ jobs:
if: ${{ matrix.no-extensions == '' }}
run: |
make cythonize
- name: Restore llhttp generated files
if: ${{ matrix.no-extensions == '' }}
uses: actions/download-artifact@v2
with:
name: llhttp
path: vendor/llhttp/build/
- name: Run unittests
env:
COLOR: 'yes'
Expand Down
1 change: 0 additions & 1 deletion .gitignore
Expand Up @@ -24,7 +24,6 @@
.idea
.install-cython
.install-deps
.llhttp-gen
.installed.cfg
.mypy_cache
.noseids
Expand Down
8 changes: 4 additions & 4 deletions .gitmodules
@@ -1,4 +1,4 @@
[submodule "vendor/llhttp"]
path = vendor/llhttp
url = https://github.com/nodejs/llhttp.git
branch = v3.0.0
[submodule "vendor/http-parser"]
path = vendor/http-parser
url = git://github.com/nodejs/http-parser.git
branch = 54f55a2
1 change: 0 additions & 1 deletion CHANGES/3561.feature

This file was deleted.

14 changes: 1 addition & 13 deletions Makefile
Expand Up @@ -62,15 +62,6 @@ aiohttp/_find_header.c: $(call to-hash,aiohttp/hdrs.py ./tools/gen.py)
aiohttp/%.c: aiohttp/%.pyx $(call to-hash,$(CYS)) aiohttp/_find_header.c
cython -3 -o $@ $< -I aiohttp

vendor/llhttp/node_modules: vendor/llhttp/package.json
cd vendor/llhttp; npm install

.llhttp-gen: vendor/llhttp/node_modules
$(MAKE) -C vendor/llhttp generate
@touch .llhttp-gen

.PHONY: generate-llhttp
generate-llhttp: .llhttp-gen

.PHONY: cythonize
cythonize: .install-cython $(PYXS:.pyx=.c)
Expand All @@ -90,7 +81,7 @@ fmt format:
mypy:
mypy

.develop: .install-deps generate-llhttp $(call to-hash,$(PYS) $(CYS) $(CS))
.develop: .install-deps $(call to-hash,$(PYS) $(CYS) $(CS))
pip install -e .
@touch .develop

Expand Down Expand Up @@ -146,9 +137,6 @@ clean:
@rm -rf aiohttp.egg-info
@rm -f .install-deps
@rm -f .install-cython
@rm -rf vendor/llhttp/node_modules
@rm -f .llhttp-gen
@$(MAKE) -C vendor/llhttp clean

.PHONY: doc
doc:
Expand Down

0 comments on commit c11a96a

Please sign in to comment.