Skip to content
This repository has been archived by the owner on May 23, 2022. It is now read-only.

Use rather case insensitive pgrep #9

Closed
wants to merge 1 commit into from
Closed

Use rather case insensitive pgrep #9

wants to merge 1 commit into from

Conversation

danielhusar
Copy link

Hi,

wouldn't be better to use case insensitive process matching?

So we can type fuck you chrome instead of fuck you Chrome :)

@danyshaanan
Copy link

👍

@danielhusar danielhusar changed the title Use rather canse insensitive pgrep Use rather case insensitive pgrep Sep 21, 2014
@robotlolita
Copy link
Member

Hmm, pgrep doesn't have a -i option. Which version of it are you using?

@danielhusar
Copy link
Author

Hi,

hmm its strange because its working for me.
I found it here: https://gist.github.com/KevinSjoberg/5003408

pgrep chrome prints nothing but pgrep -i chrome prints all chrome processes.

Im using mac os 10.10

I can also see it in man pgrep:

-i          Ignore case distinctions in both the process table and the supplied pattern.

@robotlolita
Copy link
Member

It's probably some BSD-only option, then. I guess parsing ps axo pid,comm, then handling the filtering in the JS would be the best option.

@danielhusar
Copy link
Author

Yep sounds good, I can take a look on it if you interested in this as well.

@robotlolita
Copy link
Member

If you've got the time for it, go ahead 👍

robotlolita added a commit that referenced this pull request May 16, 2015
This should address #9 on all platforms.
@robotlolita
Copy link
Member

@danielhusar hey, can you try the latest release and see if the case-insensitive patterns work for you?

@danielhusar
Copy link
Author

It doest work for me at all now :)

@danyshaanan
Copy link

@danielhusar Are you on OSX? See issue #19 and PR #18.

@danielhusar
Copy link
Author

Yeah I'm on 10.10.3

@danyshaanan
Copy link

@danielhusar Check again now. (re-install). Fixed by a change in xps.

@danielhusar
Copy link
Author

Yep works fine now

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.

3 participants