-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Move powershell to 2.0.0-preview3-25426-01 using the .NET CLI 2.0.0-preview2-006502
#4144
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -17,17 +17,17 @@ | |
| </ItemGroup> | ||
|
|
||
| <ItemGroup> | ||
| <PackageReference Include="System.Data.SqlClient" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.IO.Packaging" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.Net.Http.WinHttpHandler" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.ServiceModel.Duplex" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.ServiceModel.Http" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.ServiceModel.NetTcp" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.ServiceModel.Primitives" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.ServiceModel.Security" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.Text.Encodings.Web" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.Threading.AccessControl" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.Private.ServiceModel" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.Data.SqlClient" Version="4.4.0-preview3-25423-02" /> | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is 02 expected? or should it be 01?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, it is expected. |
||
| <PackageReference Include="System.IO.Packaging" Version="4.4.0-preview3-25423-02" /> | ||
| <PackageReference Include="System.Net.Http.WinHttpHandler" Version="4.4.0-preview3-25423-02" /> | ||
| <PackageReference Include="System.ServiceModel.Duplex" Version="4.4.0-preview3-25423-01" /> | ||
| <PackageReference Include="System.ServiceModel.Http" Version="4.4.0-preview3-25423-01" /> | ||
| <PackageReference Include="System.ServiceModel.NetTcp" Version="4.4.0-preview3-25423-01" /> | ||
| <PackageReference Include="System.ServiceModel.Primitives" Version="4.4.0-preview3-25423-01" /> | ||
| <PackageReference Include="System.ServiceModel.Security" Version="4.4.0-preview3-25423-01" /> | ||
| <PackageReference Include="System.Text.Encodings.Web" Version="4.4.0-preview3-25423-02" /> | ||
| <PackageReference Include="System.Threading.AccessControl" Version="4.4.0-preview3-25423-02" /> | ||
| <PackageReference Include="System.Private.ServiceModel" Version="4.4.0-preview3-25423-01" /> | ||
| </ItemGroup> | ||
|
|
||
| </Project> | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -11,12 +11,12 @@ | |
| <ItemGroup> | ||
| <ProjectReference Include="..\Microsoft.PowerShell.CoreCLR.AssemblyLoadContext\Microsoft.PowerShell.CoreCLR.AssemblyLoadContext.csproj" /> | ||
| <ProjectReference Include="..\Microsoft.PowerShell.CoreCLR.Eventing\Microsoft.PowerShell.CoreCLR.Eventing.csproj" /> | ||
| <PackageReference Include="Microsoft.Win32.Registry.AccessControl" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="Microsoft.Win32.Registry.AccessControl" Version="4.4.0-preview3-25423-02" /> | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same here 02 or 01?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 02 is expected. Please see my response above. |
||
| <PackageReference Include="Newtonsoft.Json" Version="10.0.1" /> | ||
| <PackageReference Include="System.IO.FileSystem.AccessControl" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.Security.AccessControl" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.Security.Cryptography.Pkcs" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.Text.Encoding.CodePages" Version="4.4.0-preview1-25302-01" /> | ||
| <PackageReference Include="System.IO.FileSystem.AccessControl" Version="4.4.0-preview3-25423-02" /> | ||
| <PackageReference Include="System.Security.AccessControl" Version="4.4.0-preview3-25423-02" /> | ||
| <PackageReference Include="System.Security.Cryptography.Pkcs" Version="4.4.0-preview3-25423-02" /> | ||
| <PackageReference Include="System.Text.Encoding.CodePages" Version="4.4.0-preview3-25423-02" /> | ||
| <PackageReference Include="Microsoft.Management.Infrastructure" Version="1.0.0-alpha*" /> | ||
| </ItemGroup> | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| <Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003"> | ||
| <PropertyGroup> | ||
| <Product>PowerShell Core Test</Product> | ||
| <Company>Microsoft Corporation</Company> | ||
| <Copyright>(c) Microsoft Corporation. All rights reserved.</Copyright> | ||
|
|
||
| <TargetFramework>netcoreapp2.0</TargetFramework> | ||
| <RuntimeFrameworkVersion>2.0.0-preview3-25426-01</RuntimeFrameworkVersion> | ||
|
|
||
| <TreatWarningsAsErrors>true</TreatWarningsAsErrors> | ||
| <AllowUnsafeBlocks>true</AllowUnsafeBlocks> | ||
| </PropertyGroup> | ||
| </Project> |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -66,7 +66,9 @@ Describe "Get-ChildItem" -Tags "CI" { | |
| $file.Count | Should be 1 | ||
| $file.Name | Should be $item_F | ||
| } | ||
| It "Should continue enumerating a directory when a contained item is deleted" { | ||
| # Test is pending on Unix platforms because of a behavior change in the latest .NET Core. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The issue #20456 is closed, do we still need to have the test case pending?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The behavior change was by design, to better align with Full .NET. So yes, this test and the corresponding fix need to be revisited. Here is the tracking issue: #4145 |
||
| # See https://github.com/dotnet/corefx/issues/20456 for more information. | ||
| It "Should continue enumerating a directory when a contained item is deleted" -Pending:(!$IsWindows) { | ||
| $Error.Clear() | ||
| [System.Management.Automation.Internal.InternalTestHooks]::SetTestHook("GciEnumerationActionDelete", $true) | ||
| $result = Get-ChildItem -Path $TestDrive -ErrorAction SilentlyContinue | ||
|
|
@@ -84,7 +86,9 @@ Describe "Get-ChildItem" -Tags "CI" { | |
| $result.Count | Should BeExactly 4 | ||
| } | ||
| } | ||
| It "Should continue enumerating a directory when a contained item is renamed" { | ||
| # Test is pending on Unix platforms because of a behavior change in the latest .NET Core. | ||
| # See https://github.com/dotnet/corefx/issues/20456 for more information. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same as above.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ditto. |
||
| It "Should continue enumerating a directory when a contained item is renamed" -Pending:(!$IsWindows) { | ||
| $Error.Clear() | ||
| [System.Management.Automation.Internal.InternalTestHooks]::SetTestHook("GciEnumerationActionRename", $true) | ||
| $result = Get-ChildItem -Path $TestDrive -ErrorAction SilentlyContinue | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
preview3?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
preview3 .NET CLI is still broken, please see the description of this PR.
Now we are using .NET CLI preivew2 to build, but target preview3 .NET Core Framework.