-
Notifications
You must be signed in to change notification settings - Fork 22
Start updating tardy to Zig 0.15.1 #28
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
Conversation
49e1bef to
687ce3d
Compare
|
@mookums, instead of making this PR very big, I will make it small and do consecutive updates for the new Io API after this. I would appreciate it if you could review this PR at your earliest convenience. |
|
Hi, would prefer if we could have one PR for upgrading to 0.15.1. I can take a look at this one now but would prefer to take one look at the end. |
|
Okay then, I will move the |
impl enough changes to get examples and tests passing Enable llvm for building exe's to prevent error: undefined symbol: tardy_swap_frame Pass args to examples in build.zig Signed-off-by: Bernard Assan <mega.alpha100@gmail.com>
dcaa38d to
3e03f48
Compare
Start adopting new Reader & Writer Io interface Signed-off-by: Bernard Assan <mega.alpha100@gmail.com> Update Reader|Writer interface to match std disable unimplemented methods for now Remove deprecated Reader|Writer debug cat example it isn't giving the expected output I think I forgot to flush (hahahaha) Signed-off-by: Bernard Assan <mega.alpha100@gmail.com> get cat example working correctly investigate why readSliceShort and readVec behave differently Replace stream interface with a stream method TODO: fix all examples for correctness Signed-off-by: Bernard Assan <mega.alpha100@gmail.com> Fix readSliceShort for files and sockets rename stream to stream_to with updated Io API remove mode field in file and simplify code Remove unimplemented functions Signed-off-by: Bernard Assan <mega.alpha100@gmail.com> fix socket reading and detection of end of file Signed-off-by: Bernard Assan <mega.alpha100@gmail.com> more idiomatic Zig updates use smp_allocator in multithreaded context use the `t` format spec for enum use result location types for initialization replace Stream type with stream_to in socket and file Signed-off-by: Bernard Assan <mega.alpha100@gmail.com> Use zls's organize imports to organize imports Signed-off-by: Bernard Assan <mega.alpha100@gmail.com>
3e03f48 to
bf3b9b8
Compare
|
@mookums, this PR is Ready for review |
|
@bernardassan Just saw this, will review tomorrow afternoon 🎆 |
|
That'd be nice |
|
@bernardassan e2e test seems to work locally (on Linux) so that's a good sign!
{
description = "an asynchronous runtime for Zig";
inputs = {
nixpkgs.url = "github:nixos/nixpkgs/release-25.05";
zigPkgs.url = "github:mitchellh/zig-overlay";
zigPkgs.inputs.nixpkgs.follows = "nixpkgs";
zlsPkg.url = "github:zigtools/zls/0.15.0";
zlsPkg.inputs.zig-overlay.follows = "zigPkgs";
zlsPkg.inputs.nixpkgs.follows = "nixpkgs";
flake-utils.url = "github:numtide/flake-utils";
};
outputs =
{
nixpkgs,
zigPkgs,
zlsPkg,
flake-utils,
...
}:
flake-utils.lib.eachDefaultSystem (
system:
let
overlays = [
(final: prev: {
zigpkgs = zigPkgs.packages.${prev.system};
zls = zlsPkg.packages.${prev.system}.default;
})
];
pkgs = import nixpkgs { inherit system overlays; };
in
{
devShells.default = pkgs.mkShell {
packages = with pkgs; [
zigpkgs."0.15.1"
zls
openssl
inetutils
wrk
];
};
}
);
} |
update flake.nix Co-authored-by: Muki <muki@muki.gg> Signed-off-by: Bernard Assan <mega.alpha100@gmail.com>
currently just done enough changes to get examples and tests passing
Enable llvm for building exe's to prevent error: undefined symbol: tardy_swap_frame
Pass args to examples in build.zig