* zsh (live version): setting globassign crashes completion
@ 2015-06-02 12:17 Aleksandrina Nikolova
2015-06-02 13:36 ` Peter Stephenson
0 siblings, 1 reply; 3+ messages in thread
From: Aleksandrina Nikolova @ 2015-06-02 12:17 UTC (permalink / raw)
To: zsh-users
Today I installed the latest git commit and was greeted by:
_main_complete:388: _lastcomp: assignment to invalid subscript range
after any completion I tried to do. Removing all configs (global and
local) fixed the problem so I started enabling things one by one to see
what is causing this. Bottom line:
a default setup with only:
autoload -Uz compinit
compinit
setopt globassign
gives the above stated error. Sorry, I don't have time to further
investigate, maybe someone else can look it up.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: zsh (live version): setting globassign crashes completion
2015-06-02 12:17 zsh (live version): setting globassign crashes completion Aleksandrina Nikolova
@ 2015-06-02 13:36 ` Peter Stephenson
2015-06-02 13:45 ` Mikael Magnusson
0 siblings, 1 reply; 3+ messages in thread
From: Peter Stephenson @ 2015-06-02 13:36 UTC (permalink / raw)
To: zsh-users
On Tue, 2 Jun 2015 22:17:02 +1000
Aleksandrina Nikolova <infinite.craziness@gmail.com> wrote:
> Today I installed the latest git commit and was greeted by:
> _main_complete:388: _lastcomp: assignment to invalid subscript range
> after any completion I tried to do. Removing all configs (global and
> local) fixed the problem so I started enabling things one by one to see
> what is causing this. Bottom line:
> a default setup with only:
> autoload -Uz compinit
> compinit
> setopt globassign
> gives the above stated error. Sorry, I don't have time to further
> investigate, maybe someone else can look it up.
We certainly shouldn't ever depend on the setting of the glob_assign
option within the completion scripts, so this fix is a sensible thing to
do anyway and ought to deal with it.
What is actually triggering the error is a bit obscure,
though... nothing obviously relevant has changed. I guess some new and
otherwise unproblematic assignment just picked up this alternative
meaning.
pws
diff --git a/Completion/compinit b/Completion/compinit
index 9470c92..79f9d42 100644
--- a/Completion/compinit
+++ b/Completion/compinit
@@ -142,6 +142,7 @@ _comp_options=(
NO_cshnullglob
NO_cshjunkiequotes
NO_errexit
+ NO_globassign
NO_globsubst
NO_histsubstpattern
NO_ignorebraces
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: zsh (live version): setting globassign crashes completion
2015-06-02 13:36 ` Peter Stephenson
@ 2015-06-02 13:45 ` Mikael Magnusson
0 siblings, 0 replies; 3+ messages in thread
From: Mikael Magnusson @ 2015-06-02 13:45 UTC (permalink / raw)
To: Peter Stephenson; +Cc: Zsh Users
On Tue, Jun 2, 2015 at 3:36 PM, Peter Stephenson
<p.stephenson@samsung.com> wrote:
> On Tue, 2 Jun 2015 22:17:02 +1000
> Aleksandrina Nikolova <infinite.craziness@gmail.com> wrote:
>> Today I installed the latest git commit and was greeted by:
>> _main_complete:388: _lastcomp: assignment to invalid subscript range
>> after any completion I tried to do. Removing all configs (global and
>> local) fixed the problem so I started enabling things one by one to see
>> what is causing this. Bottom line:
>> a default setup with only:
>> autoload -Uz compinit
>> compinit
>> setopt globassign
>> gives the above stated error. Sorry, I don't have time to further
>> investigate, maybe someone else can look it up.
>
> We certainly shouldn't ever depend on the setting of the glob_assign
> option within the completion scripts, so this fix is a sensible thing to
> do anyway and ought to deal with it.
>
> What is actually triggering the error is a bit obscure,
> though... nothing obviously relevant has changed. I guess some new and
> otherwise unproblematic assignment just picked up this alternative
> meaning.
My money is on
"33816, 33819: GLOB_ASSIGN changes integer and floating type variables
to string scalars"
I checked that my patch still passes the test added by those patches,
and it does.
--
Mikael Magnusson
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-06-02 13:45 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-06-02 12:17 zsh (live version): setting globassign crashes completion Aleksandrina Nikolova
2015-06-02 13:36 ` Peter Stephenson
2015-06-02 13:45 ` Mikael Magnusson
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).