Skip to content

Support override env var for gusi arg - #244

Merged
Josh Wittner (jwittner) merged 1 commit into
microsoft:developfrom
jwittner:dev/setupInstanceBasePathEnvVar
Feb 8, 2023
Merged

Support override env var for gusi arg#244
Josh Wittner (jwittner) merged 1 commit into
microsoft:developfrom
jwittner:dev/setupInstanceBasePathEnvVar

Conversation

@jwittner

Copy link
Copy Markdown
Member

Add support for overriding the default value of BasePath in Get-UnitySetupInstance. Setting $env:UNITY_SETUP_INSTANCE_BASEPATH_DEFAULT to a comma separated list will default the search paths to the elements of that list.

Comment thread UnitySetup/UnitySetup.psm1
.PARAMETER BasePath
Under what base patterns should we look for Unity installs?
Defaults to Unity default locations by platform
Default can be configured by comma separated paths in $env:UNITY_SETUP_INSTANCE_DEFAULT_BASEPATH

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is kind of confusing since we refer to this as a "pattern" as well as a "path".

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree, think the arg should be 'Pattern', but don't want to break the api. Out of scope of this change I think too.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants