Add back ADSI and WMI type accelerators #7085
Conversation
SteveL-MSFT
requested review from
BrucePay,
TravisEz13,
adityapatwardhan and
anmenaga
as code owners
|
I believe the problem is we haven't published an updated pwrshplugin.dll yet so System.Management.dll and System.DirectoryServices.dll aren't in the trusted list and thus fails to initialize on the remote side. I'll work on getting pwrshplugin.dll published. |
There was a problem hiding this comment.
Can sort the using list?
There was a problem hiding this comment.
I'll put the two here in alpha order. Won't touch the ones above.
There was a problem hiding this comment.
The CommaDelimitedStringCollection class is in .Net Core 2.0+.
https://apisof.net/catalog/System.Configuration.CommaDelimitedStringCollection
There was a problem hiding this comment.
Can we sort the using list?
There was a problem hiding this comment.
Can we sort the using list?
There was a problem hiding this comment.
We should do a global pass if we want them all sorted. Separate PR.
There was a problem hiding this comment.
I ask only because you have already edited the list.
There was a problem hiding this comment.
I suppose I could update it just for this file since I'm touching this part anyways.
There was a problem hiding this comment.
Maybe use Compile Remove in csproj file?
|
Seems we add back 5 type accelerators - please update the PR description. |
SteveL-MSFT
changed the title
There was a problem hiding this comment.
We should enable the test on Windows and correct $totalAccelerators = 95 above.
There was a problem hiding this comment.
Missed the -Skip part of this test. Will fix. Not sure what you mean by "correct $totalAcclerators = 95 above" as 95 is the correct number.
There was a problem hiding this comment.
Seems we add 5 accelerators on Windows. I wonder why the count is increased only +1
|
@iSazonov can you formally review and signoff? |
update count of accelerators in test
|
I still have a concern about my comment
|
4 participants
PR Summary
Now that we include Microsoft.Windows.Compatibility assemblies with PowerShell Core 6.1, we can add back the [adsi], [adsisearcher], [wmi], [wmiclass], and [wmisearcher] accelerators putting back code that already exists.
Fix #3266
PR Checklist
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright headerWIP:to the beginning of the title and remove the prefix when the PR is ready.[feature]if the change is significant or affects feature tests