Skip to content
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

Transmute refactor #308

Closed
wants to merge 3 commits into from
Closed

Transmute refactor #308

wants to merge 3 commits into from

Conversation

mxcl
Copy link
Contributor

@mxcl mxcl commented May 3, 2016

Preparing for --edit implementation

@@ -26,7 +26,8 @@ public class Options {
set { _build = newValue }
}

private var _build = getenv("SWIFT_BUILD_PATH")
private var _Packages: String?
private var _build: String?
Copy link
Contributor

Choose a reason for hiding this comment

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

wrong indent

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

mxcl added 3 commits May 6, 2016 11:35
We have --build-path now.
Fixes a few exclude bugs in the process. Extensions were not sufficient, needed a “manager” for the process.
@mxcl
Copy link
Contributor Author

mxcl commented May 6, 2016

@ddunbar you ok with this?

@mxcl
Copy link
Contributor Author

mxcl commented May 6, 2016

@swift-ci Please test

import Utility


class Apparatus {
Copy link
Contributor

Choose a reason for hiding this comment

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

What is the purpose of this class? Can we get a doc-comment here?

@ddunbar
Copy link
Contributor

ddunbar commented May 9, 2016

Sorry, I missed the question here... the changes seem ok to me but I'm not clear what conceptual role Apparatus fills yet. Also, I pushed a different fix for the TOOLCHAINS issue (although it might not be as complete as this one).

@ddunbar
Copy link
Contributor

ddunbar commented May 12, 2016

Closing this out while we figure out the right answer here...

@ddunbar ddunbar closed this May 12, 2016
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