We believe that your issue has been closed by the upload of
Version 0.4.2 of grml-x from Michael Prokop <mika@grml.org>.
The explanation is attached below

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Thu, 12 Feb 2009 18:10:53 +0100
Source: grml-x
Binary: grml-x
Architecture: source all
Version: 0.4.2
Distribution: unstable
Urgency: low
Maintainer: Michael Prokop <mika@grml.org>
Changed-By: Michael Prokop <mika@grml.org>
Description: 
 grml-x     - wrapper for startx for running X Window System
Changes: 
 grml-x (0.4.2) unstable; urgency=low
 .
   * Improve zsh completion:
     - use double brackets for the test command
     - make sure the right usage text gets displayed
     - support fallback options
     - improve description of force option
     Thanks to ft and z3ttacht for their work! [Closes: issue398]
   * Use Build-Depends-Indep in debian/control to make lintian happy.
   * Consequently use mktemp for handling tmp files.
   * Use fluxbox for all usage examples in grml-x itself (on grml-medium
     we ship fluxbox only and people might be confused otherwise).
Checksums-Sha1: 
 9075fe3c3ac7d3dc42f312c327d26f9eee136564 863 grml-x_0.4.2.dsc
 1463d4528aae2a8af1f58d538a98f5af9e7b5f92 24719 grml-x_0.4.2.tar.gz
 f9dc1e338b22af3e367ebf5b917ed7dfc3a18f9c 24438 grml-x_0.4.2_all.deb
Checksums-Sha256: 
 2c3f504e13c8fb9fe6bde2c0e1e7303ff1b218d516040b06542a4cdac3ee97ba 863 grml-x_0.4.2.dsc
 4c7d5757c6db9a723c40768e50698d7435f8fa694a9516507b070664783459d4 24719 grml-x_0.4.2.tar.gz
 9505cee0d354a0c7d5850c721bd07d10ed0b9920b6f80ee59eca9f87c70eee23 24438 grml-x_0.4.2_all.deb
Files: 
 694f336d4344fe49934e0ac1d4cd5a65 863 grml optional grml-x_0.4.2.dsc
 c84f0bd7173949abb5c81d75e9f691e5 24719 grml optional grml-x_0.4.2.tar.gz
 00cccf2e7a5c4bd260e30895925c5986 24438 grml optional grml-x_0.4.2_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iD8DBQFJlGLT2N9T+zficugRAgL7AJ9VzuZ7W/CTq8RwfgqUM+KrGN1NrQCaA2Np
viIi7cenwxYIvxwquey1n9w=
=p7Uq
-----END PGP SIGNATURE-----

