Skip to content

Conversation

@dilijev
Copy link
Contributor

@dilijev dilijev commented Jun 13, 2018

  • Update Windows build matrix to use a modernized CI configuration: Modern = RS4 + Dev 15.7 and Legacy7 = Server 2008 R2 + Dev14 and Legacy8 = Server 2012 R2 + Dev14
  • Update script logic for forcing detection of a specific msbuild version

Note that the legacy OS is the same as before (2008 R2 ~= Windows 7 SP1) which is the lowest Windows OS version ChakraCore supports, and the legacy version of VS has been updated to VS 2015 / Dev14, which is now the lowest Visual Studio version ChakraCore supports (since Dev12 support was dropped).

This change also adds a second legacy configuration for 2012 R2 ~= Windows 8.1 Blue.

@dilijev
Copy link
Contributor Author

dilijev commented Jun 13, 2018

Replaces #5280

@dilijev
Copy link
Contributor Author

dilijev commented Jun 14, 2018

@dotnet-bot test ci please

2 similar comments
@dilijev
Copy link
Contributor Author

dilijev commented Jun 25, 2018

@dotnet-bot test ci please

@dilijev
Copy link
Contributor Author

dilijev commented Jun 25, 2018

@dotnet-bot test ci please

@dilijev dilijev changed the title Modernize CI configuration: Modern = 2016 + Dev15; Legacy = 2012 R2 + Dev14 Modernize CI configuration: Modern = RS4 + Dev 15.7; Legacy = 2012 R2 + Dev14 Jun 25, 2018
… + Dev14

* Update Windows build matrix to use a modernized CI configuration: Modern = RS4 + Dev 15.7 and Legacy = Server 2008 R2 + Dev14
* Update script logic for forcing detection of a specific msbuild version

Note that the legacy config is the same as before (2008 R2 ~= Windows 7 SP1) which is the lowest Windows OS version ChakraCore supports, and VS 2015 / Dev14 is the lowest Visual Studio version ChakraCore supports. Unfortunately this configuration leaves out coverage for any Windows 8 versions (Modern config was previously 2012 R2 ~= Windows 8.1).
@dilijev dilijev changed the title Modernize CI configuration: Modern = RS4 + Dev 15.7; Legacy = 2012 R2 + Dev14 Modernize CI configuration: Modern = RS4 + Dev 15.7; Legacy = 2008 R2 + Dev14 Jun 26, 2018
@dilijev dilijev force-pushed the ci-updated-legacy-110 branch from fa7ca17 to 98cbeff Compare June 26, 2018 00:17
@dilijev
Copy link
Contributor Author

dilijev commented Jun 26, 2018

@dotnet-bot test ci please

@dilijev
Copy link
Contributor Author

dilijev commented Jun 26, 2018

@MSLaguana @jackhorton @digitalinfinity @Penguinwizzard I have added the Win 8.1 Blue daily builds.

@dotnet-bot test ci please

@dilijev dilijev force-pushed the ci-updated-legacy-110 branch from 4d99cdc to 7ea3fce Compare June 26, 2018 23:15
@dilijev
Copy link
Contributor Author

dilijev commented Jun 26, 2018

@dotnet-bot test ci please

@dilijev dilijev changed the title Modernize CI configuration: Modern = RS4 + Dev 15.7; Legacy = 2008 R2 + Dev14 Modernize CI configuration: Modern = RS4 + Dev 15.7; Legacy = (2008 R2 & 2012 R2) + Dev14 Jun 26, 2018
@dilijev
Copy link
Contributor Author

dilijev commented Jun 28, 2018

@dotnet-bot test ci please

@dilijev
Copy link
Contributor Author

dilijev commented Jun 28, 2018

@dotnet-bot
test slow tests
test nojit tests
test legacy tests

@dilijev
Copy link
Contributor Author

dilijev commented Jun 28, 2018

@dotnet-bot test ci please

@dilijev
Copy link
Contributor Author

dilijev commented Jun 28, 2018

@MSLaguana @Penguinwizzard @boingoing would you mind reviewing the latest revision?

@dilijev dilijev requested a review from boingoing June 28, 2018 22:37
Copy link
Contributor

@MSLaguana MSLaguana left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@boingoing boingoing left a comment

Choose a reason for hiding this comment

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

Looks good to me

@chakrabot chakrabot merged commit 282688b into chakra-core:release/1.10 Jun 29, 2018
chakrabot pushed a commit that referenced this pull request Jun 29, 2018
… 15.7; Legacy = (2008 R2 & 2012 R2) + Dev14

Merge pull request #5307 from dilijev:ci-updated-legacy-110

* Update Windows build matrix to use a modernized CI configuration: Modern = RS4 + Dev 15.7 and Legacy7 = Server 2008 R2 + Dev14 and Legacy8 = Server 2012 R2 + Dev14
* Update script logic for forcing detection of a specific msbuild version

Note that the legacy OS is the same as before (2008 R2 ~= Windows 7 SP1) which is the lowest Windows OS version ChakraCore supports, and the legacy version of VS has been updated to VS 2015 / Dev14, which is now the lowest Visual Studio version ChakraCore supports (since Dev12 support was dropped).

This change also adds a second legacy configuration for 2012 R2 ~= Windows 8.1 Blue.
@dilijev dilijev deleted the ci-updated-legacy-110 branch June 29, 2018 01:17
chakrabot pushed a commit that referenced this pull request Jun 29, 2018
…rn = RS4 + Dev 15.7; Legacy = (2008 R2 & 2012 R2) + Dev14

Merge pull request #5307 from dilijev:ci-updated-legacy-110

* Update Windows build matrix to use a modernized CI configuration: Modern = RS4 + Dev 15.7 and Legacy7 = Server 2008 R2 + Dev14 and Legacy8 = Server 2012 R2 + Dev14
* Update script logic for forcing detection of a specific msbuild version

Note that the legacy OS is the same as before (2008 R2 ~= Windows 7 SP1) which is the lowest Windows OS version ChakraCore supports, and the legacy version of VS has been updated to VS 2015 / Dev14, which is now the lowest Visual Studio version ChakraCore supports (since Dev12 support was dropped).

This change also adds a second legacy configuration for 2012 R2 ~= Windows 8.1 Blue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants