Skip to content
This repository has been archived by the owner on Feb 23, 2021. It is now read-only.

Update RazorTooling to run on CoreCLR on other platforms asides from Windows. #38

Merged
merged 2 commits into from
Dec 14, 2015

Conversation

NTaylorMullen
Copy link
Member

…Windows.

- Refactored classes to not require inheriting from `Assembly`. It's not available on CoreCLR.

#30
@NTaylorMullen
Copy link
Member Author

/cc @CesarBS

@@ -13,7 +13,7 @@ addons:
before_install:
- if test "$TRAVIS_OS_NAME" == "osx"; then brew update; brew install icu4c; fi
env:
- KOREBUILD_DNU_RESTORE_CORECLR=true
- KOREBUILD_DNU_RESTORE_CORECLR=true MONO_THREADS_PER_CPU=2000 KOREBUILD_TEST_DNXCORE=true
Copy link
Contributor

Choose a reason for hiding this comment

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

Can't you set one per line?

Copy link
Contributor

Choose a reason for hiding this comment

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

I believe they had to be on a single line.

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Actually, https://docs.travis-ci.com/user/environment-variables/#Defining-Multiple-Variables-per-Item is more pertinent since we're avoiding matrix variables i.e. additional builds.

@cesarblum
Copy link
Contributor

Comments then :shipit:

@NTaylorMullen NTaylorMullen merged commit 973253b into dev Dec 14, 2015
@NTaylorMullen NTaylorMullen deleted the nimullen/coreclr.30 branch December 14, 2015 23:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants