Fix modprobe completion for newer modprobe versions #1124

Merged
merged 1 commit into from Nov 16, 2013

Conversation

Projects
None yet
2 participants
@ByScripts
Contributor

ByScripts commented Nov 16, 2013

Newer versions of modprobe no longer support "-l" switch. This update fixes that by replacing "modprobe -l" with "find /lib/modules/(uname -r)/kernel -type f"

@siteshwar siteshwar merged commit 76ab22f into fish-shell:master Nov 16, 2013

@siteshwar

This comment has been minimized.

Show comment
Hide comment
@siteshwar

siteshwar Nov 16, 2013

Member

Nice find. Thanks for your contribution.

To git@github.com:fish-shell/fish-shell.git
cfbb511..76ab22f master -> master

Member

siteshwar commented Nov 16, 2013

Nice find. Thanks for your contribution.

To git@github.com:fish-shell/fish-shell.git
cfbb511..76ab22f master -> master

@siteshwar

This comment has been minimized.

Show comment
Hide comment
@siteshwar

siteshwar Nov 16, 2013

Member

I have also disabled showing files in modprobe completions with 6eb7530

Member

siteshwar commented Nov 16, 2013

I have also disabled showing files in modprobe completions with 6eb7530

@ByScripts

This comment has been minimized.

Show comment
Hide comment
@ByScripts

ByScripts Nov 16, 2013

Contributor

You're welcome. I discovered Fish Shell 2 hours ago. Clearly a good one.

Nicely spotted for --no-files , didn't know this option.

Contributor

ByScripts commented Nov 16, 2013

You're welcome. I discovered Fish Shell 2 hours ago. Clearly a good one.

Nicely spotted for --no-files , didn't know this option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment