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.

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

build.rs: Use libhermit-rs xtask #242

Merged
merged 1 commit into from
Mar 26, 2022
Merged

build.rs: Use libhermit-rs xtask #242

merged 1 commit into from
Mar 26, 2022

Conversation

mkroening
Copy link
Member

@mkroening mkroening commented Mar 2, 2022 •
edited
Loading

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.

@mkroening mkroening force-pushed the xtask branch 9 times, most recently from 41204c3 to 10b8ae4 Compare March 4, 2022 17:39
@mkroening mkroening force-pushed the xtask branch 4 times, most recently from 0b39c22 to 9c22867 Compare March 20, 2022 15:22
@mkroening mkroening changed the title [WIP] build.rs: Use libhermit-rs xtask [DO NOT MERGE YET] build.rs: Use libhermit-rs xtask Mar 21, 2022
@mkroening mkroening marked this pull request as ready for review March 21, 2022 13:21
@mkroening mkroening self-assigned this Mar 21, 2022
@mkroening
Copy link
Member Author

bors try

bors bot added a commit that referenced this pull request Mar 25, 2022
@bors
Try #242:
3b47bb3
@bors
Copy link
Contributor

bors bot commented Mar 25, 2022

try

Build succeeded:

This was referenced Mar 26, 2022
Merged
Merged
bors bot added a commit that referenced this pull request Mar 26, 2022
@bors @mkroening
Merge #253 #254
1d2690e 
253: Remove .cargo/config.toml r=mkroening a=mkroening

Part of #242.

254: Sys refactoring r=mkroening a=mkroening

Part of #242.

Co-authored-by: Martin Kröning <[email protected]>
@mkroening mkroening force-pushed the xtask branch 4 times, most recently from bfdcc64 to d22fb25 Compare March 26, 2022 13:34
@mkroening
Copy link
Member Author

bors try

bors bot added a commit that referenced this pull request Mar 26, 2022
@bors
Try #242:
0e2f2cc
@bors
Copy link
Contributor

bors bot commented Mar 26, 2022

try

Build succeeded:

@mkroening mkroening force-pushed the xtask branch 3 times, most recently from e6b978e to 3e021f2 Compare March 26, 2022 14:52
@mkroening mkroening changed the title [DO NOT MERGE YET] build.rs: Use libhermit-rs xtask build.rs: Use libhermit-rs xtask Mar 26, 2022
@mkroening
Copy link
Member Author

bors r=stlankes

@bors
Copy link
Contributor

bors bot commented Mar 26, 2022

Build succeeded:

@bors bors bot merged commit 463a9d8 into hermit-os:master Mar 26, 2022
@mkroening mkroening deleted the xtask branch March 26, 2022 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mkroening mkroening

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Move symbol renaming to libhermit-rs Re-enable mutable-noalias
1 participant
@mkroening