-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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
Add local session configuration -ConfigurationFile cmd line parameter to pwsh #17447
Add local session configuration -ConfigurationFile cmd line parameter to pwsh #17447
Conversation
src/Microsoft.PowerShell.ConsoleHost/host/msh/CommandLineParameterParser.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.PowerShell.ConsoleHost/host/msh/CommandLineParameterParser.cs
Show resolved
Hide resolved
src/Microsoft.PowerShell.ConsoleHost/resources/ConsoleHostStrings.resx
Outdated
Show resolved
Hide resolved
src/Microsoft.PowerShell.ConsoleHost/resources/ManagedEntranceStrings.resx
Show resolved
Hide resolved
src/System.Management.Automation/engine/remoting/fanin/InitialSessionStateProvider.cs
Show resolved
Hide resolved
src/System.Management.Automation/engine/remoting/fanin/InitialSessionStateProvider.cs
Show resolved
Hide resolved
src/System.Management.Automation/engine/remoting/fanin/InitialSessionStateProvider.cs
Show resolved
Hide resolved
src/Microsoft.PowerShell.ConsoleHost/host/msh/CommandLineParameterParser.cs
Show resolved
Hide resolved
src/Microsoft.PowerShell.ConsoleHost/host/msh/CommandLineParameterParser.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.PowerShell.ConsoleHost/resources/CommandLineParameterParserStrings.resx
Outdated
Show resolved
Hide resolved
src/Microsoft.PowerShell.ConsoleHost/resources/ConsoleHostStrings.resx
Outdated
Show resolved
Hide resolved
src/Microsoft.PowerShell.ConsoleHost/resources/ManagedEntranceStrings.resx
Outdated
Show resolved
Hide resolved
src/System.Management.Automation/engine/remoting/fanin/InitialSessionStateProvider.cs
Show resolved
Hide resolved
…meterParserStrings.resx Co-authored-by: Dongbo Wang <dongbow@microsoft.com>
…ngs.resx Co-authored-by: Dongbo Wang <dongbow@microsoft.com>
…Strings.resx Co-authored-by: Dongbo Wang <dongbow@microsoft.com>
src/Microsoft.PowerShell.ConsoleHost/host/msh/CommandLineParameterParser.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.PowerShell.ConsoleHost/host/msh/CommandLineParameterParser.cs
Show resolved
Hide resolved
Co-authored-by: Ilya <darpa@yandex.ru>
Committee decided an error should be thrown in Console.Start() API case, when there are conflicting session configuration directives. |
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
The conflict check changes look good to me, but for the broken test, are you going to make further changes to enable it? |
@daxian-dbw I was thinking of leaving the disabled (commented out) test as an FYI for future developers to understand why we can't do multiple command line parsing tests. We could add a whole new hosting SDK test, just for this error, so that it could run in a new process/session, but it doesn't seem worth while. I can remove the disabled test if you like. |
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.
LGTM. I'm fine keeping the disabled test.
@PaulHigin Could you file a docs-issue for this one? I see that the PR body's check list specifies docs are needed (and the functionality definitely needs documentation imo) but I don't see one filed. I also noticed that there was an update to New-PSSessionConfigurationFile which seems to imply that this cmdlet will be available on Unix systems as well, going forward? I think that will also require a docs update, if so. |
@michaeltlombardi Thanks for catching this! |
This change adds documentation for the new ConfigurationFile parameter added to the pwsh executable in PowerShell/PowerShell#17447. Resolves MicrosoftDocs#8907
This change adds documentation for the new ConfigurationFile parameter added to the pwsh executable in PowerShell/PowerShell#17447. Resolves MicrosoftDocs#8907
This change adds documentation for the new ConfigurationFile parameter added to the pwsh executable in PowerShell/PowerShell#17447. Resolves #8907
🎉 Handy links: |
PR Summary
This PR adds a new -ConfigurationFile command line parameter to pwsh.exe, that takes a path to a .pssc configuration file.
This is an implementation of RFC.
PR Context
This new command line parameter will allow PowerShell interactive or server mode sessions to run in a session configured by the provided .pssc file.
PR Checklist
.h
,.cpp
,.cs
,.ps1
and.psm1
files have the correct copyright headerWIP:
or[ WIP ]
to the beginning of the title (theWIP
bot will keep its status check atPending
while the prefix is present) and remove the prefix when the PR is ready.(which runs in a different PS Host).