Skip to content

Conversation

dilijev
Copy link
Contributor

@dilijev dilijev commented Jun 7, 2018

  • Update Windows build matrix to use modern configuration Server 2016 + Dev15 and Legacy configuration Server 2012 R2 + Dev14

  • Update script logic for forcing detection of a specific msbuild version.

… Dev14

* Update Windows build matrix to use modern configuration Server 2016 + Dev15 and Legacy configuration Server 2012 R2 + Dev14

* Update script logic for forcing detection of a specific msbuild version.
@dilijev
Copy link
Contributor Author

dilijev commented Jun 7, 2018

See previous PR #4767

'Windows_NT': 'Windows', // Windows Server 2012 R2, equivalent to Windows 8.1 (aka Blue)
'Windows 7': 'Windows 7', // Windows Server 2008 R2 ~= Windows 7
'Windows_NT': 'Windows', // 'latest-or-auto' -> Windows Server 2012 R2 ~= Windows 8.1 aka Blue
// 'latest-dev15-5' -> Windows Server 2016 ~= Windows 10 RS2/RS3, with Dev15.5
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mmitche do you know the exact build number here? In our case it may matter exactly which build it is. /cc @jackhorton

Copy link
Contributor

Choose a reason for hiding this comment

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

For context, I would like to try it out locally to see if we can use the ICU on the system or not. Additionally, I would be interested in knowing which Windows SDK it will have (the RS2 SDK has one set of header files for ICU, while the RS3+ SDK has a different header. I believe the DLLs are compatible, but the headers weren't because they switched from wchar_t/uint16_t to char16_t). /cc @jefgen

Copy link
Collaborator

Choose a reason for hiding this comment

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

Right, the DLLs should indeed be binary compatible, though the headers are different between RS2 and RS3+. In other words, if you use the RS2 headers and build on RS2, then your resulting binary should run on RS3. (cc: @Microsoft/target-dev )

@dilijev dilijev changed the base branch from master to release/1.10 June 13, 2018 19:26
@dilijev dilijev self-assigned this Jun 13, 2018
@dilijev
Copy link
Contributor Author

dilijev commented Jun 13, 2018

Replaced by #5307

@dilijev dilijev closed this Jun 13, 2018
@dilijev dilijev deleted the ci-master-updated-legacy branch June 13, 2018 19:45
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.

5 participants