Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set up py3-only tools/quic and its venv #23075

Merged
merged 3 commits into from
Apr 21, 2020
Merged

Set up py3-only tools/quic and its venv #23075

merged 3 commits into from
Apr 21, 2020

Conversation

Hexcles
Copy link
Member

@Hexcles Hexcles commented Apr 17, 2020

This does not include the actual server, but only sets up the skeleton:

  • Introduce a "py3only" to commands.json
  • Create the skeleton of tools/quic and set up its venv
  • git subtree add --prefix tools/third_party/aioquic https://github.com/aiortc/aioquic.git 0.8.7 --squash (note that this is checked in but not used at the moment)

The plan is to have wpt serve call wpt servequic (behind a flag).

For more details, see
https://github.com/web-platform-tests/rfcs/blob/master/rfcs/quic.md

Tracking issue: #19114

@Hexcles
Copy link
Member Author

Hexcles commented Apr 17, 2020

@jgraham you probably want to skip the two commits generated by git subtree.

@Hexcles Hexcles assigned jgraham and unassigned LukeZielinski Apr 17, 2020
@wpt-pr-bot wpt-pr-bot temporarily deployed to wpt-preview-23075 April 17, 2020 20:02 Inactive
tools/wpt/wpt.py Outdated Show resolved Hide resolved
@wpt-pr-bot wpt-pr-bot temporarily deployed to wpt-preview-23075 April 17, 2020 20:52 Inactive
@wpt-pr-bot wpt-pr-bot temporarily deployed to wpt-preview-23075 April 17, 2020 21:17 Inactive
wpt Outdated Show resolved Hide resolved
@Hexcles Hexcles requested a review from jgraham April 20, 2020 22:41
@Hexcles
Copy link
Member Author

Hexcles commented Apr 20, 2020

@jgraham I've addressed your comments; PTAL again, thanks!

@wpt-pr-bot wpt-pr-bot temporarily deployed to wpt-preview-23075 April 20, 2020 22:47 Inactive
This does not include the actual server, but only sets up the skeleton:
* Introduce a "py3only" to commands.json
* Create the skeleton of tools/quic and set up its venv

The plan is to have `wpt serve` call `wpt servequic` (behind a flag).

For more details, see
https://github.com/web-platform-tests/rfcs/blob/master/rfcs/quic.md
git-subtree-dir: tools/third_party/aioquic
git-subtree-split: 88f258ae47b7ec85de5ecf4104665054b969a9bb
Hexcles added a commit that referenced this pull request May 27, 2020
I accientally squash-merged #23075 and screwed up the subtree setup, so
I'm deleting the whole directory and re-introduce the subtree in the
next commit.
Hexcles added a commit that referenced this pull request May 27, 2020
I accidentally squash-merged #23075 and screwed up the subtree setup, so
I'm deleting the whole directory and re-introduce the subtree in the
next commit.
Hexcles added a commit that referenced this pull request May 28, 2020
I accidentally squash-merged #23075 and screwed up the subtree setup, so
I'm deleting the whole directory and re-introduce the subtree in the
next commit.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants