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

Add some basic examples for the Zig Build System. #5800

Open
wants to merge 6 commits into
base: master
from

Conversation

@nmichaels
Copy link
Contributor

nmichaels commented Jul 5, 2020

No description provided.

doc/langref.html.in Outdated Show resolved Hide resolved
doc/langref.html.in Outdated Show resolved Hide resolved
doc/langref.html.in Outdated Show resolved Hide resolved
doc/langref.html.in Outdated Show resolved Hide resolved
doc/langref.html.in Outdated Show resolved Hide resolved
Co-authored-by: Jay Petacat <jay@jayschwa.net>
Copy link
Member

andrewrk left a comment

Thanks for filling out this section. It's a bit of a tricky one

doc/langref.html.in Outdated Show resolved Hide resolved
doc/langref.html.in Outdated Show resolved Hide resolved
// Include all the safety checks in the generated library.
lib.bundle_compiler_rt = true;
Comment on lines 9539 to 9540

This comment has been minimized.

@andrewrk

andrewrk Jul 13, 2020

Member

Here's a description of what compiler-rt is, it's not related to safety: https://ziglang.org/download/0.5.0/release-notes.html#compiler-rt

Safety checks are determined entirely by the build mode. https://ziglang.org/documentation/master/#Build-Mode

This comment has been minimized.

@nmichaels

nmichaels Jul 14, 2020

Author Contributor

Whoops, I guess I conflated this with -fstack-check, which depends on it. Will change.

doc/langref.html.in Outdated Show resolved Hide resolved
doc/langref.html.in Outdated Show resolved Hide resolved
doc/langref.html.in Outdated Show resolved Hide resolved
nmichaels and others added 2 commits Jul 13, 2020
Co-authored-by: Andrew Kelley <andrew@ziglang.org>
@nmichaels nmichaels requested a review from andrewrk Jul 18, 2020
nmichaels added 2 commits Jul 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.