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

rename directory of HermitCore's OS related files #16

Merged
merged 1 commit into from
Mar 19, 2023

Conversation

stlankes
Copy link
Contributor

The latest version of libstd provides a new location of the HermitCore's OS related files. The kernel depends on the latest version of libstd. Consequently, this PR renames the location.

@dylni
Copy link
Owner

dylni commented Mar 18, 2023

I'm surprised this was merged as a breaking change. Is there any way to make this change in a backward-compatible way?

If not, can you run rustfmt and use rustversion on HermitCore to target the correct version for this change?

The latest version of libstd provides a new location of the
HermitCore's OS related files. The kernel depends on the latest
version of libstd. Consequently, this PR renames the location.
@stlankes
Copy link
Contributor Author

Yes, these are breaking change. However, HermitCore is a tier 3 platform and depends on the nightly compiler. Currently, you have always to use the latest version.

I used rustfmt to format the source code.

@dylni
Copy link
Owner

dylni commented Mar 19, 2023

Thanks, I'll keep HermitCore on nightly support then.

@dylni dylni merged commit ca4c940 into dylni:master Mar 19, 2023
dylni added a commit that referenced this pull request Mar 19, 2023
@stlankes stlankes deleted the hermit branch March 19, 2023 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants