Skip to content

[cmake] Update SwiftSupport.cmake to allow passing link flags #278

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

Merged
merged 1 commit into from
Jun 20, 2019

Conversation

drodriguez
Copy link
Contributor

Also, pass the explicit target to the Swift library creation, so one can
cross compile.

/cc @compnerd

@compnerd
Copy link
Member

Lets not duplicate that. The emission of the output map file will be done by the migration to the newer CMake. If you want to add the explicit TARGET, that is fine.

Also, pass the explicit target to the Swift library creation, so one can
cross compile.
@drodriguez drodriguez force-pushed the cmake-update-swift-support branch from e549d37 to 0f028db Compare June 19, 2019 18:09
@drodriguez
Copy link
Contributor Author

Removed all the changes except the essential to cross-compile.

@swift-ci please test

@drodriguez drodriguez changed the title [cmake] Update SwiftSupport.cmake to match Foundations's [cmake] Update SwiftSupport.cmake to allow passing link flags Jun 19, 2019
@@ -219,7 +219,7 @@ function(add_swift_executable executable)
add_swift_target(${executable} ${ARGN})
endfunction()

# Returns the current achitecture name in a variable
# Returns the current architecture name in a variable
Copy link
Member

Choose a reason for hiding this comment

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

I spell gud

@compnerd
Copy link
Member

@swift-ci please test

@drodriguez
Copy link
Contributor Author

Is there a way of fixing those OS X failures? Seems to be something with Xcode, not with the project itself.

@compnerd compnerd merged commit a15b58d into swiftlang:master Jun 20, 2019
@drodriguez drodriguez deleted the cmake-update-swift-support branch June 20, 2019 19:51
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