-
Notifications
You must be signed in to change notification settings - Fork 43
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
target-lexicon OS sometimes doesn't match rustc target jsons #78
Comments
For Target-lexicon doesn't read rustc target specs. If you want the target os rustc specifies, you can run I think the current behavior is the desired one, but @sunfishcode may disagree. |
I think target-lexicon may have taken the wrong path with In general, I think it makes sense for target-lexicon to aspire to be compatible with the rustc target json categories, including mapping "darwin" => "macos" as mentioned above. This may mean becoming less compatible with rust target names, but my understanding of the relationship between rust target names and rustc target json categories has evolved, and I now think this makes sense. This should also help with #63. |
May I ask which one is preferable to avoid confusion on selecting between |
target-lexicon was designed to be used by compilers and compiler-like tools. The use of In hindsight, That would reduce confusion in one area, thought it may cause some new confusion in another, because it would mean having both |
I've been playing around with
target-lexicon
a bit and it seems like while most of the time it is pretty faithful to the rustc target json, some of the time the result diverges.For example,
i686-linux-android
has this rustc target JSON:However,
target-lexicon
parses the same triple as having the OS Linux rather than Android.Somewhat related, for
"aarch64-apple-darwin"
:"os": "macos"
I'm not sure exactly what behavior is desired here but I thought it might be useful to flag this. Thanks!
The text was updated successfully, but these errors were encountered: