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

Make sure we actually use swift-driver #5885

Closed
wants to merge 1 commit into from
Closed

Make sure we actually use swift-driver #5885

wants to merge 1 commit into from

Conversation

neonichu
Copy link
Member

@neonichu neonichu commented Nov 8, 2022

This tries to use the swift-driver built during the first-stage for the "fake" toolchain we're constructing for bootstrapping. Basically a variation on #5842.

Trying first whether it is sufficient to just have the symlink, if not we should be able to combine the approach from #5842 and this together.

This tries to use the swift-driver built during the first-stage for the "fake" toolchain we're constructing for bootstrapping. Basically a variation on #5842.

Trying first whether it is sufficient to just have the symlink, if not we should be able to combine the approach from #5842 and this together.
@neonichu neonichu self-assigned this Nov 8, 2022
@neonichu
Copy link
Member Author

neonichu commented Nov 8, 2022

@swift-ci please smoke test

@neonichu
Copy link
Member Author

neonichu commented Nov 8, 2022

I guess either way, we also still have the mystery on why this is failing for the self-hosted macOS job :/

@neonichu
Copy link
Member Author

neonichu commented Nov 9, 2022

Doesn't look as if the driver existing is enough here. Going to try out bringing my change to bootstrap into the other PR.

@neonichu
Copy link
Member Author

neonichu commented Nov 9, 2022

Oh right, this can't work yet since the revert of #4213 is only on a branch, so we're currently not using the fake toolchain during swift test anymore

@neonichu
Copy link
Member Author

neonichu commented Dec 8, 2022

@swift-ci please smoke test

@neonichu
Copy link
Member Author

neonichu commented Dec 8, 2022

Closing, this doesn't solve the problem we're seeing.

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.

None yet

1 participant