Skip to content

Build with zig #431

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.

Sign up for GitHub

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

Jump to bottom
Merged
gchatelet merged 5 commits into from
Oct 14, 2025
Merged

Build with zig #431

gchatelet merged 5 commits into from
Oct 14, 2025

Conversation

@inge4pres
Copy link
Contributor

@inge4pres inge4pres commented Oct 8, 2025 •
edited by Mizux
Loading

Reason for this PR

Relates to:

Details

  • add build.zig and build.zig.zon
  • add a zig.yml workflow to build on Github actions

Additional context

The build scripts is derived from what CMake does; it compiles a static library and an executable that links to it for testing.

The Github workflow simply ensure compilation via build.zig succeeds and runs the executable on 4 Linux targets.

Further work

build.zig could also be extended to include a test step that runs the c++ tests in the test folder.

@inge4pres
build with Zig
5eded15 
Add the necessary files to build a static library with Zig.
The build also includes producing execuable that can be cross-compiled

Signed-off-by: inge4pres <fgualazzi@gmail.com>
@inge4pres
ci: add zig build workflow
50676c2 
Signed-off-by: inge4pres <fgualazzi@gmail.com>
@inge4pres inge4pres marked this pull request as ready for review October 9, 2025 20:37
gchatelet
gchatelet approved these changes Oct 10, 2025
View reviewed changes
Copy link
Collaborator

@gchatelet gchatelet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Mizux can you have a look as well?

Mizux
Mizux reviewed Oct 10, 2025
View reviewed changes
Mizux
Mizux reviewed Oct 10, 2025
View reviewed changes
Mizux
Mizux reviewed Oct 10, 2025
View reviewed changes
Mizux
Mizux requested changes Oct 13, 2025
View reviewed changes
@inge4pres inge4pres requested a review from Mizux October 13, 2025 14:41
Mizux
Mizux approved these changes Oct 13, 2025
View reviewed changes
Copy link
Collaborator

@Mizux Mizux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Mizux Mizux added enhancement New feature or request zig labels Oct 13, 2025
@Mizux Mizux added this to the v0.11 milestone Oct 13, 2025
@inge4pres
CI: release script updates version in build.zig.zon
14af8d0 
Signed-off-by: inge4pres <fgualazzi@gmail.com>
@inge4pres inge4pres mentioned this pull request Oct 13, 2025
Closed
Mizux
Mizux approved these changes Oct 14, 2025
View reviewed changes
Copy link
Collaborator

@Mizux Mizux left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
note: Macos ci failure were flacky (and fixed by d868036)

@gchatelet gchatelet merged commit 2182eb5 into google:main Oct 14, 2025
27 of 32 checks passed
This was referenced Oct 14, 2025
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gchatelet gchatelet gchatelet approved these changes

@Mizux Mizux Mizux approved these changes

Assignees

@gchatelet gchatelet

@Mizux Mizux

Labels

enhancement New feature or request zig

Projects

None yet

Milestone

v0.11

Development

Successfully merging this pull request may close these issues.

3 participants

@inge4pres @gchatelet @Mizux