-
-
Notifications
You must be signed in to change notification settings - Fork 14.3k
Auto-Generate Tables in platform-support.md #150571
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
base: main
Are you sure you want to change the base?
Conversation
This comment has been minimized.
This comment has been minimized.
|
This is very cool! I started a similar effort a while ago in #120745 but never got to completing it. Maybe some of the code in the linked PR thee may be useful |
|
@Noratrieb Thanks, I'll take a look. |
7908030 to
ab9e5d8
Compare
This comment has been minimized.
This comment has been minimized.
ab9e5d8 to
8c1125f
Compare
|
Alright, I'm set up properly now so no more failing tidy or pushing obvious mistakes. Unfortunately it looks like depending on rustc_target directly is a dead-end, I'm getting a variety of errors related to it being a compiler-internal crate that is supposed to be compiled as part of bootstrap. The commit I just pushed (8c1125f) is my last attempt in that direction unless someone else has a solution. I'm going to switch to pulling data from |
This comment has been minimized.
This comment has been minimized.
|
Executing rustc like that seems like the best approach, yes. |
8c1125f to
25ef1e7
Compare
|
The job Click to see the possible cause of the failure (guessed by this bot) |
This is a rough draft inspired by @ChrisDenton in #150501 (comment). I won't have much time to work on it until next month but I wanted to get it out there.
This is implemented as an mdbook preprocessor that pulls data from the rustc_target crate, so it will always be accurate.
Assuming this works as intended (I am not set up to test locally), it's still missing the following: