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

Contribution: gridcoinresearchd BASH completion #1410

Open
smoe opened this issue Feb 28, 2019 · 13 comments
Open

Contribution: gridcoinresearchd BASH completion #1410

smoe opened this issue Feb 28, 2019 · 13 comments
Assignees
Milestone

Comments

@smoe
Copy link
Contributor

smoe commented Feb 28, 2019

It may be beneficial for users to have the BASH command line completion support for gridcoinresearchd. This gist
https://gist.github.com/smoe/0cfa4ad8982b55f1ff30c40ad1593bc0
has an implementation.

@RoboticMind
Copy link
Contributor

Feel like this should probably go somewhere else besides being in a GitHub issue. Not sure where exactly it would fit best but probably not in an issue here.

@smoe
Copy link
Contributor Author

smoe commented May 13, 2019

Thank you for your support. I had already forgotten about it. Maybe someone could ping the individual providing the Debian package about it? With a bit of look someone knows where this fits and transforms it into a pull request.

@RoboticMind
Copy link
Contributor

@caraka is the builder of the Debian package. @caraka See the messages above for more information

@caraka
Copy link
Member

caraka commented May 13, 2019

Thanks @smoe I hadn't seen this. IIRC, it needs a separate 'package' that gets pulled in when gridcoinresearchd is installed. It won't be immediate, but I will look into how this is normally done. Thanks for your contribution! I tire kicked this idea about 4 years ago when I first started building, and put it into the too hard bin in part because our RPC stuff was such a mess. With our recently purged/cleaned RPC commands, this is timely.

@smoe
Copy link
Contributor Author

smoe commented May 14, 2019

This is nice to hear, @caraka . Many thanks. This was my very first shot at bash completions, so there may be a point to have this reviewed by someone more experienced. I don't think this merrits a separate package, really. I propose upstream (you?) to maintain it somewhere in the gridcoinresearchd's source tree and the Ubuntu/SuSE/RedHat/Centos/... packaging places that file in /etc/bash-completion.d/ .

@denravonska denravonska added this to the Elizabeth milestone May 14, 2019
@smoe
Copy link
Contributor Author

smoe commented May 14, 2019

Btw, I am one of the co-maintainers of the Debian BOINC package (https://salsa.debian.org/pkg-boinc-team/boinc). If you want something sponsored or if there is something else that Debian should do for you - please tell me. Also, having no idea if you currently have a repository for the Ubuntu packaging somewhere - you are most cordially invited to join the https://salsa.debian.org/pkg-boinc-team.

@jamescowens jamescowens modified the milestones: Elizabeth, Fern Jul 6, 2019
@jamescowens jamescowens modified the milestones: Fern, Gladys Aug 3, 2019
@RoboticMind
Copy link
Contributor

Maybe this could go in the contrib folder somewhere?

@jamescowens jamescowens modified the milestones: Gladys, Hilda Oct 27, 2020
@jamescowens
Copy link
Member

Yes. And this file simply needs to be placed in /etc/bash.completion.d and logout login (to source the file) for it to become active. We ought to be able to do this in the normal packages and the install command of make.

@jamescowens
Copy link
Member

jamescowens commented Dec 5, 2020

@caraka I have assigned this to you for the Debian/Ubuntu builds. I don't think this requires a separate package. We probably should detect whether the standard bash-completion package is already installed as a precedent.

@jamescowens jamescowens modified the milestones: Hilda, Ingrid Feb 25, 2021
@jamescowens
Copy link
Member

@caraka, this seems pretty simple to put in the packaging... can we get this done for Ingrid?

@smoe
Copy link
Contributor Author

smoe commented Jun 15, 2021

@caraka I have assigned this to you for the Debian/Ubuntu builds. I don't think this requires a separate package. We probably should detect whether the standard bash-completion package is already installed as a precedent.

The directory /etc/bash_completion.d can exist independently from the bash-completion package.

$ apt-file search /etc/bash_completion.d/
cdcd: /etc/bash_completion.d/cdcd
dlocate: /etc/bash_completion.d/dlocate-completion
git: /etc/bash_completion.d/git-prompt
git-flow: /etc/bash_completion.d/git-flow
gmic: /etc/bash_completion.d/gmic
lxctl: /etc/bash_completion.d/lxctl
...
lxctl: /etc/bash_completion.d/lxctl_vz2lxc
qtltools: /etc/bash_completion.d/QTLtools_bash_autocomplete.bash
rosbash: /etc/bash_completion.d/rosbash
svn-buildpackage: /etc/bash_completion.d/svn-buildpackage

I just wished someone with more experience on these bash-completions would give that patch a peer review.

@jamescowens
Copy link
Member

I am moving this to Janice. It is a nice to have, and we can push it to the next milestone.

@jamescowens jamescowens modified the milestones: Ingrid, Janice Jun 20, 2021
@jamescowens jamescowens modified the milestones: Janice, Kermit's Mom Dec 10, 2021
@jamescowens
Copy link
Member

@caraka ping on this. This is independent enough that this can still go in the Kermit's Mom release.

@jamescowens jamescowens removed this from the Kermit's Mom milestone Jul 27, 2022
@jamescowens jamescowens added this to the LaVerne milestone Jul 27, 2022
@jamescowens jamescowens modified the milestones: LaVerne, Miss Piggy Feb 2, 2023
@jamescowens jamescowens modified the milestones: Miss Piggy, Natasha Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants