-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Build.zig rename orgy. Renames FileSource to LazyPath and others #16446
Merged
Commits on Jul 30, 2023
-
Build.zig rename orgy (aka: ziglang#16353). Renames FileSource to Laz…
…yPath and removes functions that take literal paths instead of LazyPath.
Configuration menu - View commit details
-
Copy full SHA for ce95a3b - Browse repository at this point
Copy the full SHA ce95a3bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 5c01818 - Browse repository at this point
Copy the full SHA 5c01818View commit details -
Configuration menu - View commit details
-
Copy full SHA for 35d0a49 - Browse repository at this point
Copy the full SHA 35d0a49View commit details -
Configuration menu - View commit details
-
Copy full SHA for ae11fba - Browse repository at this point
Copy the full SHA ae11fbaView commit details -
Configuration menu - View commit details
-
Copy full SHA for f8386de - Browse repository at this point
Copy the full SHA f8386deView commit details -
build system: follow-up enhancements regarding LazyPath
* introduce LazyPath.cwd_relative variant and use it for --zig-lib-dir. closes ziglang#12685 * move overrideZigLibDir and setMainPkgPath to options fields set once and then never mutated. * avoid introducing Build/util.zig * use doc comments for deprecation notices so that they show up in generated documentation. * introduce InstallArtifact.Options, accept it as a parameter to addInstallArtifact, and move override_dest_dir into it. Instead of configuring the installation via Compile step, configure the installation via the InstallArtifact step. In retrospect this is obvious. * remove calls to pushInstalledFile in InstallArtifact. See ziglang#14943 * rewrite InstallArtifact to not incorrectly observe whether a Compile step has any generated outputs. InstallArtifact is meant to trigger output generation. * fix child process evaluation code handling of `-fno-emit-bin`. * don't store out_h_filename, out_ll_filename, etc., pointlessly. these are all just simple extensions appended to the root name. * make emit_directory optional. It's possible to have nothing outputted, for example, if you're just type-checking. * avoid passing -femit-foo/-fno-emit-foo when it is the default * rename ConfigHeader.getTemplate to getOutput * deprecate addOptionArtifact * update the random number seed of Options step caching. * avoid using `inline for` pointlessly * avoid using `override_Dest_dir` pointlessly * avoid emitting an executable pointlessly in test cases Removes forceBuild and forceEmit. Let's consider these additions separately. Nearly all of the usage sites were suspicious.
Configuration menu - View commit details
-
Copy full SHA for 38840e2 - Browse repository at this point
Copy the full SHA 38840e2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6393928 - Browse repository at this point
Copy the full SHA 6393928View commit details
Commits on Jul 31, 2023
-
std.Build.Step.InstallArtifact: disable emit-h
This branch was not intended to introduce new test coverage on the emit-h feature. See ziglang#9698
Configuration menu - View commit details
-
Copy full SHA for bdbd617 - Browse repository at this point
Copy the full SHA bdbd617View commit details -
std.Build.LazyPath: fix resolution of cwd_relative
The callsites of getPath rely on the result being absolute so that they can pass the path to a child process with the cwd set to the build root.
Configuration menu - View commit details
-
Copy full SHA for 25a9487 - Browse repository at this point
Copy the full SHA 25a9487View commit details -
Configuration menu - View commit details
-
Copy full SHA for acbb641 - Browse repository at this point
Copy the full SHA acbb641View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.