Skip to content

Fix proc/pid completion in OS X#129

Closed
bobthecow wants to merge 1 commit into
fish-shell:masterfrom
bobthecow:patch-1
Closed

Fix proc/pid completion in OS X#129
bobthecow wants to merge 1 commit into
fish-shell:masterfrom
bobthecow:patch-1

Conversation

@bobthecow
Copy link
Copy Markdown

No description provided.

@ridiculousfish
Copy link
Copy Markdown
Member

I'd love to get these completions working with OS X, but I see a few problems with these changes:

  • pgrep is not installed by default on OS X
  • __fish_complete_proc doesn't seem to return many processes. (I think you need to pass -x to ps)

Please address these, and I'll be glad to merge.

@bobthecow
Copy link
Copy Markdown
Author

Now that it's the future, both of your concerns should have been resolved. Can we merge it now? :)

@ridiculousfish
Copy link
Copy Markdown
Member

I ended up fixing these a different way. Commit is 6416cb4

@bobthecow
Copy link
Copy Markdown
Author

Sweet :)

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 18, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants