-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Conversation
I'm probably going to merge this in shortly. We (DNX) want to build out the library that provides the "project model" (i.e. parsing and interpreting project.json/global.json/project.lock.json) and this serves as a good proving ground to see how it works. It also moves us towards self-hosting. At @davidfowl 's suggestion, pinging @jaredpar @piotrpMSFT . Feel free to take a look, but the code definitely needs clean-up ;). One thing in particular I'm not 100% sure on yet is breaking out dependency and source resolution into separate apps. They may be able to just be libraries without going all the way down to individual exes. Bring in any other reviewers you'd like. |
@brthor can you take a look please? |
|
||
# work around restore timeouts on Mono | ||
[ -z "$MONO_THREADS_PER_CPU" ] && export MONO_THREADS_PER_CPU=50 |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
} | ||
} | ||
|
||
private static int Publish(string path, string framework, string runtime, DirectoryInfo projectDir, string outputPath) |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
👍 I think this is definitely pushing us in the right direction. |
Alright. There are still some small concerns going around but I think most of them are tracked in issues. I'm going to merge this in to move us forward a bit. |
Sounds good 👍 |
b424dac
to
6395ffb
Compare
Port GenerateDepsFile and GenerateRuntimeConfigurationFiles
It's not fully self-hosted, but the output does NOT require DNX to run. The apps themselves DO use DNX.
This is mostly just a prototype/spike to see what it would look like, but I do think with some clean-up it can start to take shape as a real implementation. This PR is mostly just to look at and try out. You should be able to clone this branch and use the
scripts\dotnet.cmd
file to run the various commands./cc @davidfowl