Skip to content

Add --release flag for bootstrap script #231

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 2 commits into from
Apr 2, 2016
Merged

Add --release flag for bootstrap script #231

merged 2 commits into from
Apr 2, 2016

Conversation

mxcl
Copy link
Contributor

@mxcl mxcl commented Mar 31, 2016

This allows the bootstrap script to build SwiftPM in release mode.

We need to add integration to the swift build-script so that we are built in release mode as part of snapshots.

@mxcl
Copy link
Contributor Author

mxcl commented Apr 1, 2016

@swift-ci Please test

@@ -648,7 +653,6 @@ def main():

swift_build_path = os.path.join(build_path, "debug", "swift-build")
swift_test_path = os.path.join(build_path, "debug", "swift-test")
note("built: %s" % (swift_build_path,))
Copy link
Contributor

Choose a reason for hiding this comment

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

These paths aren't correct, and they are used in the testing step below...

Never mind, I see now that was fixed in the other commit.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, sorry, I spotted it later though at least.

@mxcl mxcl merged commit fcb9fe2 into swiftlang:master Apr 2, 2016
@mxcl mxcl deleted the bootstrap-release branch April 3, 2016 07:48
aciidgh pushed a commit to aciidgh/swift-package-manager that referenced this pull request Jan 11, 2019
* Add file-system client config attribute support

Allows setting 'device-agnostic' file system in the build file

rdar://problem/37445150
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