Skip to content

[build_script.py] Provide default temp build dir #55

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
Feb 26, 2016

Conversation

modocache
Copy link
Contributor

If a user doesn't care about where the built products are located, they may omit the "--build-dir" flag, and the build script will generate a temporary directory for them.

This allows the tests to be run on Linux with a very simple command:

$ ./build_script.py --swiftc `which swiftc` --test

@briancroom
Copy link
Contributor

Nice! Great way to lower the barrier to entry.

@mike-ferris
Copy link

Cool. Thanks, Brian!

@mike-ferris
Copy link

Looks like this is now conflicting with your other pull I merged... Can you rebase?

If a user doesn't care about where the built products are located,
they may omit the "--build-dir" flag, and the build script will
generate a temporary directory for them.

This allows the tests to be run on Linux with a very simple command:

```
$ ./build_script.py --swiftc `which swiftc` --test
```
@modocache modocache force-pushed the build-script-temp-build-dir branch from a3367b6 to 6351b6c Compare February 26, 2016 17:22
@modocache
Copy link
Contributor Author

@mike-ferris-apple Rebased and ready to go! The novelty still hasn't worn off on me--should we ask @swift-ci to "please test"? 😁

@mike-ferris
Copy link

@swift-ci please test

mike-ferris pushed a commit that referenced this pull request Feb 26, 2016
[build_script.py] Provide default temp build dir
@mike-ferris mike-ferris merged commit e480d02 into swiftlang:master Feb 26, 2016
@modocache modocache deleted the build-script-temp-build-dir branch February 27, 2016 20:34
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.

3 participants