Skip to content
This repository has been archived by the owner on Jan 11, 2024. It is now read-only.

Clean remaining OSEnvironment usages when corefx changes are merged #1323

Closed
mellinoe opened this issue Feb 7, 2017 · 2 comments
Closed

Clean remaining OSEnvironment usages when corefx changes are merged #1323

mellinoe opened this issue Feb 7, 2017 · 2 comments
Assignees
@weshaggard
Labels
0 - Backlog area-buildtools-code

Comments

@mellinoe
Copy link

mellinoe commented Feb 7, 2017

dotnet/corefx#15728 (comment)

There are a handful of things that are still using $(OSEnvironment) in here that are not being overridden by files in corefx. When we integrate corefx back into here, we should make sure to clean those up. All instances should most likely be replaced with a check like: "'$(RunningOnUnix)' == 'true'".
@mellinoe mellinoe mentioned this issue Feb 7, 2017
Merged
@weshaggard weshaggard self-assigned this Apr 27, 2017
@weshaggard
Copy link
Member

Doing this as part of the BT2 effort

@weshaggard
Copy link
Member

Done

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@weshaggard weshaggard

Labels
0 - Backlog area-buildtools-code
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@MichaelSimons @mellinoe @weshaggard