[Bash-completion-devel] [PATCH] check if ruptime is installed

isabella parakiss izaberina at gmail.com
Sat Jun 13 04:56:07 UTC 2015


Ok this is not exactly the answer I was hoping to receive.

On 6/12/15, Ville Skyttä <ville.skytta at iki.fi> wrote:
> I'm going to see if the approach I proposed works sometime, when I
> find time and interest, and helping with testing would quite likely
> make that happen sooner. Then, if it does work, I'll probably spend
> time documenting it in bash-completion's README as well as submit a
> related fix to the Fedora PackageKit-command-not-found package. I
> suppose that's about as far as I'm prepared to go. I suggest other
> distro users and maintainers to do the same thing for the environments
> they know and can control.
>

Brilliant idea, but I'd rather fix *your* code than force everyone else in
the world to adapt.  There's no reason to forbid people to use a bash
feature in completions just to be lazy.

> You are entitled to your opinion wrt what's sloppy etc, but that tone
> won't help with getting me to spend my personal time fixing something
> I don't consider necessarily broken, doesn't affect me, and is
> hard/messy/semi-unmaintainable to fix everywhere in bash-completion
> code whereas it could be trivial to fix/workaround in command not
> found handlers. And taking care of it there would do it also for other
>

I sent you a patch.  I did this so that you didn't have to spend much time
fixing it.  I'm willing to help, I went through the whole git repo to find
and fix all (or most of) the occurrencies.  See the attached patch.



---
xoxo iza
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Check-if-commands-exist-before-using-them.patch
Type: application/octet-stream
Size: 91723 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/bash-completion-devel/attachments/20150613/c0e2ac1e/attachment-0001.obj>


More information about the Bash-completion-devel mailing list