search

AcademySoftwareFoundation / rez

An integrated package configuration, build and deployment system for software
https://rez.readthedocs.io
Apache License 2.0
916 stars 328 forks source link

Improve PowerShell plugin's profile handling functionality #1777

Open nrusch opened 2 weeks ago

nrusch commented 2 weeks ago

Improvements to the PowerShell plugin to support profile-related settings/options.

Add support for the package_commands_sourced_first config/runtime option

Prior to this change, the rez PowerShell implementation would always source the rez context script after the user/host profile scripts, so profile-level modifications to environment variables like PATH were always squashed by the unconditional overrides in the context script.

With this change, the relative source order of the shell profile vs. the context script can be properly controlled using the package_commands_sourced_first config option, matching the behavior of SH-based shells.

NOTE: Because this option defaults to True, this also implicitly changes rez's default behavior to source the shell profile after the context when using PowerShell.

Support the norc shell plugin option.

This enables the use of rez-env --norc to bypass the sourcing of any profile scripts when using PowerShell.

linux-foundation-easycla[bot] commented 2 weeks ago

CLA Signed

The committers listed above are authorized under a signed CLA.

nrusch commented 2 weeks ago

This may indirectly "resolve" #1279, since the user's profile will have the final say on any shell customization.

nrusch commented 6 days ago

The CLA has been sorted for rez. I'm not sure why this PR's status isn't updating though...

JeanChristopheMorinPerso commented 6 days ago

/easycla

JeanChristopheMorinPerso commented 6 days ago

@nrusch it should be good now. I manually triggered a refresh and it's now green.