-
Notifications
You must be signed in to change notification settings - Fork 240
Conversation
Hi @jasonwilliams200OK, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution! The agreement was validated by .NET Foundation and real humans are currently evaluating your PR. TTYL, DNFBOT; |
Linux = 2, | ||
OSX = 4, | ||
AnyUnix = Linux | OSX, | ||
Any = Windows | Linux | OSX | ||
Windows = 1 |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Did you build and test this locally to build corefx? The CI system for buildtools only builds it; it doesn't then try to use the newly built build tools to build corefx. We shouldn't take a PR in unless we know we at least haven't broken support for building corefx for the other platforms. |
} | ||
} | ||
} |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Agree with @stephentoub, we need to test the latest buildtools with corefx build. Steps to test locally are outlined here |
@stephentoub, @Priya91, thanks for the review. I tested building it on Windows box as per the wiki instructions. On executing
Whereas when I execute only I have ditto result with Can you confirm if there no other hack required to make it build? |
I believe this is a result of some newly published packages that are not supported with the current version of dnx. My attempt to upgrade to a newer version yesterday hit a road-block. Can you confirm whether any project.lock.json files got edited when you built using the new tools in corefx? |
@mmitche, with current master, it does not edit any lock file when I run: git clone https://github.com/dotnet/buildtools
cd build tools
"C:\Program Files (x86)\Microsoft Visual Studio 12.0\VC\vcvarsall.bat" x86 && build.cmd
:: or
"C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\vcvarsall.bat" x86 && build.cmd It throws:
|
This is what I get trying to build it using MSBuild xplat dotnet/msbuild@dc043c7 |
Refs: dotnet/corefx#2068 PR-URL: #180
Taken from corefx: commit 67fd826 Author: Jeremy Kuhne <jeremy.kuhne@microsoft.com> Date: Thu Apr 30 15:56:28 2015 -0700 Workarounds for building with Roslyn/MSBuild on Unix. Allows a successful compile with xplat MSBuild branch and Roslyn on Mono. Adds the Roslyn package and UseRoslyn property to use Roslyn. Disable PDB generation (NYI), skip packaging, disables ECMA delay signing.
Thanks @saper, @akoeplinger for https://github.com/jasonwilliams200OK/buildtools/pull/1 and confirming it works on FreeBSD and Linux. 👍 |
Only nuget fails with:
most probably a slash problem, because the files are there. |
@mmitche, these changes are tested on FreeBSD and Linux. |
LGTM |
I'll publish a new buildtools this afternoon. |
Thanks! 👍 |
Refs: dotnet/corefx#2068
R=@stephentoub,@mmitche,@josteink,@janhenke,@ghuntley