Michael Prokop <mika@grml.org>:
> * Alexander Steinböck <bts@bts.grml.org> [20080222 23:26]:
> > > '7' and '8' as the only completions for -display? Reason?
> 
> > Afaics the first six are used for tty{1..6}. (/etc/inittab.grml)
> > Maybe the author hasn't had any need for more than two displays so
> > far? But that's just guesses, since I'm not responsible for that
> > entry. (o;
> 
> Yeah, tty's 7 and 8 are known to be available for X on grml and I
> want to make sure grml-x is working OOTB as stable as possible.
> 
> If a user is aware of the fact that tty $NUMBER != [7|8] is
> available for X he can run grml-x the way he likes it to anyway.

This means, the patches I send earlier should do the right thing[tm]
and close this bug.

Regards, Frank

Alexander Steinböck <bts@bts.grml.org>:
> > '7' and '8' as the only completions for -display? Reason?
> Afaics the first six are used for tty{1..6}. (/etc/inittab.grml)
> Maybe the author hasn't had any need for more than two displays so
> far? But that's just guesses, since I'm not responsible for that
> entry. (o;

...I'm almost far enough to check the grml-x code myself. :-)
I thought of $DISPLAY, the tty on the other hand would make sense.

> > > > However, I tend to agree with you. Leave check_com() out.
> > Just for the record:
> >   check_com() is available outside of our zshrc now.
> Great. But as discussed earlier, we still do agree to not use it here,
> don't we?

Yeah, hence I said JFTR. :-)

Regards, Frank

* Alexander Steinböck <bts@bts.grml.org> [20080222 23:26]:

> > '7' and '8' as the only completions for -display? Reason?

> Afaics the first six are used for tty{1..6}. (/etc/inittab.grml)
> Maybe the author hasn't had any need for more than two displays so
> far? But that's just guesses, since I'm not responsible for that
> entry. (o;

Yeah, tty's 7 and 8 are known to be available for X on grml and I
want to make sure grml-x is working OOTB as stable as possible.

If a user is aware of the fact that tty $NUMBER != [7|8] is
available for X he can run grml-x the way he likes it to anyway.

JFYI && regards,
-mika-

* Frank Terbeck <bts@bts.grml.org> [2008-02-22 15:20]:
> However, I think your fix is not the best solution, we can provide. I
> am attaching two exports. Please review. I *think* they are ready for
> inclusion.
Your extensions are really sophisticated. I'm all for them. Didn't
even think to extend the other switches. Have been just fixated on the
'bug' concerning `-help'. (o;

> '7' and '8' as the only completions for -display? Reason?
Afaics the first six are used for tty{1..6}. (/etc/inittab.grml)
Maybe the author hasn't had any need for more than two displays so
far? But that's just guesses, since I'm not responsible for that
entry. (o;

> > > However, I tend to agree with you. Leave check_com() out.
> Just for the record:
>   check_com() is available outside of our zshrc now.
Great. But as discussed earlier, we still do agree to not use it here,
don't we?

Alexander Steinböck <bts@bts.grml.org>:
> Alexander Steinböck <z3ttacht@grml.org> added the comment:
> * Frank Terbeck <bts@bts.grml.org> [2008-02-06 12:25]:
> > > And `help' is completing with brackets attached.
> > > Culprit's the leading colon.
> > Um, I just tried the completion. What is your problem with the way it
> > currently works? I don't see any 'brackets attached'.
> When completing, I get
> 
>      % grml-x -<TAB>
>      Completing - option
>      [...]
>      -help         -- [display help]
>      [...]
> 
> because in _grml-x there's defined:
> 
>     '-help:[display help]'

Yeah, well. I tried this with my zsh setup, which obviously does some
things differently then grml's setup. I am able to reproduce the
problem, using grml's setup.

However, I think your fix is not the best solution, we can provide. I
am attaching two exports. Please review. I *think* they are ready for
inclusion.

One minor bit, which requires fixing, too (I guess):

[snip]
-  '-display:display for xserver:(7 8)'
+  '-display[display for xserver]:display:(7 8)'
[snap]

'7' and '8' as the only completions for -display? Reason?


> > check_com() *would* be available for completions, too. I mean, in what
> > environment are you using compsys *without* being in an interactive
> > shell?
> Right.
> 
> > However, I tend to agree with you. Leave check_com() out.
> I guess, I did mean it the same way, but you obviously managed to
> explain it in a  more understandable way.

Just for the record:
  check_com() is available outside of our zshrc now.

Regards, Frank

-- 
In protocol design, perfection has been reached not when there is
nothing left to add, but when there is nothing left to take away.
                                                  -- RFC 1925

* Frank Terbeck <bts@bts.grml.org> [2008-02-06 13:32]:
> > check_com() *would* be available for completions, too. I mean, in what
> Wrong wrong wrong wrong...
> check_com() is currently *not* available outside of our zshrc.
> This has nothing to do with the (non-)interactive state of the shell,
> though. We are unfunctioning the function definition after the zshrc
> is sourced.
I knew it!!!1oneelven! :p

* Frank Terbeck <bts@bts.grml.org> [2008-02-06 12:25]:
> > And `help' is completing with brackets attached.
> > Culprit's the leading colon.
> Um, I just tried the completion. What is your problem with the way it
> currently works? I don't see any 'brackets attached'.
When completing, I get

     % grml-x -<TAB>
     Completing - option
     [...]
     -help         -- [display help]
     [...]

because in _grml-x there's defined:

    '-help:[display help]'

And according to the manual and my understanding, the colon's wrong
here.
    ,----[ man 1 zshcompsys | less -p '^\s+_arguments' ]-
    | optspec[explanation]
    | An  explanation  string may be appended to any of
    | the preceding forms of optspec by enclosing it in
    | brackets, as in ‘-q[query operation]’.
    | Just checked out the hg Repository again.
    | whereas a colon is, afaics, used to separate for messages and
    | actions.
    `----

Just checked out the hg Repository again.

> check_com() *would* be available for completions, too. I mean, in what
> environment are you using compsys *without* being in an interactive
> shell?
Right.

> However, I tend to agree with you. Leave check_com() out.
I guess, I did mean it the same way, but you obviously managed to
explain it in a  more understandable way.

---------------
P.S.: I marginally changed the diff concerning the brackets, because I
just found a 'typo'. It's the bracket at the end.

  % -mode   -- resolution-mode for xserver (<width>x<height> - % e.g. 1024x768)]

Frank Terbeck <ft@bewatermyfriend.org>:
[...]
> check_com() *would* be available for completions, too. I mean, in what
[...]

Wrong wrong wrong wrong...
check_com() is currently *not* available outside of our zshrc.
This has nothing to do with the (non-)interactive state of the shell,
though. We are unfunctioning the function definition after the zshrc
is sourced.

Regards, Frank

Alexander Steinböck <bts@bts.grml.org>:
> In grml-x' completion there's the switch `fallback' missing. As well
> as in the manpage. And `help' is completing with brackets attached.
> Culprit's the leading colon.

Um, I just tried the completion. What is your problem with the way it
currently works? I don't see any 'brackets attached'.

> Furthermore, done in a separate diff, I guess we should stick to
> double brackets for the test command here as well.

ACK.

> I first wanted to use check_com instead of `[[ -x ... ]]', but then
> realised, that's not available in noninteractive shells. Besides,
> it may be better to stay independent from grml's zsh-config for those
> (non-grml) users, who just checkout the completion. Or am I missing
> something in this regard?

Well, no. You are right. More or less.

check_com() *would* be available for completions, too. I mean, in what
environment are you using compsys *without* being in an interactive
shell?

However, I tend to agree with you. Leave check_com() out.
The reason it got implemented was to have a simple uniform way to
check for commands/aliases/functions etc. in various places in our
zshrc - because we check for more then one program in there. :-)


If you could the explain the 'brackets attached' thing above, I'd be
happy to apply this one. (A first quick look didn't reveal any
breakage).

Regards, Frank

-- 
In protocol design, perfection has been reached not when there is
nothing left to add, but when there is nothing left to take away.
                                                  -- RFC 1925

In grml-x' completion there's the switch `fallback' missing. As well
as in the manpage.
And `help' is completing with brackets attached. Culprit's the leading
colon.

Furthermore, done in a separate diff, I guess we should stick to
double brackets for the test command here as well.
I first wanted to use check_com instead of `[[ -x ... ]]', but then
realised, that's not available in noninteractive shells. Besides,
it may be better to stay independent from grml's zsh-config for those
(non-grml) users, who just checkout the completion. Or am I missing
something in this regard?