Skip to content

Replace external PATH executable lookup#4563

Merged
Gabriel Dufresne (GabrielDuf) merged 1 commit intomainfrom
mamoreau/path-executable-lookup
Apr 8, 2026
Merged

Replace external PATH executable lookup#4563
Gabriel Dufresne (GabrielDuf) merged 1 commit intomainfrom
mamoreau/path-executable-lookup

Conversation

@mamoreau-devolutions
Copy link
Copy Markdown
Contributor

Summary

  • replace the where.exe/which subprocess call in CoreTools.WhichMultiple() with an in-process PATH resolver
  • preserve Windows PATH ordering, current-directory precedence, PATHEXT lookup, and WSL PATH filtering
  • add regression tests for direct paths, PATH ordering, Windows PATHEXT, and Unix executable-bit handling

Validation

  • dotnet test .\src\UniGetUI.Core.Tools.Tests\UniGetUI.Core.Tools.Tests.csproj --verbosity q --nologo
  • dotnet test .\src\UniGetUI.sln --verbosity q --nologo

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@GabrielDuf Gabriel Dufresne (GabrielDuf) merged commit 69b1954 into main Apr 8, 2026
1 check passed
@GabrielDuf Gabriel Dufresne (GabrielDuf) deleted the mamoreau/path-executable-lookup branch April 8, 2026 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants