Skip to content

Conversation

ahoppen
Copy link
Member

@ahoppen ahoppen commented Aug 30, 2025

This can be useful when implementing a system that needs to configure the active run destination of a build request but only knows the platform that the user is requesting a build for, but no architecture etc.

@ahoppen ahoppen force-pushed the preferred-run-destination branch 2 times, most recently from 2543368 to 64da735 Compare August 30, 2025 08:25
@ahoppen ahoppen changed the title Add requests that can be used to implement a BSP server based on swift-builds build description Add request to get preferred run destination for a platform Aug 30, 2025
@ahoppen ahoppen force-pushed the preferred-run-destination branch from 64da735 to c1e0629 Compare August 30, 2025 09:33
@ahoppen ahoppen marked this pull request as ready for review August 30, 2025 09:39
Copy link
Collaborator

@owenv owenv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but we should get @jakepetroules's opinion

Copy link
Collaborator

@jakepetroules jakepetroules left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems reasonable to me, if the API contract is basically "give me a suggested destination given the specified input parameters", without defining/promising the semantics used to define that.

@ahoppen
Copy link
Member Author

ahoppen commented Sep 1, 2025

Seems reasonable to me, if the API contract is basically "give me a suggested destination given the specified input parameters", without defining/promising the semantics used to define that.

That’s the idea. Do you think this needs to be clarified somewhere in a doc comment or is it clean enough from the current ones?

@ahoppen ahoppen force-pushed the preferred-run-destination branch from c1e0629 to 17b6273 Compare September 1, 2025 09:24
@ahoppen
Copy link
Member Author

ahoppen commented Sep 1, 2025

Added a few test cases as well based on Owen’s comment here: #761. Would be good to get a review for those as well.

@ahoppen
Copy link
Member Author

ahoppen commented Sep 1, 2025

@swift-ci Please test

@jakepetroules
Copy link
Collaborator

@swift-ci test

@ahoppen ahoppen force-pushed the preferred-run-destination branch from 17b6273 to 87a881d Compare September 2, 2025 18:44
This can be useful when implementing a system that needs to configure the active run destination of a build request but only knows the platform that the user is requesting a build for, but no architecture etc.
@ahoppen ahoppen force-pushed the preferred-run-destination branch from 87a881d to 461ae49 Compare September 2, 2025 18:46
@ahoppen
Copy link
Member Author

ahoppen commented Sep 2, 2025

@swift-ci Please test

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.

4 participants