Extract Build verb from Compile

[cdmihai]

Compile is having two main responsibilities (happy paths):

1. find dependencies and decide which of them to compile
2. know how to invoke the bytecode and the native compiler for a project

In preparing for incremental compilation we need to separate responsibility 1. into a new verb, Build.
Eventually, Build needs to also:

• compute incrementality safety checks for the whole dependency tree before doing any compilation (issue Perform safety checks for incremental compilation #4493)
• decide between incremental or non incremental

I'll start with two PRs:

• split Build from Compile. CLI uses build instead of Compile
• add safety checks and --profile flag to Build ()

Here is the updated user experience from issue #4403

[blackdwarf]

@cdmihai we still need to define what this command does all-up. Can you add also a Readme.md that is tailored as with the rest of the commands in the repo that outlines this?

[cdmihai]

@blackdwarf

Sure. I'll edit the readme and update the PR.

In short, given a root project, Build expands it into a graph of projects (root project + dependencies) and then compiles that graph either incrementally or non incrementally. For each node in the graph, Build invokes Compile to compile that node. Compile is then responsible for knowing how to setup and call the compiler.

For users, the difference mainly lies in calling build instead of compile.

[dannyvv]

Agreed we should have. I'm okay if that happens succinctly on a future pull request. Since lots of folks are blocked on this I wouldn't hold off the PR

[blackdwarf]

For sure, no need to block the pr on this. Just for tracking.

[cdmihai]

I created a readme for Build: https://github.com/cdmihai/cli/wiki/Build-readme
And also modified the readme for compile: https://github.com/cdmihai/cli/wiki/Compiler---readme

I'll include them in my current PR once I'm done resolving conflicts :)

[blackdwarf]

Thanks, will take a look at the PR.

Thanks,
Z

Sent from my tricorder

On Fri, Dec 18, 2015 at 4:55 PM -0800, "Mihai Codoban" <[email protected]mailto:[email protected]> wrote:

I created a readme for Build: https://github.com/cdmihai/cli/wiki/Build-readme
And also modified the readme for compile: https://github.com/cdmihai/cli/wiki/Compiler---readme

I'll include them in my current PR once I'm done resolving conflicts :)

Reply to this email directly or view it on GitHubhttps://github.com/dotnet/cli/issues/532#issuecomment-165929075.

[blackdwarf]

Done in dotnet/cli#663