-
Notifications
You must be signed in to change notification settings - Fork 85
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 xtask crate for kernel building #378
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mkroening
force-pushed
the
xtask
branch
2 times, most recently
from
March 2, 2022 14:32
bcb4f40
to
b491b96
Compare
mkroening
force-pushed
the
xtask
branch
18 times, most recently
from
March 7, 2022 14:01
386a335
to
802d617
Compare
mkroening
force-pushed
the
xtask
branch
7 times, most recently
from
March 20, 2022 15:20
583b695
to
f3632e7
Compare
mkroening
force-pushed
the
xtask
branch
2 times, most recently
from
March 21, 2022 13:19
2b6f24f
to
3478c5a
Compare
Canceled. |
bors try |
bors cancel |
tryBuild failed: |
bors try |
tryBuild failed: |
mkroening
force-pushed
the
xtask
branch
2 times, most recently
from
March 26, 2022 13:31
2a4ad86
to
1855700
Compare
bors try |
tryBuild failed: |
bors try |
tryBuild failed: |
bors r=stlankes |
bors bot
added a commit
that referenced
this pull request
Mar 26, 2022
378: Add xtask crate for kernel building r=stlankes a=mkroening Closes hermit-os/hermit-rs#188. This adds an xtask package, with two subcommands `build` and `clippy`. `build` builds the kernel with the correct arguments and does any necessary post processing (symbol renaming). `clippy` runs clippy on the kernel for multiple architectures (though only `x86_64` at this time - see #381). Co-authored-by: Martin Kröning <[email protected]>
Build failed: |
bors retry |
bors bot
added a commit
to hermit-os/hermit-rs
that referenced
this pull request
Mar 26, 2022
242: build.rs: Use libhermit-rs xtask r=stlankes a=mkroening Closes #188. Closes #136. (The flag is now entirely handled by the kernel) Depends on hermit-os/kernel#378. This removes the dependency on cargo-binutils. This moves the kernel build logic and it's post processing into the kernel repository. Co-authored-by: Martin Kröning <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Closes hermit-os/hermit-rs#188.
This adds an xtask package, with two subcommands
build
andclippy
.build
builds the kernel with the correct arguments and does any necessary post processing (symbol renaming).clippy
runs clippy on the kernel for multiple architectures (though onlyx86_64
at this time - see #381).