Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix yum package completions similar to the rpm fix #1269

Closed
dbrodie opened this issue Jan 27, 2014 · 1 comment
Closed

Fix yum package completions similar to the rpm fix #1269

dbrodie opened this issue Jan 27, 2014 · 1 comment
Assignees
Milestone

Comments

@dbrodie
Copy link

@dbrodie dbrodie commented Jan 27, 2014

In commit 71f6986 rpm package completions was fixed, a similar fix is needed to be done to yum completions a few lines up.

--- __fish_print_packages.fish  2014-01-27 23:40:33.991162908 +0200
+++ __fish_print_packages.fish.bak  2014-01-27 23:39:29.373575262 +0200
@@ -46,7 +46,7 @@
        end

        # Remove package version information from output and pipe into cache file
-       /usr/share/yum-cli/completion-helper.py list all -d 0 -C | cut -d '.' -f 1 | sed '1d' | sed '/^\s/d' | sed -e 's/$/'\t$package'/' >$cache_file &
+       /usr/share/yum-cli/completion-helper.py list all -d 0 -C >$cache_file | cut -d '.' -f 1 | sed '1d' | sed '/^\s/d' | sed -e 's/$/'\t$package'/' &
    end

    # Rpm is too slow for this job, so we set it up to do completions
@zanchey zanchey closed this in 8fbffe2 Jan 29, 2014
@zanchey
Copy link
Member

@zanchey zanchey commented Jan 29, 2014

As far as I can tell, sed '1d' | sed '/^\s/d' does nothing.

8fbffe2 should fix this; try rm /tmp/.yum-cache.$USER /tmp/.rpm-cache.$USER to force a new cache after installing the corrected version.

@ghost ghost assigned zanchey Jan 29, 2014
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.