Skip to content
This repository has been archived by the owner on Jan 27, 2023. It is now read-only.

Integration of zuri inside http #3

Open
ducdetronquito opened this issue Sep 26, 2020 · 2 comments
Open

Integration of zuri inside http #3

ducdetronquito opened this issue Sep 26, 2020 · 2 comments

Comments

@ducdetronquito
Copy link

ducdetronquito commented Sep 26, 2020

Hi @Vexu !

Few days ago, I started a small library to bundle all HTTP core types for Zig into one place.
I really want to use zuri as the URI parser implementation so I had two choices:

  • Use git submodules referencing your repository directly of eventually a fork of mine
  • Include zuri's codebase directly

I found the latter to be the easiest one, but I wanted to have your approval on this before merging anything.

Here is a pull request that makes a plain copy of zuri, with the original license kept, and then adds minor changes for some unit tests.

At some point, when the package will be feature complete, I hope it could go back to the standard library or a dedicated organization depending on how the Zig community want to do things (cf: ziglang/zig#2007).

I am looking forward to your response; have a nice weekend 🌴

@ducdetronquito ducdetronquito changed the title Integration of zuri inside of http Integration of zuri inside http Sep 26, 2020
@Vexu
Copy link
Owner

Vexu commented Sep 26, 2020

  • Include zuri's codebase directly

I'm OK with that, just beware that this might receive some extensive changes at some point since this was originally built on 0.4.0 and is also not very optimized.

@ducdetronquito
Copy link
Author

Thanks !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants