PSResourceGet
PSResourceGet copied to clipboard
`Get-PSResource` does not search in user context environment variable `PSModulePath`
Prerequisites
- [X] Write a descriptive title.
- [X] Make sure you are able to repro it on the latest released version
- [X] Search the existing issues.
Steps to reproduce
PackageManagement, PowerShellGetv2 and PowerShellGet 3.0.18-beta (and prior versions) does not search in directorories specified in user context environmental variable PSModulePath when searching for modules. But at least v3 has a -Path parameter.
Reproduce:
- Install
Az.Accountsto%LOCALAPPDATA%\Microsoft\PowerShell\Modulesby usingSave-PSResource. - Make sure said module is not installed any other places specified in
$env:PSModulePath.- Or rather any other default locations for PowerShell modules I guess.
- Make sure
%LOCALAPPDATA%\Microsoft\PowerShell\Modulesis specified inHKCU:\Environment\PSModulePath, and that PowerShell loaded it into$env:PSModulePathat start. Should also be seen in[System.Environment]::GetEnvironmentVariable('PSModulePath','User'). - Search for installed modules with
PowerShellGet\Get-PSResource -Name 'Az.Accounts'. Result: Not found. - Add
-Scope 'CurrentUser: Nope, still not found. - Add
-Path ('{0}\Microsoft\PowerShell\Modules' -f $env:LOCALAPPDATA): Found.
Microsoft.PowerShell.Core\Import-Module and Get-Module works as expected.
Expected behavior
PowerShellGet\Get-PSResource should search in all PSModulePath paths, also those specified in user context environmental variable.
Actual behavior
It does not.
Error details
No response
Environment data
PowerShellGet v3.0.18
Windows PowerShell 5.1 x64 on Windows 10 22H2
Visuals
No response
Here's the responsible code as far as I can see:
Seems doable to add PSModulePath here?
if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows) && scope is not null) {
Environment.GetEnvironmentVariable(
"PSModulePath",
scope.Value is ScopeType.CurrentUser ? EnvironmentVariableTarget.User : EnvironmentVariableTarget.Machine
);
}
I understand it's more complex than that. One must have tests etc.
Is this one up for grabs, or do you not want environment variable PSModulePath to have any say?
Edit: Here's a PowerShellers attempt. 😁
Click to view
if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows) && scope is not null) {
string PSModulePath = Environment.GetEnvironmentVariable(
"PSModulePath",
scope.Value is ScopeType.CurrentUser ? EnvironmentVariableTarget.User : EnvironmentVariableTarget.Machine
);
if (string.IsNullOrEmpty(PSModulePath)) {
psCmdlet.WriteVerbose("Found no value for PSModulePath given the scope.");
}
else {
foreach (string element in PSModulePath.Split(Path.PathSeparator)) {
if (Directory.Exists(element)) {
if (
string.Equals(
element.Split(Path.DirectorySeparatorChar).Last(),
"Modules",
StringComparison.OrdinalIgnoreCase
)
) {
psCmdlet.WriteVerbose(
string.Format(
"'{0}' exists, adding it to 'resourcePaths'.",
element
)
);
resourcePaths.Add(element);
resourcePaths.Add(Path.Combine(Directory.GetParent(element).FullName, "Scripts"));
}
else {
psCmdlet.WriteVerbose(
string.Format(
"'{0}' exists but does not match expected pattern, not adding it to 'resourcePaths'.",
element
)
);
}
}
else {
psCmdlet.WriteVerbose(
string.Format(
"'{0}' does not exist, not adding it to 'resourcePaths'.",
element
)
);
}
}
}
}