Skip to content

[CMake] Make swift-frontend the primary Swift binary. #28003

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 9 commits into from
Jun 24, 2020

Conversation

DougGregor
Copy link
Member

Build swift-frontend as the primary Swift binary, and have
swift/swiftc/etc. symlink over to it. This is a step toward allowing
swift-driver to replace swift and swiftc binaries.

@DougGregor
Copy link
Member Author

@swift-ci please test

@DougGregor
Copy link
Member Author

@swift-ci please test platform windows

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 034daa821c3b3843aecb51d0f4963a09ecba3b6c

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 034daa821c3b3843aecb51d0f4963a09ecba3b6c

@DougGregor DougGregor force-pushed the swift-frontend-rename branch from 034daa8 to 409509f Compare November 1, 2019 16:58
@DougGregor
Copy link
Member Author

@swift-ci please test

@DougGregor
Copy link
Member Author

@swift-ci please test platform Windows

@swift-ci
Copy link
Contributor

swift-ci commented Nov 1, 2019

Build failed
Swift Test Linux Platform
Git Sha - 034daa821c3b3843aecb51d0f4963a09ecba3b6c

@swift-ci
Copy link
Contributor

swift-ci commented Nov 1, 2019

Build failed
Swift Test OS X Platform
Git Sha - 034daa821c3b3843aecb51d0f4963a09ecba3b6c

@compnerd
Copy link
Member

compnerd commented Nov 6, 2019

@swift-ci please test Windows platform

@DougGregor
Copy link
Member Author

@swift-ci please test Windows platform

How can I track down the results when we run these tests?

@drodriguez
Copy link
Contributor

https://ci-external.swift.org/view/Pull%20Request/job/swift-PR-windows/160/ (but I don't know why it wasn't reported back here).

@DougGregor
Copy link
Member Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 28d1b23f6783b4c53184611e2e9beb15bd24137e

@DougGregor DougGregor force-pushed the swift-frontend-rename branch from 28d1b23 to cc23d4d Compare June 16, 2020 23:53
@DougGregor
Copy link
Member Author

@swift-ci test

@DougGregor
Copy link
Member Author

@swift-ci test source compatibility

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - cc23d4d52b2cb96a8c684f0ab5e619b63a8282f4

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - cc23d4d52b2cb96a8c684f0ab5e619b63a8282f4

@DougGregor
Copy link
Member Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - cc23d4d52b2cb96a8c684f0ab5e619b63a8282f4

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - cc23d4d52b2cb96a8c684f0ab5e619b63a8282f4

@DougGregor
Copy link
Member Author

@swift-ci please test

1 similar comment
@DougGregor
Copy link
Member Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 7e13bf704f3fb7292f8cf2280633cf8230da0cea

@DougGregor
Copy link
Member Author

@swift-ci please test

@DougGregor
Copy link
Member Author

@swift-ci test macOS

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 7e13bf704f3fb7292f8cf2280633cf8230da0cea

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 9fbf83a4f48b8dc4bd7913c3d1188cc370724a50

@swift-ci
Copy link
Contributor

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

@DougGregor
Copy link
Member Author

@swift-ci please test source compatibility

@DougGregor
Copy link
Member Author

@swift-ci please smoke test

1 similar comment
@DougGregor
Copy link
Member Author

@swift-ci please smoke test

Build swift-frontend as the primary Swift binary, and have
swift/swiftc/etc. symlink over to it. This is a step toward allowing
swift-driver to replace the swift and swiftc binaries.
This obviates the need for "-frontend", although we still accept it
for backward compatibility. Switch lit's %target-swift-frontend et al
over to calling swift-frontend.
Hardlinking to a symlink is not permitted, so use "swift-frontend"
instead of "swift" as the hard-link target.

Also, update one Linux-specific test to check for swift-frontend.
@DougGregor DougGregor force-pushed the swift-frontend-rename branch from 9fbf83a to d49e4a2 Compare June 23, 2020 21:14
@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 9fbf83a4f48b8dc4bd7913c3d1188cc370724a50

@swift-ci
Copy link
Contributor

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

@DougGregor
Copy link
Member Author

@swift-ci please test source compatibility Debug

2 similar comments
@DougGregor
Copy link
Member Author

@swift-ci please test source compatibility Debug

@DougGregor
Copy link
Member Author

@swift-ci please test source compatibility Debug

@DougGregor
Copy link
Member Author

@swift-ci please test source compatibility debug

@DougGregor
Copy link
Member Author

Source-compatibility failures are actually UPASS's, so we'll get that cleaned up. Windows failure is unrelated. Time to merge!

@DougGregor DougGregor merged commit 1725ece into swiftlang:master Jun 24, 2020
@DougGregor DougGregor deleted the swift-frontend-rename branch June 24, 2020 04:40
@davezarzycki
Copy link
Contributor

Hi @DougGregor – This broke unified builds:

CMake Error at /home/dave/s/u/swift/tools/swift-remoteast-test/CMakeLists.txt:13 (add_dependencies):
  Cannot add target-level dependencies to non-existent target "swift".

  The add_dependencies works for top-level logical targets created by the
  add_executable, add_library, or add_custom_target commands.  If you want to
  add file-level dependencies see the DEPENDS option of the add_custom_target
  and add_custom_command commands.


davezarzycki added a commit to davezarzycki/swift that referenced this pull request Jun 24, 2020
davezarzycki added a commit that referenced this pull request Jun 24, 2020
[CMake] Fix unified builds after #28003
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.

5 participants