-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[swift-inspect] Start at least building swift-inspect on Darwin. #37377
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
[swift-inspect] Start at least building swift-inspect on Darwin. #37377
Conversation
@shahmishal What do you think? |
@swift-ci test |
1 similar comment
@swift-ci test |
Already passed, just want to check that we actually built it. |
Build failed |
Build failed |
@swift-ci test macOS platform |
@swift-ci smoke test |
@swift-ci smoke test linux platform |
@mikeash one thing we may want to do is make this use the shared swiftpm build root. I think you are building ArgumentParser for the Nth time. I still think enabling this in the short terms good. I checked the macOS build and it did build swift inspect. That way we at least know it keeps working. |
Makes sense to me. How hard is it to make it use the shared build root? |
Not much. It is easy to fix. |
@mikeash looks like swift inspect doesn't work on Linux. (Windows it isn't built). |
…latforms. It isn't setup appropriately to build on Linux. But at least this will keep the Darwin side working.
8fa12de
to
2bdb681
Compare
@swift-ci smoke test |
I was wrong about the shared root. I thought we had more infrastructure for it. I am going to look into it in the future. Not much increase in compile time overall though. |
Just for testing for now.