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

fish: fix systemctl completion #78422

Closed
wants to merge 1 commit into from

Conversation

edef1c
Copy link
Member

@edef1c edef1c commented Jan 24, 2020

Motivation for this change

Previously, typing a command starting with systemctl would result in an
error:

Integer 243 in '243 (243)' followed by non-digit

Link: fish-shell/fish-shell#5689

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

Previously, typing a command starting with systemctl would result in an
error:

    Integer 243 in '243 (243)' followed by non-digit

Link: fish-shell/fish-shell#5689
@flokli
Copy link
Contributor

flokli commented Jan 24, 2020

@edef1c Shouldn't this be fixed by #77951 already?

@Profpatsch
Copy link
Member

Exactly the same, this one has a better comment, let’s merge that.

@flokli
Copy link
Contributor

flokli commented Jan 26, 2020 via email

@Profpatsch
Copy link
Member

Is this a problem? Just rebase and merge.

@edef1c
Copy link
Member Author

edef1c commented Jan 28, 2020

Exactly the same, this one has a better comment, let’s merge that.

Resolved in private conversation: This was based on a misunderstanding, and #77951 is the preferred one. They both contain the same information—save for the note of what release is expected to contain this—but split differently across the nixpkgs tree and the commit message. Closing this PR.

@edef1c edef1c closed this Jan 28, 2020
@edef1c edef1c deleted the fish-systemctl-completion branch January 28, 2020 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants