Skip to content

Process.Unix: consider executable permission while searching PATH. - #55504

Merged
stephentoub merged 1 commit into
dotnet:mainfrom
tmds:process_x
Jul 13, 2021
Merged

Process.Unix: consider executable permission while searching PATH.#55504
stephentoub merged 1 commit into
dotnet:mainfrom
tmds:process_x

Conversation

@tmds

@tmdstmds commented Jul 12, 2021

Copy link
Copy Markdown
Member

@ghost

ghost commented Jul 12, 2021

Copy link
Copy Markdown

Tagging subscribers to this area: @dotnet/area-system-diagnostics-process
See info in area-owners.md if you want to be subscribed.

Issue Details

Fixes #29940

@stephentoub@carlossanlop@adamsitnik@jozkee ptal

Author:tmds
Assignees:-
Labels:

area-System.Diagnostics.Process

Milestone:-

tmds
tmds commented Jul 12, 2021
View reviewed changes
Comment threadsrc/libraries/System.Private.CoreLib/src/System.Private.CoreLib.Shared.projitems Outdated
stephentoub
stephentoub reviewed Jul 12, 2021
View reviewed changes
Comment threadsrc/libraries/Common/src/Interop/Unix/System.Native/Interop.GetGroups.cs Outdated
stephentoub
stephentoub reviewed Jul 12, 2021
View reviewed changes
Comment threadsrc/libraries/System.Diagnostics.Process/src/System/Diagnostics/Process.Unix.cs Outdated
stephentoub
stephentoub reviewed Jul 12, 2021
View reviewed changes
Comment threadsrc/libraries/Common/src/Interop/Unix/System.Native/Interop.GetGroups.cs Outdated
stephentoub
stephentoub reviewed Jul 12, 2021
View reviewed changes
Comment threadsrc/libraries/Common/src/Interop/Unix/System.Native/Interop.GetGroups.cs Outdated
stephentoub
stephentoub approved these changes Jul 12, 2021
View reviewed changes

@stephentoubstephentoub left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@tmds

tmds commented Jul 13, 2021

Copy link
Copy Markdown
MemberAuthor

CI failures are unrelated.

@stephentoub
stephentoub merged commit f915132 into dotnet:mainJul 13, 2021
@runfoapprunfoappBot mentioned this pull request Jul 15, 2021
Closed
@adamsitnikadamsitnik added this to the 6.0.0 milestone Jul 19, 2021
@ManickaPManickaP mentioned this pull request Jul 20, 2021
Merged
@ghostghost locked as resolved and limited conversation to collaborators Aug 18, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@stephentoubstephentoubstephentoub approved these changes

Assignees

No one assigned

Labels

area-System.Diagnostics.Process

Projects

None yet

Milestone

6.0.0

Development

Successfully merging this pull request may close these issues.

broken symlinks and non-executable files not ignored in FindProgramInPath()

3 participants

@tmds@stephentoub@adamsitnik