Simplified build, replaced ziget dependency with std.http and added option to download zls #98
Simplified build, replaced ziget dependency with std.http and added option to download zls #98FabricatorZayac wants to merge 20 commits intomarler8997:masterfrom
Conversation
added zigup-init script
removed ziget dependency
|
I also added a zigup-init.sh script that downloads and installs the latest version of zigup |
|
I originally just wanted to just add a zls downloader (kinda like |
|
Wow you've done alot here! Note that there's currently an intermittent issue around TLS with Really like that you've made a |
|
Here's how I'd like to see these changes organized. First, let's get "zig fmt" out of the way (#99 done). Here's a list of PR's I would split this into:
There is already a PR for switching to |
don't allocate body, use fifo pipe intead
|
I'm not super experienced with github PRs, so I am not exactly sure how to split up my changes into multiple PRs. I've only done very minor contributions in the past. I am just going to look up how to do that |
|
As I understand, to do that I have to branch from the old master and cherry pick commits into the new branches and then make PRs from those? |
|
One problem I am running into trying to cherry-pick the "remove deprecated names" commits is that I did that alongside removing the ziget dependency and removing the first build step that fetched it. It should probably be separate from other misc changes |
|
ZLS installation didn't work with ziget, so that can only be added on top of migrating to std.http. |
|
Created the zigup-init pr (#100) |
No description provided.