Re: [Bash-completion-devel] Towards dynamic completion loading

2011-11-14 Thread Igor Murzov
On Tue, 15 Nov 2011 01:00:13 +0300 Igor Murzov wrote: > Thanks! This fixed the issue for me. > Something is still broken here, though: > > $ nice /us > > works, but > > $ /usr/bin/nice /us > > doesn't. This should be fixed by 6843989. I haven't saw any test regression, so I pushed this t

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-11-14 Thread Igor Murzov
On Mon, 14 Nov 2011 21:54:47 +0200 Ville Skyttä wrote: > On 11/14/2011 10:21 PM, Igor Murzov wrote: > > On Wed, 26 Oct 2011 20:47:43 +0300 > > Ville Skyttä wrote: > > > >> On 10/26/2011 08:10 PM, Ville Skyttä wrote: > >>> On 10/26/2011 10:24 AM, David Paleino wrote: > >>> > Broken dynamic

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-11-14 Thread Ville Skyttä
On 11/14/2011 09:54 PM, Ville Skyttä wrote: > On 11/14/2011 10:21 PM, Igor Murzov wrote: >> On Wed, 26 Oct 2011 20:47:43 +0300 >> Ville Skyttä wrote: >> >>> On 10/26/2011 08:10 PM, Ville Skyttä wrote: On 10/26/2011 10:24 AM, David Paleino wrote: > Broken dynamic loading with meta-com

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-11-14 Thread Ville Skyttä
On 11/14/2011 10:21 PM, Igor Murzov wrote: > On Wed, 26 Oct 2011 20:47:43 +0300 > Ville Skyttä wrote: > >> On 10/26/2011 08:10 PM, Ville Skyttä wrote: >>> On 10/26/2011 10:24 AM, David Paleino wrote: >>> Broken dynamic loading with meta-commands? >>> >>> Yes, definitely, thanks for pointing

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-11-14 Thread Igor Murzov
On Wed, 26 Oct 2011 20:47:43 +0300 Ville Skyttä wrote: > On 10/26/2011 08:10 PM, Ville Skyttä wrote: > > On 10/26/2011 10:24 AM, David Paleino wrote: > > > >> Broken dynamic loading with meta-commands? > > > > Yes, definitely, thanks for pointing it out - I haven't considered this > > case at a

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-26 Thread Ville Skyttä
On 10/22/2011 12:57 PM, Ville Skyttä wrote: > Testsuite still needs to be fixed, Done in commit 6e08dc5. ___ Bash-completion-devel mailing list Bash-completion-devel@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/bash-c

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-26 Thread Ville Skyttä
On 10/26/2011 08:10 PM, Ville Skyttä wrote: > On 10/26/2011 10:24 AM, David Paleino wrote: > >> Broken dynamic loading with meta-commands? > > Yes, definitely, thanks for pointing it out - I haven't considered this > case at all but I'm looking into it right now. Should be fixed in commit 22f888

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-26 Thread David Paleino
On Wed, 26 Oct 2011 19:33:48 +0300, Ville Skyttä wrote: > On 10/26/2011 10:40 AM, David Paleino wrote: > > > One more thing: even with the apt-get completion loaded (see my previous > > post), I get: > > > > $ sudo apt-get --purge remove bash: _comp_dpkg_installed_packages: > > command not found

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-26 Thread Ville Skyttä
On 10/26/2011 10:24 AM, David Paleino wrote: > Broken dynamic loading with meta-commands? Yes, definitely, thanks for pointing it out - I haven't considered this case at all but I'm looking into it right now. ___ Bash-completion-devel mailing list Bash

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-26 Thread Ville Skyttä
On 10/26/2011 10:40 AM, David Paleino wrote: > One more thing: even with the apt-get completion loaded (see my previous > post), > I get: > > $ sudo apt-get --purge remove bash: _comp_dpkg_installed_packages: > command > not found > > I see that _comp_dpkg_installed_packages exists in completi

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-26 Thread David Paleino
On Sat, 22 Oct 2011 12:57:49 +0300, Ville Skyttä wrote: > On 10/20/2011 09:04 PM, David Paleino wrote: > > On Thu, 20 Oct 2011 20:47:45 +0300, Ville Skyttä wrote: > >> > >> No comments yet? If there are none or there are no strong objections > >> soon, I'm going to merge this to master soon becau

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-26 Thread David Paleino
On Sat, 22 Oct 2011 12:57:49 +0300, Ville Skyttä wrote: > On 10/20/2011 09:04 PM, David Paleino wrote: > > On Thu, 20 Oct 2011 20:47:45 +0300, Ville Skyttä wrote: > >> > >> No comments yet? If there are none or there are no strong objections > >> soon, I'm going to merge this to master soon becau

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-24 Thread Igor Murzov
> > Also how you want to distinguish among different programs with the same > > name? Is the path to a binary really makes any sense for this? > > For some case such as init.d scripts it does, for many others it > doesn't. Anyway, as said earlier, we don't "really need" this at the > moment becau

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-24 Thread Ville Skyttä
On 10/24/2011 01:21 AM, Igor Murzov wrote: > Also how you want to distinguish among different programs with the same name? > Is the path to a binary really makes any sense for this? For some case such as init.d scripts it does, for many others it doesn't. Anyway, as said earlier, we don't "real

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-24 Thread Ville Skyttä
On 10/24/2011 01:29 AM, Igor Murzov wrote: >> But then again, completing variable names doesn't really work in the >> master branch either for commands we install completions for. > > At least `echo $B` works as expected in the master branch. Yes, but please note that I said "for commands we ins

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-23 Thread Igor Murzov
> > Found a regression. Completing shell variables names no longer works. > > > > $ echo $B > > > > gives me > > > > $ echo \$BASH > > > > for "dynamic-loading" branch. Further hits of tab button give no effect. > But then again, completing variable names doesn't really work in the > master

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-23 Thread Igor Murzov
> > Why try to load different completions for the same program. > > Because it might not be the same program; for example we want different > completions for /etc/init.d/mysql and /usr/bin/mysql. Ok, if you really need this, then we should work with real path here at least. Using relative paths

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-23 Thread Ville Skyttä
On 10/23/2011 10:41 AM, Ville Skyttä wrote: > On 10/23/2011 03:08 AM, Igor Murzov wrote: >> Found a regression. Completing shell variables names no longer works. >> $ echo $B >> gives me >> $ echo \$BASH >> for "dynamic-loading" branch. Further hits of tab button give no effect. > > I don't know

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-23 Thread Ville Skyttä
On 10/23/2011 02:08 AM, Igor Murzov wrote: > Is backward compatibility necessary? I suppose that if we raise major > version number, then we can drop this compatibility and remove all > deprecated functions :) This will make bash_completion slimmer. I think this is not something that we should do

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-23 Thread Ville Skyttä
On 10/23/2011 03:08 AM, Igor Murzov wrote: > Found a regression. Completing shell variables names no longer works. > $ echo $B > gives me > $ echo \$BASH > for "dynamic-loading" branch. Further hits of tab button give no effect. I don't know if there's much/anything we can do about this. _comple

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-23 Thread Ville Skyttä
On 10/23/2011 01:19 AM, Igor Murzov wrote: > On Thu, 20 Oct 2011 20:47:45 +0300 > Ville Skyttä wrote: >> No comments yet? If there are none or there are no strong objections >> soon, I'm going to merge this to master soon > > I have tested dynamic loading now. That's what I don't like or underst

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-22 Thread Igor Murzov
Found a regression. Completing shell variables names no longer works. $ echo $B gives me $ echo \$BASH for "dynamic-loading" branch. Further hits of tab button give no effect. -- Igor ___ Bash-completion-devel mailing list Bash-completion-devel@lists

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-22 Thread Igor Murzov
On Sat, 22 Oct 2011 13:00:57 +0300 Ville Skyttä wrote: > On 10/03/2011 04:03 AM, Igor Murzov wrote: > > And maybe it > > is a good idea to put functions used by more than one completion to its > > own file. > > No objections, but on the other hand no hurry either as far as I'm > concerned. Mayb

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-22 Thread Igor Murzov
On Thu, 20 Oct 2011 20:47:45 +0300 Ville Skyttä wrote: > No comments yet? If there are none or there are no strong objections > soon, I'm going to merge this to master soon I have tested dynamic loading now. That's what I don't like or understand: 1. ># If full path below completions dir ex

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-22 Thread David Paleino
On Sat, 22 Oct 2011 12:57:49 +0300, Ville Skyttä wrote: > On 10/20/2011 09:04 PM, David Paleino wrote: > > On Thu, 20 Oct 2011 20:47:45 +0300, Ville Skyttä wrote: > >> > >> No comments yet? If there are none or there are no strong objections > >> soon, I'm going to merge this to master soon becau

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-22 Thread Ville Skyttä
On 10/03/2011 04:03 AM, Igor Murzov wrote: > And maybe it > is a good idea to put functions used by more than one completion to its > own file. No objections, but on the other hand no hurry either as far as I'm concerned. Maybe introduce a "functions" dir where these files would be placed, and a

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-22 Thread Ville Skyttä
On 10/20/2011 09:04 PM, David Paleino wrote: > On Thu, 20 Oct 2011 20:47:45 +0300, Ville Skyttä wrote: >> >> No comments yet? If there are none or there are no strong objections >> soon, I'm going to merge this to master soon because the way it >> reorganizes some things makes it hard to merge cha

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-20 Thread David Paleino
On Thu, 20 Oct 2011 20:47:45 +0300, Ville Skyttä wrote: > On 10/18/2011 11:40 PM, Ville Skyttä wrote: > > On 09/25/2011 10:07 PM, Ville Skyttä wrote: > >> Hello, > >> > >> Even though we don't have a roadmap item for dynamic completion loading > >> (bash 4+'s complete -D), I'd like to start implem

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-20 Thread Ville Skyttä
On 10/18/2011 11:40 PM, Ville Skyttä wrote: > On 09/25/2011 10:07 PM, Ville Skyttä wrote: >> Hello, >> >> Even though we don't have a roadmap item for dynamic completion loading >> (bash 4+'s complete -D), I'd like to start implementing it. > > Okay, first cut of this is now done; I've just pushed

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-18 Thread Ville Skyttä
On 10/18/2011 11:40 PM, Ville Skyttä wrote: > On 09/25/2011 10:07 PM, Ville Skyttä wrote: >> Hello, >> >> Even though we don't have a roadmap item for dynamic completion loading >> (bash 4+'s complete -D), I'd like to start implementing it. > > Okay, first cut of this is now done; I've just pushed

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-18 Thread Ville Skyttä
On 09/25/2011 10:07 PM, Ville Skyttä wrote: > Hello, > > Even though we don't have a roadmap item for dynamic completion loading > (bash 4+'s complete -D), I'd like to start implementing it. Okay, first cut of this is now done; I've just pushed a new branch "dynamic-loading" containing this stuff

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-10-02 Thread Igor Murzov
> Anything else? I have a bunch of completions that don't need anything > from other files already split up locally (see 1) above) and unless > there are objections, I'm going to commit that to master soon. Ok, I understood everything now :) No objections from me. But testing of all that dynamic

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-09-26 Thread Ville Skyttä
On 09/26/2011 02:46 AM, Igor Murzov wrote: >> 1) Start splitting stuff in completions/ into smaller, preferably >> per-command files. This is quite straightforward step for a lot of >> cases, and shouldn't be that intrusive change even though the rest of >> the steps would for some reason be delay

Re: [Bash-completion-devel] Towards dynamic completion loading

2011-09-25 Thread Igor Murzov
> 1) Start splitting stuff in completions/ into smaller, preferably > per-command files. This is quite straightforward step for a lot of > cases, and shouldn't be that intrusive change even though the rest of > the steps would for some reason be delayed. Can you explain why this step is necessary