Skip to content

Conversation

@compnerd
Copy link
Member

Use a newly introduced swift_gyb_target_sources to gyb and use the
generated sources when building. Let CMake figure out when to run the
command, let it invoke the command properly, and indicate that the
sources being added to the target are generated.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@compnerd
Copy link
Member Author

@swift-ci please clean test

@compnerd
Copy link
Member Author

CC: @gottesmm @drexin

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 951fc473dc639274c041aab53c4a778a45e0d5ea

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 951fc473dc639274c041aab53c4a778a45e0d5ea

@compnerd
Copy link
Member Author

@swift-ci please clean test

@compnerd
Copy link
Member Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 9c92d5f8fc839d21e1baac548ceee06b6846c8ca

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 9c92d5f8fc839d21e1baac548ceee06b6846c8ca

Use a newly introduced `swift_gyb_target_sources` to gyb and use the
generated sources when building.  Let CMake figure out when to run the
command, let it invoke the command properly, and indicate that the
sources being added to the target are generated.
@compnerd
Copy link
Member Author

@swift-ci please clean test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 9c92d5f8fc839d21e1baac548ceee06b6846c8ca

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 9c92d5f8fc839d21e1baac548ceee06b6846c8ca

@compnerd
Copy link
Member Author

@swift-ci please smoke test

@compnerd
Copy link
Member Author

Batched in #31125

@compnerd compnerd closed this Apr 20, 2020
@compnerd compnerd deleted the host-gyb branch April 20, 2020 15:14
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.

2 participants