compaudit problem

compaudit problem

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 338 bytes --]

Hello,

Recent zsh snapshots from cvs (i.e. 4.3.2-20060729 and 4.3.2-20060817)
have the following problem:

compaudit:50: bad substitution
compinit:469: bad pattern: _i_test[=yes
colors:39: bad substitution
at@localhost ~ $ 

The error messages are issued at zsh startup.  4.3.2-20060619 snapshot
did not have such a problem.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: compaudit problem

[Bart Schaefer]

On Aug 19,  3:50pm, Alexey Tourbin wrote:
}
} compaudit:50: bad substitution

That would be on this line:

  local _compdir=${fpath[(r)*/$ZSH_VERSION/*]}

} compinit:469: bad pattern: _i_test[=yes

Interesting; that's on:

      _i_test[$_i_name]=yes

} colors:39: bad substitution

Maybe zsh has a problem with function names that begin with "co". :-)
Seriously, this is pretty clearly a problem with subscript parsing.

for k in ${(k)color}; do color[${color[$k]}]=$k; done

} The error messages are issued at zsh startup.  4.3.2-20060619 snapshot
} did not have such a problem.

In all cases the parse of the subscript appears to have run one byte
too far and consumed the ']'.  What operating system and compiler are
you using?

I can't reproduce any of these with my build of 08/17 sources from CVS.
I've tried compiling both with and without multibyte support.  I've even
tried forcing Test/D07multibyte.ztst to use ru_RU.UTF-8 as $LANG when
running "make check".

Does anything change if you run "zsh -o posix_identifiers"?  Have you
tried building with "configure --disable-multibyte"?

Changes between 06/19 and 07/29 that may have affected this:

        * 22518: Src/params.c, Src/utils,c, Src/zsh.h,
        Test/B02typeset.ztst: Initial go at making parameter subscripts
        use multibyte characters.

        * 22524: Src/params.c, Test/D07multibyte.ztst: searchable
        subscripts with multibyte characters.

        * 22544: README, Doc/Zsh/options.yo, Src/builtin.c, Src/glob.c,
        Src/lex.c, Src/math.c, Src/module.c, Src/options.c, Src/params.c,
        Src/parse.c, Src/subst.c, Src/utils.c, Src/zsh.h, Src/ztype.h,
        Src/Zle/compcore.c, Src/Zle/zle_tricky.c, Test/D07multibyte.ztst:
        Improve use of ztype tests for multibyte characters.  Add
        POSIX_IDENTIFIERS option to control allowability of multibyte
        alphanumeric characters in parameter and module names.

        * 22556: Doc/Zsh/builtins.yo, Functions/Zle/insert-composed-char,
        Src/builtin.c, Src/pattern.c, Src/subst.c, Src/utils.c, Src/zsh.h,
        Src/ztype.h, Src/Zle/zle.h, Src/Zle/zle_main.c,
        Test/D04parameter.ztst, Test/D07multibyte.ztst: Multibyte
        separators and delimiters.

        * unposted: Src/pattern.c, Src/utils.c: minor typos in
        22556 found when MULTIBYTE_SUPPORT is not defined.

        * 22557: Doc/Zsh/options.yo, Misc/globtests, Src/options.c,
        Src/pattern.c, Test/D02glob.ztst, Test/D07multibyte.ztst:
        Turn on multibyte option by default for MULTIBYTE_SUPPORT and fix
        tests and patterns.


-- 

Re: compaudit problem

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 1179 bytes --]

On Sat, Aug 19, 2006 at 10:30:33AM -0700, Bart Schaefer wrote:
> Seriously, this is pretty clearly a problem with subscript parsing.
> 
> for k in ${(k)color}; do color[${color[$k]}]=$k; done
> 
> } The error messages are issued at zsh startup.  4.3.2-20060619 snapshot
> } did not have such a problem.
> 
> In all cases the parse of the subscript appears to have run one byte
> too far and consumed the ']'.  What operating system and compiler are
> you using?

Linux 2.6.16
glibc 2.3.6
gcc 4.1.1 mostly in sync with gcc-4.1.1-6 from Fedora

> I can't reproduce any of these with my build of 08/17 sources from CVS.
> I've tried compiling both with and without multibyte support.  I've even
> tried forcing Test/D07multibyte.ztst to use ru_RU.UTF-8 as $LANG when
> running "make check".

make check works fine but the problem arise after installation when I
launch new zsh in my screen(1) session.  So I rolled back to 20060619
(not only rolled back but also recompiled) and it works just fine.

> Does anything change if you run "zsh -o posix_identifiers"?  Have you
> tried building with "configure --disable-multibyte"?

Nope, I'll try just now.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: compaudit problem

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 272 bytes --]

On Sat, Aug 19, 2006 at 10:30:33AM -0700, Bart Schaefer wrote:
> Does anything change if you run "zsh -o posix_identifiers"?  Have you
> tried building with "configure --disable-multibyte"?

posix_identifiers doesn't help, but --disable-multibyte makes things
work again.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: compaudit problem

[Bart Schaefer]

On Aug 19,  9:45pm, Alexey Tourbin wrote:
}
} Linux 2.6.16
} glibc 2.3.6
} gcc 4.1.1 mostly in sync with gcc-4.1.1-6 from Fedora

I have 2.6.9, 2.3.4, 3.4.5 (RHEL4).  I wonder if there's something
about gcc-4 that's an issue.  I've just installed Xcode on my Mac,
which has gcc-4, so perhaps I'll attempt a zsh compile on that.

Re: compaudit problem

[Peter Stephenson]

On Sat, 19 Aug 2006 22:00:46 +0400
Alexey Tourbin <at@altlinux.ru> wrote:
> On Sat, Aug 19, 2006 at 10:30:33AM -0700, Bart Schaefer wrote:
> > Does anything change if you run "zsh -o posix_identifiers"?  Have you
> > tried building with "configure --disable-multibyte"?
> 
> posix_identifiers doesn't help, but --disable-multibyte makes things
> work again.

Do you have multibyte characters in files in your $fpath?  I did a quick
sanity check by creating ~/tmp/cyrillic/тест/_тест with
"#compdef тест" in it and putting the directory in my fpath, but it
seemed to work OK.

Can you put "set -x" in compinit and see what it's doing when it
complains?

-- 
Peter Stephenson <p.w.stephenson@ntlworld.com>
Web page now at http://homepage.ntlworld.com/p.w.stephenson/

Re: compaudit problem

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 25878 bytes --]

On Sun, Aug 20, 2006 at 06:16:02PM +0100, Peter Stephenson wrote:
> On Sat, 19 Aug 2006 22:00:46 +0400
> Alexey Tourbin <at@altlinux.ru> wrote:
> > On Sat, Aug 19, 2006 at 10:30:33AM -0700, Bart Schaefer wrote:
> > > Does anything change if you run "zsh -o posix_identifiers"?  Have you
> > > tried building with "configure --disable-multibyte"?
> > 
> > posix_identifiers doesn't help, but --disable-multibyte makes things
> > work again.
> 
> Do you have multibyte characters in files in your $fpath?  I did a quick
> sanity check by creating ~/tmp/cyrillic/тест/_тест with
> "#compdef тест" in it and putting the directory in my fpath, but it
> seemed to work OK.

No, I don't use non-ascii paths or commands.

> Can you put "set -x" in compinit and see what it's doing when it
> complains?

Here is full -x transcript, maybe it can shed light.

at@localhost ~ 3 $ zsh -x
+/etc/zshenv:9> [ -f /etc/profile.d/tmpdir.sh ']'
+/etc/zshenv:9> . /etc/profile.d/tmpdir.sh
+/etc/profile.d/tmpdir.sh:4> [ -z /tmp/.private/at ']'
+/etc/profile.d/tmpdir.sh:4> [ ! -d /tmp/.private/at -o ! -w /tmp/.private/at ']'
+/etc/zshenv:10> TMPPREFIX=/tmp/.private/at/zsh
+/etc/zshrc:11> emulate sh
+/etc/zshrc:12> [ -f /etc/profile ']'
+/etc/zshrc:12> . /etc/profile
+/etc/profile:8> ulimit -Sc 0
+/etc/profile:10> umask 022
+/etc/profile:12> [ -n /home/at/cvs/qa/qa-robot:/home/at/bin:/bin:/usr/bin:/usr/X11R6/bin:/usr/local/bin:/usr/games:/sbin:/usr/sbin:/usr/local/sbin:/usr/lib/qt3/bin ']'
+/etc/profile:13> [ /home/at/cvs/qa/qa-robot:/home/at/bin:/bin:/usr/bin:/usr/X11R6/bin:/usr/local/bin:/usr/games:/sbin:/usr/sbin:/usr/local/sbin:/usr/lib/qt3/bin '=' /home/at/bin -o -z /home/at/cvs/qa/qa-robot:/home/at/bin:/bin:/usr/bin:/usr/X11R6/bin:/usr/local/bin:/usr/games:/sbin:/usr/sbin:/usr/local/sbin:/usr/lib/qt3/bin -o -z /home/at/cvs/qa/qa-robot:/home/at/bin:/bin:/usr/bin:/usr/X11R6/bin:/usr/local/bin:/usr/games:/sbin:/usr/sbin:/usr/local/sbin:/usr/lib/qt3/bin -o -z '' ']'
+/etc/profile:16> [ /home/at/cvs/qa/qa-robot:/home/at/bin:/bin:/usr/bin:/usr/X11R6/bin:/usr/local/bin:/usr/games:/sbin:/usr/sbin:/usr/local/sbin:/usr/lib/qt3/bin '=' /usr/X11R6/bin -o -z /home/at/cvs/qa/qa-robot:/home/at/bin:/bin:/usr/bin:/usr/X11R6/bin:/usr/local/bin:/usr/games:/sbin:/usr/sbin:/usr/local/sbin:/usr/lib/qt3/bin -o -z /home/at/cvs/qa/qa-robot:/home/at/bin:/bin:/usr/bin:/usr/X11R6/bin:/usr/local/bin:/usr/games:/sbin:/usr/sbin:/usr/local/sbin:/usr/lib/qt3/bin -o -z '' ']'
+/etc/profile:19> [ /home/at/cvs/qa/qa-robot:/home/at/bin:/bin:/usr/bin:/usr/X11R6/bin:/usr/local/bin:/usr/games:/sbin:/usr/sbin:/usr/local/sbin:/usr/lib/qt3/bin '=' /usr/games -o -z /home/at/cvs/qa/qa-robot:/home/at/bin:/bin:/usr/bin:/usr/X11R6/bin:/usr/local/bin:/usr/games:/sbin:/usr/sbin:/usr/local/sbin:/usr/lib/qt3/bin -o -z /home/at/cvs/qa/qa-robot:/home/at/bin:/bin:/usr/bin:/usr/X11R6/bin:/usr/local/bin:/usr/games:/sbin:/usr/sbin:/usr/local/sbin:/usr/lib/qt3/bin -o -z '' ']'
+/etc/profile:22> export PATH
+/etc/profile:24> [ -n at ']'
+/etc/profile:25> [ -n at ']'
+/etc/profile:26> [ -n /var/mail/at ']'
+/etc/profile:27> export USER LOGNAME MAIL
+/etc/profile:29> HOSTNAME=+/etc/profile:1> /bin/hostname
+/etc/profile:29> HOSTNAME=localhost.localdomain
+/etc/profile:30> HISTSIZE=999
+/etc/profile:31> HISTFILESIZE=9999
+/etc/profile:32> export HOSTNAME HISTSIZE HISTFILESIZE
+/etc/profile:34> f=/etc/profile.d/0lang.sh
+/etc/profile:35> [ -f /etc/profile.d/0lang.sh -a -r /etc/profile.d/0lang.sh -a -x /etc/profile.d/0lang.sh -a -s /etc/profile.d/0lang.sh -a ! -L /etc/profile.d/0lang.sh ']'
+/etc/profile:36> . /etc/profile.d/0lang.sh
+/etc/profile.d/0lang.sh:4> sourced=''
+/etc/profile.d/0lang.sh:5> f=/home/at/.i18n
+/etc/profile.d/0lang.sh:6> [ -s /home/at/.i18n ']'
+/etc/profile.d/0lang.sh:6> . /home/at/.i18n
+/home/at/.i18n:1> LANG=ru_RU.CP1251
+/home/at/.i18n:2> SYSFONTACM=cp1251
+/home/at/.i18n:3> SYSFONT=UniCyr_8x16
+/home/at/.i18n:4> DICTIONARY=russianw
+/home/at/.i18n:5> MPAGE=-CCP1251
+/home/at/.i18n:6> export DICTIONARY MPAGE
+/home/at/.i18n:7> LC_MESSAGES=C
+/home/at/.i18n:8> LC_TIME=C
+/home/at/.i18n:9> LC_COLLATE=C
+/home/at/.i18n:10> LC_NUMERIC=C
+/etc/profile.d/0lang.sh:7> sourced=1
+/etc/profile.d/0lang.sh:8> break
+/etc/profile.d/0lang.sh:12> unset f
+/etc/profile.d/0lang.sh:27> [ -n 1 ']'
+/etc/profile.d/0lang.sh:28> CondSet LANGUAGE LC_ADDRESS LC_COLLATE LC_CTYPE LC_IDENTIFICATION LC_MEASUREMENT LC_MESSAGES LC_MONETARY LC_NAME LC_NUMERIC LC_PAPER LC_TELEPHONE LC_TIME
+CondSet:2> local n
+CondSet:3> n=LANGUAGE
+CondSet:1> eval 'echo "$LANGUAGE"'
+CondSet:1> echo ''
+CondSet:4> [ -n '' ']'
+CondSet:4> Unset LANGUAGE
+Unset:2> unset LANGUAGE
+CondSet:3> n=LC_ADDRESS
+CondSet:1> eval 'echo "$LC_ADDRESS"'
+CondSet:1> echo ''
+CondSet:4> [ -n '' ']'
+CondSet:4> Unset LC_ADDRESS
+Unset:2> unset LC_ADDRESS
+CondSet:3> n=LC_COLLATE
+CondSet:1> eval 'echo "$LC_COLLATE"'
+CondSet:1> echo C
+CondSet:4> [ -n C ']'
+CondSet:4> export LC_COLLATE
+CondSet:3> n=LC_CTYPE
+CondSet:1> eval 'echo "$LC_CTYPE"'
+CondSet:1> echo ''
+CondSet:4> [ -n '' ']'
+CondSet:4> Unset LC_CTYPE
+Unset:2> unset LC_CTYPE
+CondSet:3> n=LC_IDENTIFICATION
+CondSet:1> eval 'echo "$LC_IDENTIFICATION"'
+CondSet:1> echo ''
+CondSet:4> [ -n '' ']'
+CondSet:4> Unset LC_IDENTIFICATION
+Unset:2> unset LC_IDENTIFICATION
+CondSet:3> n=LC_MEASUREMENT
+CondSet:1> eval 'echo "$LC_MEASUREMENT"'
+CondSet:1> echo ''
+CondSet:4> [ -n '' ']'
+CondSet:4> Unset LC_MEASUREMENT
+Unset:2> unset LC_MEASUREMENT
+CondSet:3> n=LC_MESSAGES
+CondSet:1> eval 'echo "$LC_MESSAGES"'
+CondSet:1> echo C
+CondSet:4> [ -n C ']'
+CondSet:4> export LC_MESSAGES
+CondSet:3> n=LC_MONETARY
+CondSet:1> eval 'echo "$LC_MONETARY"'
+CondSet:1> echo ''
+CondSet:4> [ -n '' ']'
+CondSet:4> Unset LC_MONETARY
+Unset:2> unset LC_MONETARY
+CondSet:3> n=LC_NAME
+CondSet:1> eval 'echo "$LC_NAME"'
+CondSet:1> echo ''
+CondSet:4> [ -n '' ']'
+CondSet:4> Unset LC_NAME
+Unset:2> unset LC_NAME
+CondSet:3> n=LC_NUMERIC
+CondSet:1> eval 'echo "$LC_NUMERIC"'
+CondSet:1> echo C
+CondSet:4> [ -n C ']'
+CondSet:4> export LC_NUMERIC
+CondSet:3> n=LC_PAPER
+CondSet:1> eval 'echo "$LC_PAPER"'
+CondSet:1> echo ''
+CondSet:4> [ -n '' ']'
+CondSet:4> Unset LC_PAPER
+Unset:2> unset LC_PAPER
+CondSet:3> n=LC_TELEPHONE
+CondSet:1> eval 'echo "$LC_TELEPHONE"'
+CondSet:1> echo ''
+CondSet:4> [ -n '' ']'
+CondSet:4> Unset LC_TELEPHONE
+Unset:2> unset LC_TELEPHONE
+CondSet:3> n=LC_TIME
+CondSet:1> eval 'echo "$LC_TIME"'
+CondSet:1> echo C
+CondSet:4> [ -n C ']'
+CondSet:4> export LC_TIME
+/etc/profile.d/0lang.sh:29> [ -n ru_RU.CP1251 ']'
+/etc/profile.d/0lang.sh:29> export LANG
+/etc/profile.d/0lang.sh:30> [ -n '' ']'
+/etc/profile.d/0lang.sh:33> Unset LC_ALL
+Unset:2> unset LC_ALL
+/etc/profile.d/0lang.sh:35> [ -n '' ']'
+/etc/profile.d/0lang.sh:38> Unset LINGUAS
+Unset:2> unset LINGUAS
+/etc/profile.d/0lang.sh:41> [ -n '' ']'
+/etc/profile.d/0lang.sh:41> Unset ENC
+Unset:2> unset ENC
+/etc/profile.d/0lang.sh:42> [ -n '' ']'
+/etc/profile.d/0lang.sh:42> Unset XIM
+Unset:2> unset XIM
+/etc/profile.d/0lang.sh:43> [ -n '' ']'
+/etc/profile.d/0lang.sh:43> Unset XIM_PROGRAM
+Unset:2> unset XIM_PROGRAM
+/etc/profile.d/0lang.sh:44> [ -n '' ']'
+/etc/profile.d/0lang.sh:44> Unset XMODIFIERS
+Unset:2> unset XMODIFIERS
+/etc/profile.d/0lang.sh:45> [ -n '' ']'
+/etc/profile.d/0lang.sh:45> Unset _XKB_CHARSET
+Unset:2> unset _XKB_CHARSET
+/etc/profile.d/0lang.sh:47> Unset SYSFONTACM SYSFONT
+Unset:2> unset SYSFONTACM SYSFONT
+/etc/profile.d/0lang.sh:51> [ -n :0.0 ']'
+/etc/profile.d/0lang.sh:52> [ '' '=' yes ']'
+/etc/profile.d/0lang.sh:56> [ -n '' ']'
+/etc/profile.d/0lang.sh:59> [ -n ru_RU.CP1251 -a -z '' ']'
+/etc/profile.d/0lang.sh:60> LANGUAGE=+/etc/profile.d/0lang.sh:1> grep -s '^ru_RU.CP1251:' /etc/sysconfig/langmap
+/etc/profile.d/0lang.sh:60> LANGUAGE=+/etc/profile.d/0lang.sh:1> :
+/etc/profile.d/0lang.sh:60> LANGUAGE=''
+/etc/profile.d/0lang.sh:62> [ -n '' ']'
+/etc/profile.d/0lang.sh:65> Unset LANGUAGE
+Unset:2> unset LANGUAGE
+/etc/profile.d/0lang.sh:70> unset sourced
+/etc/profile.d/0lang.sh:71> unset -f Unset CondSet
+/etc/profile:34> f=/etc/profile.d/color_ls.sh
+/etc/profile:35> [ -f /etc/profile.d/color_ls.sh -a -r /etc/profile.d/color_ls.sh -a -x /etc/profile.d/color_ls.sh -a -s /etc/profile.d/color_ls.sh -a ! -L /etc/profile.d/color_ls.sh ']'
+/etc/profile:36> . /etc/profile.d/color_ls.sh
+/etc/profile.d/color_ls.sh:3> COLORS=/etc/DIR_COLORS
+/etc/profile.d/color_ls.sh:4> [ -f /etc/DIR_COLORS.screen -a -r /etc/DIR_COLORS.screen ']'
+/etc/profile.d/color_ls.sh:7> [ -f /home/at/.dircolors -a -r /home/at/.dircolors ']'
+/etc/profile.d/color_ls.sh:10> [ -f /home/at/.dircolors.screen -a -r /home/at/.dircolors.screen ']'
+/etc/profile.d/color_ls.sh:13> [ -f /home/at/.dir_colors -a -r /home/at/.dir_colors ']'
+/etc/profile.d/color_ls.sh:16> [ -f /home/at/.dir_colors.screen -a -r /home/at/.dir_colors.screen ']'
+/etc/profile.d/color_ls.sh:20> [ -f /etc/DIR_COLORS -a -r /etc/DIR_COLORS ']'
+/etc/profile.d/color_ls.sh:1> dircolors --sh /etc/DIR_COLORS
+/etc/profile.d/color_ls.sh:21> eval 'LS_COLORS='\''no=00:fi=00:di=01;34:ln=01;36:pi=40;33:so=01;35:do=01;35:bd=40;33;01:cd=40;33;01:or=40;31;01:su=37;41:sg=30;43:tw=30;42:ow=34;42:st=37;44:ex=01;32:*.sh=01;32:*.csh=01;32:*.tar=01;31:*.tgz=01;31:*.arc=01;31:*.arj=01;31:*.taz=01;31:*.lzh=01;31:*.lha=01;31:*.zip=01;31:*.z=01;31:*.Z=01;31:*.gz=01;31:*.bz2=01;31:*.deb=01;31:*.rpm=01;31:*.jar=01;31:*.cpio=01;31:*.jpg=01;35:*.jpeg=01;35:*.gif=01;35:*.bmp=01;35:*.pbm=01;35:*.pgm=01;35:*.ppm=01;35:*.tga=01;35:*.xbm=01;35:*.xpm=01;35:*.tif=01;35:*.tiff=01;35:*.png=01;35:*.mov=01;35:*.mpg=01;35:*.mpeg=01;35:*.avi=01;35:*.fli=01;35:*.gl=01;35:*.dl=01;35:*.xcf=01;35:*.xwd=01;35:*.flac=01;35:*.mp3=01;35:*.mpc=01;35:*.ogg=01;35:*.wav=01;35:'\'';' export LS_COLORS
+/etc/profile.d/color_ls.sh:21> LS_COLORS='no=00:fi=00:di=01;34:ln=01;36:pi=40;33:so=01;35:do=01;35:bd=40;33;01:cd=40;33;01:or=40;31;01:su=37;41:sg=30;43:tw=30;42:ow=34;42:st=37;44:ex=01;32:*.sh=01;32:*.csh=01;32:*.tar=01;31:*.tgz=01;31:*.arc=01;31:*.arj=01;31:*.taz=01;31:*.lzh=01;31:*.lha=01;31:*.zip=01;31:*.z=01;31:*.Z=01;31:*.gz=01;31:*.bz2=01;31:*.deb=01;31:*.rpm=01;31:*.jar=01;31:*.cpio=01;31:*.jpg=01;35:*.jpeg=01;35:*.gif=01;35:*.bmp=01;35:*.pbm=01;35:*.pgm=01;35:*.ppm=01;35:*.tga=01;35:*.xbm=01;35:*.xpm=01;35:*.tif=01;35:*.tiff=01;35:*.png=01;35:*.mov=01;35:*.mpg=01;35:*.mpeg=01;35:*.avi=01;35:*.fli=01;35:*.gl=01;35:*.dl=01;35:*.xcf=01;35:*.xwd=01;35:*.flac=01;35:*.mp3=01;35:*.mpc=01;35:*.ogg=01;35:*.wav=01;35:'
+/etc/profile.d/color_ls.sh:21> export LS_COLORS
+/etc/profile.d/color_ls.sh:24> [ -z /bin/zsh -o -z '' ']'
+/etc/profile.d/color_ls.sh:25> [ screen '=' emacs ']'
+/etc/profile.d/color_ls.sh:28> alias 'ls=ls --color=auto'
+/etc/profile:34> f=/etc/profile.d/configure_keyboard.sh
+/etc/profile:35> [ -f /etc/profile.d/configure_keyboard.sh -a -r /etc/profile.d/configure_keyboard.sh -a -x /etc/profile.d/configure_keyboard.sh -a -s /etc/profile.d/configure_keyboard.sh -a ! -L /etc/profile.d/configure_keyboard.sh ']'
+/etc/profile:36> . /etc/profile.d/configure_keyboard.sh
+/etc/profile.d/configure_keyboard.sh:2> [ -z '' ']'
+/etc/profile.d/configure_keyboard.sh:2> tty --silent
+/etc/profile.d/configure_keyboard.sh:3> [ -x /usr/share/console-common-scripts/configure_keyboard ']'
+/etc/profile.d/configure_keyboard.sh:4> /usr/share/console-common-scripts/configure_keyboard
+/etc/profile:34> f=/etc/profile.d/console.sh
+/etc/profile:35> [ -f /etc/profile.d/console.sh -a -r /etc/profile.d/console.sh -a -x /etc/profile.d/console.sh -a -s /etc/profile.d/console.sh -a ! -L /etc/profile.d/console.sh ']'
+/etc/profile:36> . /etc/profile.d/console.sh
+/etc/profile.d/console.sh:1> [ -x /usr/share/console-common-scripts/vt_activate_user_map ']'
+/etc/profile.d/console.sh:2> /usr/share/console-common-scripts/vt_activate_user_map
+/etc/profile.d/console.sh:5> [ -x /usr/share/console-common-scripts/vt_activate_unicode ']'
+/etc/profile.d/console.sh:6> /usr/share/console-common-scripts/vt_activate_unicode
+/etc/profile:34> f=/etc/profile.d/cvs.sh
+/etc/profile:35> [ -f /etc/profile.d/cvs.sh -a -r /etc/profile.d/cvs.sh -a -x /etc/profile.d/cvs.sh -a -s /etc/profile.d/cvs.sh -a ! -L /etc/profile.d/cvs.sh ']'
+/etc/profile:36> . /etc/profile.d/cvs.sh
+/etc/profile.d/cvs.sh:4> export 'CVS_RSH=ssh'
+/etc/profile:34> f=/etc/profile.d/givertcap.sh
+/etc/profile:35> [ -f /etc/profile.d/givertcap.sh -a -r /etc/profile.d/givertcap.sh -a -x /etc/profile.d/givertcap.sh -a -s /etc/profile.d/givertcap.sh -a ! -L /etc/profile.d/givertcap.sh ']'
+/etc/profile:36> . /etc/profile.d/givertcap.sh
+/etc/profile.d/givertcap.sh:2> GIVERTCAP=/usr/bin/givertcap
+/etc/profile.d/givertcap.sh:3> export GIVERTCAP
+/etc/profile:34> f=/etc/profile.d/gnupg-agent.sh
+/etc/profile:35> [ -f /etc/profile.d/gnupg-agent.sh -a -r /etc/profile.d/gnupg-agent.sh -a -x /etc/profile.d/gnupg-agent.sh -a -s /etc/profile.d/gnupg-agent.sh -a ! -L /etc/profile.d/gnupg-agent.sh ']'
+/etc/profile:36> . /etc/profile.d/gnupg-agent.sh
+/etc/profile.d/gnupg-agent.sh:3> [ -n '' ']'
+/etc/profile.d/gnupg-agent.sh:3> GNUPGHOME=/home/at/.gnupg
+/etc/profile.d/gnupg-agent.sh:5> [ -r /home/at/.gnupg/gpg.conf ']'
+/etc/profile.d/gnupg-agent.sh:7> CFG=/home/at/.gnupg/gpg.conf
+/etc/profile.d/gnupg-agent.sh:12> grep -qs '^use-agent' /home/at/.gnupg/gpg.conf
+/etc/profile:34> f=/etc/profile.d/ladspa.sh
+/etc/profile:35> [ -f /etc/profile.d/ladspa.sh -a -r /etc/profile.d/ladspa.sh -a -x /etc/profile.d/ladspa.sh -a -s /etc/profile.d/ladspa.sh -a ! -L /etc/profile.d/ladspa.sh ']'
+/etc/profile:36> . /etc/profile.d/ladspa.sh
+/etc/profile.d/ladspa.sh:2> LADSPA_PATH=/usr/lib/ladspa
+/etc/profile.d/ladspa.sh:3> export LADSPA_PATH
+/etc/profile:34> f=/etc/profile.d/lang.sh
+/etc/profile:35> [ -f /etc/profile.d/lang.sh -a -r /etc/profile.d/lang.sh -a -x /etc/profile.d/lang.sh -a -s /etc/profile.d/lang.sh -a ! -L /etc/profile.d/lang.sh ']'
+/etc/profile:34> f=/etc/profile.d/less.sh
+/etc/profile:35> [ -f /etc/profile.d/less.sh -a -r /etc/profile.d/less.sh -a -x /etc/profile.d/less.sh -a -s /etc/profile.d/less.sh -a ! -L /etc/profile.d/less.sh ']'
+/etc/profile:36> . /etc/profile.d/less.sh
+/etc/profile.d/less.sh:1> [ -z -MM ']'
+/etc/profile.d/less.sh:5> [ -z /etc/.less -a ! -f /home/at/.less ']'
+/etc/profile.d/less.sh:9> [ -z '|/usr/share/less/lesspipe.sh %s' -a -x /usr/share/less/lesspipe.sh ']'
+/etc/profile:34> f=/etc/profile.d/libglib2.sh
+/etc/profile:35> [ -f /etc/profile.d/libglib2.sh -a -r /etc/profile.d/libglib2.sh -a -x /etc/profile.d/libglib2.sh -a -s /etc/profile.d/libglib2.sh -a ! -L /etc/profile.d/libglib2.sh ']'
+/etc/profile:36> . /etc/profile.d/libglib2.sh
+/etc/profile.d/libglib2.sh:5> export 'G_BROKEN_FILENAMES=1'
+/etc/profile.d/libglib2.sh:12> NATSPEC=/usr/bin/natspec
+/etc/profile.d/libglib2.sh:13> test -x /usr/bin/natspec
+/etc/profile.d/libglib2.sh:1> natspec -f
+/etc/profile.d/libglib2.sh:13> export 'G_FILENAME_ENCODING=cp1251'
+/etc/profile:34> f=/etc/profile.d/qt3dir.sh
+/etc/profile:35> [ -f /etc/profile.d/qt3dir.sh -a -r /etc/profile.d/qt3dir.sh -a -x /etc/profile.d/qt3dir.sh -a -s /etc/profile.d/qt3dir.sh -a ! -L /etc/profile.d/qt3dir.sh ']'
+/etc/profile:36> . /etc/profile.d/qt3dir.sh
+/etc/profile.d/qt3dir.sh:2> [ -n /usr/lib/qt3/ ']'
+/etc/profile.d/qt3dir.sh:3> [ -n /usr/lib/qt3/include/ ']'
+/etc/profile.d/qt3dir.sh:4> [ -n /usr/lib/qt3/lib/ ']'
+/etc/profile:34> f=/etc/profile.d/shared-mime-info.sh
+/etc/profile:35> [ -f /etc/profile.d/shared-mime-info.sh -a -r /etc/profile.d/shared-mime-info.sh -a -x /etc/profile.d/shared-mime-info.sh -a -s /etc/profile.d/shared-mime-info.sh -a ! -L /etc/profile.d/shared-mime-info.sh ']'
+/etc/profile:36> . /etc/profile.d/shared-mime-info.sh
+/etc/profile.d/shared-mime-info.sh:1> export 'XDG_DATA_DIRS=/usr/share:/var/cache'
+/etc/profile:34> f=/etc/profile.d/tmpdir.sh
+/etc/profile:35> [ -f /etc/profile.d/tmpdir.sh -a -r /etc/profile.d/tmpdir.sh -a -x /etc/profile.d/tmpdir.sh -a -s /etc/profile.d/tmpdir.sh -a ! -L /etc/profile.d/tmpdir.sh ']'
+/etc/profile:36> . /etc/profile.d/tmpdir.sh
+/etc/profile.d/tmpdir.sh:4> [ -z /tmp/.private/at ']'
+/etc/profile.d/tmpdir.sh:4> [ ! -d /tmp/.private/at -o ! -w /tmp/.private/at ']'
+/etc/profile:34> f=/etc/profile.d/xhost.sh
+/etc/profile:35> [ -f /etc/profile.d/xhost.sh -a -r /etc/profile.d/xhost.sh -a -x /etc/profile.d/xhost.sh -a -s /etc/profile.d/xhost.sh -a ! -L /etc/profile.d/xhost.sh ']'
+/etc/profile:36> . /etc/profile.d/xhost.sh
+/etc/profile.d/xhost.sh:4> [ -z /home/at/.Xauthority -a -z '' -a -n :0.0 -a 500 -gt 14 ']'
+/etc/profile:40> unset f
+/etc/zshrc:13> [ -f /etc/bashrc ']'
+/etc/zshrc:13> . /etc/bashrc
+/etc/bashrc:4> [ -z '' ']'
+/etc/bashrc:6> [ '%m%# ' '=' '\s-\v\$ ' ']'
+/etc/bashrc:11> [ PROMPT_COMMAND '=' PROMPT_COMMAND ']'
+/etc/bashrc:12> case screen (*xterm*|*rxvt*|*Eterm*)
+/etc/bashrc:20> f=/etc/bashrc.d/alias.sh
+/etc/bashrc:21> [ -r /etc/bashrc.d/alias.sh ']'
+/etc/bashrc:22> . /etc/bashrc.d/alias.sh
+/etc/bashrc.d/alias.sh:9> [ -n '' -o ! -f /home/at/.alias ']'
+/etc/bashrc.d/alias.sh:10> [ -z '' ']'
+/etc/bashrc.d/alias.sh:13> alias 'du=du -h'
+/etc/bashrc.d/alias.sh:14> alias 'df=df -h -x supermount'
+/etc/bashrc.d/alias.sh:17> alias 'rm=rm -i'
+/etc/bashrc.d/alias.sh:18> alias 'mv=mv -i'
+/etc/bashrc.d/alias.sh:19> alias 'cp=cp -i'
+/etc/bashrc.d/alias.sh:22> case screen (*emacs*)
+/etc/bashrc.d/alias.sh:22> case screen (*)
+/etc/bashrc.d/alias.sh:29> alias 'ls=ls --color=auto'
+/etc/bashrc.d/alias.sh:32> alias 'l=ls -lapt'
+/etc/bashrc.d/alias.sh:33> alias 'll=ls -laptc'
+/etc/bashrc.d/alias.sh:36> alias 'md=mkdir'
+/etc/bashrc.d/alias.sh:37> alias 'rd=rmdir'
+/etc/bashrc.d/alias.sh:38> alias 'cd..=cd ..'
+/etc/bashrc.d/alias.sh:39> alias 'su-=su -'
+/etc/bashrc:20> f=/etc/bashrc.d/mc.sh
+/etc/bashrc:21> [ -r /etc/bashrc.d/mc.sh ']'
+/etc/bashrc:22> . /etc/bashrc.d/mc.sh
+/etc/bashrc.d/mc.sh:1> alias 'mc=. /usr/share/mc/bin/mc-wrapper.sh'
+/etc/bashrc:20> f=/etc/bashrc.d/mplayer.sh
+/etc/bashrc:21> [ -r /etc/bashrc.d/mplayer.sh ']'
+/etc/bashrc:22> . /etc/bashrc.d/mplayer.sh
+/etc/bashrc.d/mplayer.sh:1> alias 'mplayer=/usr/bin/soundwrapper /usr/bin/mplayer'
+/etc/bashrc:25> unset f
+/etc/bashrc:27> [ -n '' -a -n '' ']'
+/etc/zshrc:17> emulate zsh
+/etc/zshrc:18> [ -f /etc/zlerc ']'
+/etc/zshrc:18> . /etc/zlerc
+/etc/zlerc:7> [[ -o zle ]]
+/etc/zlerc:7> zmodload -i zsh/zle
+/etc/zlerc:9> bindkey -d
+/etc/zlerc:11> bindkey -e
+/etc/zlerc:14> bindkey '\e[2~' quoted-insert
+/etc/zlerc:15> bindkey '\e[3~' delete-char
+/etc/zlerc:16> bindkey '\e[1~' beginning-of-line
+/etc/zlerc:17> bindkey '\e[4~' end-of-line
+/etc/zlerc:18> bindkey '\e[5~' up-line-or-history
+/etc/zlerc:19> bindkey '\e[6~' down-line-or-history
+/etc/zlerc:22> bindkey '\e[7~' beginning-of-line
+/etc/zlerc:23> bindkey '\e[8~' end-of-line
+/etc/zlerc:26> bindkey '\e[H' beginning-of-line
+/etc/zlerc:27> bindkey '\e[F' end-of-line
+/etc/zlerc:30> bindkey '\eOH' beginning-of-line
+/etc/zlerc:31> bindkey '\eOF' end-of-line
+/etc/zlerc:32> bindkey '\eOd' backward-word
+/etc/zlerc:33> bindkey '\eOc' forward-word
+/etc/zlerc:36> bindkey '\e[\C-@' beginning-of-line
+/etc/zlerc:37> bindkey '\e[e' end-of-line
+/etc/zlerc:45> bindkey -s '\eOo' /
+/etc/zlerc:46> bindkey -s '\eOj' '*'
+/etc/zlerc:47> bindkey -s '\eOm' -
+/etc/zlerc:48> bindkey -s '\eOk' +
+/etc/zlerc:49> bindkey -s '\eOl' +
+/etc/zlerc:50> bindkey '\eOM' accept-line
+/etc/zlerc:54> bindkey -s '\eOn' .
+/etc/zlerc:57> bindkey -s '\eOp' 0
+/etc/zlerc:58> bindkey -s '\eOq' 1
+/etc/zlerc:59> bindkey -s '\eOr' 2
+/etc/zlerc:60> bindkey -s '\eOs' 3
+/etc/zlerc:61> bindkey -s '\eOt' 4
+/etc/zlerc:62> bindkey -s '\eOu' 5
+/etc/zlerc:63> bindkey -s '\eOv' 6
+/etc/zlerc:64> bindkey -s '\eOw' 7
+/etc/zlerc:65> bindkey -s '\eOx' 8
+/etc/zlerc:66> bindkey -s '\eOy' 9
+/etc/zlerc:73> bindkey '\eOD' backward-char
+/etc/zlerc:74> bindkey '\eOC' forward-char
+/etc/zlerc:75> bindkey '\eOA' up-line-or-history
+/etc/zlerc:76> bindkey '\eOB' down-line-or-history
+/etc/zshrc:21> PROMPT='%n@%m %3~ %(!.#.$) '
+/etc/zshrc:24> HISTFILE=/home/at/.zsh_history
+/etc/zshrc:25> HISTSIZE=11111
+/etc/zshrc:26> SAVEHIST=9999
+/etc/zshrc:29> setopt INC_APPEND_HISTORY
+/etc/zshrc:30> setopt HIST_IGNORE_ALL_DUPS
+/etc/zshrc:31> setopt HIST_SAVE_NO_DUPS
+/etc/zshrc:32> setopt HIST_REDUCE_BLANKS
+/etc/zshrc:35> autoload -U compinit
+/etc/zshrc:35> compinit
+compinit:70> emulate -L zsh
+compinit:71> setopt extendedglob
+compinit:73> typeset _i_dumpfile _i_files _i_line _i_done _i_dir '_i_autodump=1'
+compinit:74> typeset _i_tag _i_file _i_addfiles '_i_fail=ask' '_i_check=yes' _i_name
+compinit:76> [[ 0 -gt 0 ]]
+compinit:108> typeset -gA _comps _services _patcomps _postpatcomps
+compinit:113> typeset -gA _compautos
+compinit:118> typeset -gA _lastcomp
+compinit:121> [[ -n '' ]]
+compinit:125> _comp_dumpfile=/home/at/.zcompdump
+compinit:130> _comp_options=( extendedglob bareglobqual glob multibyte nullglob rcexpandparam unset NO_markdirs NO_globsubst NO_shwordsplit NO_shglob NO_kshglob NO_ksharrays NO_kshtypeset NO_cshnullglob NO_allexport NO_aliases NO_errexit NO_octalzeroes NO_warncreateglobal )
+compinit:162> _comp_setup='setopt localoptions localtraps ${_comp_options[@]};
             local IFS=$'\'' \t\r\n\0'\''
             exec </dev/null;
             trap - ZERR'
+compinit:167> compprefuncs=( )
+compinit:168> comppostfuncs=( )
+compinit:172> : compinit
+compinit:408> typeset _i_wdirs _i_wfiles
+compinit:410> _i_wdirs=( )
+compinit:411> _i_wfiles=( )
+compinit:413> autoload -Uz compaudit
+compinit:414> [[ -n yes ]]
+compinit:415> typeset _i_q
+compinit:416> eval compaudit
+(eval):1> compaudit
+compaudit:143> compaudit
+compaudit:13> emulate -L zsh
+compaudit:14> setopt extendedglob
+compaudit:16> [[ -x /usr/bin/getent ]]
+compaudit:25> ((  0  ))
+compaudit:27> ((  22 == 0  ))
+compaudit:31> set -- /usr/local/share/zsh /usr/share/zsh/Completion /usr/share/zsh/Completion/AIX /usr/share/zsh/Completion/BSD /usr/share/zsh/Completion/Base /usr/share/zsh/Completion/Cygwin /usr/share/zsh/Completion/Darwin /usr/share/zsh/Completion/Debian /usr/share/zsh/Completion/Linux /usr/share/zsh/Completion/Mandrake /usr/share/zsh/Completion/Redhat /usr/share/zsh/Completion/Unix /usr/share/zsh/Completion/X /usr/share/zsh/Completion/Zsh /usr/share/zsh/Exceptions /usr/share/zsh/MIME /usr/share/zsh/Misc /usr/share/zsh/Newuser /usr/share/zsh/Prompts /usr/share/zsh/TCP /usr/share/zsh/Zftp /usr/share/zsh/Zle
+compaudit:38> ((  1  ))
+compaudit:44> fpath=( /usr/local/share/zsh /usr/share/zsh/Completion /usr/share/zsh/Completion/AIX /usr/share/zsh/Completion/BSD /usr/share/zsh/Completion/Base /usr/share/zsh/Completion/Cygwin /usr/share/zsh/Completion/Darwin /usr/share/zsh/Completion/Debian /usr/share/zsh/Completion/Linux /usr/share/zsh/Completion/Mandrake /usr/share/zsh/Completion/Redhat /usr/share/zsh/Completion/Unix /usr/share/zsh/Completion/X /usr/share/zsh/Completion/Zsh /usr/share/zsh/Exceptions /usr/share/zsh/MIME /usr/share/zsh/Misc /usr/share/zsh/Newuser /usr/share/zsh/Prompts /usr/share/zsh/TCP /usr/share/zsh/Zftp /usr/share/zsh/Zle )
+compaudit:49> ((  0  ))
compaudit:50: bad substitution
+compinit:417> [[ -n '' ]]
+compinit:435> _comp_secure=yes
+compinit:440> autoload -Uz compdump compinstall
+compinit:444> _i_done=''
+compinit:446> [[ -f /home/at/.zcompdump ]]
+compinit:447> [[ -n yes ]]
+compinit:448> read -rA _i_line
+compinit:449> [[ _i_autodump -eq 1 && '' -eq 0 && 4.3.2-dev-1 == '' ]]
+compinit:460> [[ -z '' ]]
+compinit:461> typeset -A _i_test
+compinit:463> _i_dir=/usr/local/share/zsh
+compinit:464> [[ /usr/local/share/zsh == . ]]
+compinit:465> ((  0  ))
+compinit:463> _i_dir=/usr/share/zsh/Completion
+compinit:464> [[ /usr/share/zsh/Completion == . ]]
+compinit:465> ((  0  ))
+compinit:463> _i_dir=/usr/share/zsh/Completion/AIX
+compinit:464> [[ /usr/share/zsh/Completion/AIX == . ]]
+compinit:465> ((  0  ))
+compinit:466> _i_file=/usr/share/zsh/Completion/AIX/_floppy
+compinit:467> _i_name=_floppy
+compinit:468> ((  0 + 0  ))
compinit:469: bad pattern: _i_test[=yes

+/home/at/.zshrc:2> HISTFILE=/home/at/.zsh_history
+/home/at/.zshrc:3> HISTSIZE=111111
+/home/at/.zshrc:4> SAVEHIST=99999
+/home/at/.zshrc:5> setopt INC_APPEND_HISTORY
+/home/at/.zshrc:6> setopt HIST_IGNORE_ALL_DUPS
+/home/at/.zshrc:7> setopt HIST_SAVE_NO_DUPS
+/home/at/.zshrc:8> setopt HIST_REDUCE_BLANKS
+/home/at/.zshrc:9> unsetopt HIST_SAVE_BY_COPY
+/home/at/.zshrc:12> setopt NO_BEEP
+/home/at/.zshrc:15> setopt AUTO_CD
+/home/at/.zshrc:16> export 'sisyphus=/raid/ALT/Sisyphus'
+/home/at/.zshrc:17> sources=/home/at/RPM/SOURCES
+/home/at/.zshrc:18> specs=/home/at/RPM/SPECS
+/home/at/.zshrc:19> build=/home/at/RPM/BUILD
+/home/at/.zshrc:20> : /raid/ALT/Sisyphus /home/at/RPM/SOURCES /home/at/RPM/SPECS /home/at/RPM/BUILD
+/home/at/.zshrc:21> tmp=/tmp/.private/at
+/home/at/.zshrc:22> : /tmp/.private/at
+/home/at/.zshrc:33> myprompt
+myprompt:2> autoload colors -U
+myprompt:2> colors
+colors:4> typeset -Ag color colour
+colors:6> color=( 00 none 01 bold 02 faint 22 normal 03 standout 23 no-standout 04 underline 24 no-underline 05 blink 25 no-blink 07 reverse 27 no-reverse 08 conceal 30 black 40 bg-black 31 red 41 bg-red 32 green 42 bg-green 33 yellow 43 bg-yellow 34 blue 44 bg-blue 35 magenta 45 bg-magenta 36 cyan 46 bg-cyan 37 white 47 bg-white 39 default 49 bg-default )
+colors:38> local k
colors:39: bad substitution
+colors:39> k=
at@localhost ~ $

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: compaudit problem

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 513 bytes --]

On Sat, Aug 19, 2006 at 12:20:04PM -0700, Bart Schaefer wrote:
> On Aug 19,  9:45pm, Alexey Tourbin wrote:
> }
> } Linux 2.6.16
> } glibc 2.3.6
> } gcc 4.1.1 mostly in sync with gcc-4.1.1-6 from Fedora
> 
> I have 2.6.9, 2.3.4, 3.4.5 (RHEL4).  I wonder if there's something
> about gcc-4 that's an issue.  I've just installed Xcode on my Mac,
> which has gcc-4, so perhaps I'll attempt a zsh compile on that.

I recompiled with gcc 3.4.5 but it's all the same.
So apparently it's not a compiler bug.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: compaudit problem

[Peter Stephenson]

On Sun, 20 Aug 2006 21:32:06 +0400
Alexey Tourbin <at@altlinux.ru> wrote:
> Here is full -x transcript, maybe it can shed light.

Thanks.  Unfortunately it still looks weird; the instructions causing
one of the errors reduce to:
 
> +compinit:461> typeset -A _i_test
> +compinit:467> _i_name=_floppy

and then

_i_test[$_i_name]=yes

which causes

> compinit:469: bad pattern: _i_test[=yes

This doesn't happen here.  It looks like this would need analysing with
a debugger somewhere that shows the problem.  I see:

+/home/at/.i18n:1> LANG=ru_RU.CP1251

which is a little different from what I've been using to test but
there's no reason why it should make ASCII text go funny.

My guess is it's somewhere down in parse_subscript that checks for
balanced square brackets, but I don't see why.

-- 
Peter Stephenson <p.w.stephenson@ntlworld.com>
Web page now at http://homepage.ntlworld.com/p.w.stephenson/

Re: compaudit problem

[Alexey Tourbin]

[-- Attachment #1.1: Type: text/plain, Size: 2385 bytes --]

On Sat, Aug 19, 2006 at 10:30:33AM -0700, Bart Schaefer wrote:
> I can't reproduce any of these with my build of 08/17 sources from CVS.
> I've tried compiling both with and without multibyte support.  I've even
> tried forcing Test/D07multibyte.ztst to use ru_RU.UTF-8 as $LANG when
> running "make check".
> 
> Does anything change if you run "zsh -o posix_identifiers"?  Have you
> tried building with "configure --disable-multibyte"?
> 
> Changes between 06/19 and 07/29 that may have affected this:
> 
>         * 22518: Src/params.c, Src/utils,c, Src/zsh.h,
>         Test/B02typeset.ztst: Initial go at making parameter subscripts
>         use multibyte characters.
> 
>         * 22524: Src/params.c, Test/D07multibyte.ztst: searchable
>         subscripts with multibyte characters.
> 
>         * 22544: README, Doc/Zsh/options.yo, Src/builtin.c, Src/glob.c,
>         Src/lex.c, Src/math.c, Src/module.c, Src/options.c, Src/params.c,
>         Src/parse.c, Src/subst.c, Src/utils.c, Src/zsh.h, Src/ztype.h,
>         Src/Zle/compcore.c, Src/Zle/zle_tricky.c, Test/D07multibyte.ztst:
>         Improve use of ztype tests for multibyte characters.  Add
>         POSIX_IDENTIFIERS option to control allowability of multibyte
>         alphanumeric characters in parameter and module names.
> 
>         * 22556: Doc/Zsh/builtins.yo, Functions/Zle/insert-composed-char,
>         Src/builtin.c, Src/pattern.c, Src/subst.c, Src/utils.c, Src/zsh.h,
>         Src/ztype.h, Src/Zle/zle.h, Src/Zle/zle_main.c,
>         Test/D04parameter.ztst, Test/D07multibyte.ztst: Multibyte
>         separators and delimiters.
> 
>         * unposted: Src/pattern.c, Src/utils.c: minor typos in
>         22556 found when MULTIBYTE_SUPPORT is not defined.
> 
>         * 22557: Doc/Zsh/options.yo, Misc/globtests, Src/options.c,
>         Src/pattern.c, Test/D02glob.ztst, Test/D07multibyte.ztst:
>         Turn on multibyte option by default for MULTIBYTE_SUPPORT and fix
>         tests and patterns.

I can still reproduce this with current cvs snapshot.
I imported cvs sources into git and tried git-bisect.
git-bisect says the problem started with change #22557.

[refs/bisect/bad] [bisect] 22557: turn on multibyte option by default
[refs/bisect/good-b76d3fa3515823810784266bbf0c9cc1c58c504d] fix typos in 22556

Build script is attached.

[-- Attachment #1.2: bzsh --]
[-- Type: text/plain, Size: 1802 bytes --]

#!/bin/sh -e
autoreconf -fisv

# Disable libnsl/NIS support.
export ac_cv_search_yp_all=no zsh_cv_sys_nis=no zsh_cv_sys_nis_plus=no

# YODL is not required, pre-generated man pages are shipped.
export YODL=yodl PDFETEX=pdfetex

# Avoid autoconf thinking it should strip when linking.
export LDFLAGS=

# Disable sys/capability.h (see #8142).
export ac_cv_header_sys_capability_h=no


  CFLAGS="${CFLAGS:--pipe -Wall -O2 -march=athlon -mtune=athlon-xp}" ; export CFLAGS ; 
  CXXFLAGS="${CXXFLAGS:--pipe -Wall -O2 -march=athlon -mtune=athlon-xp}" ; export CXXFLAGS ; 
  FFLAGS="${FFLAGS:--pipe -Wall -O2 -march=athlon -mtune=athlon-xp}" ; export FFLAGS ; 
   
   
  export lt_cv_deplibs_check_method=pass_all ; 
  [ -f configure.ac -o -f configure.in ] && libtoolize --copy --force ; 
  ./configure --build=athlon-alt-linux --host=athlon-alt-linux \
 	--prefix=/usr \
	--exec-prefix=/usr \
	--bindir=/usr/bin \
	--sbindir=/usr/sbin \
	--sysconfdir=/etc \
	--datadir=/usr/share \
	--includedir=/usr/include \
	--libdir=/usr/lib \
	--libexecdir=/usr/lib \
	--localstatedir=/var/lib \
	--sharedstatedir=/usr/com \
	--mandir=/usr/share/man \
	--infodir=/usr/share/info \
	--disable-dependency-tracking \
	--without-included-gettext \
	--enable-etcdir=/etc \
	--enable-fndir=/usr/share/zsh \
	--enable-scriptdir=/usr/share/zsh/scripts \
	--enable-site-fndir=/usr/local/share/zsh \
	--enable-site-scriptdir=/usr/local/share/zsh/scripts \
	--enable-function-subdirs \
	--enable-maildir-support \
	--with-curses-terminfo \
	--with-tcsetpgrp \
	--disable-cap \
	--enable-pcre \
	--enable-multibyte \
	 \
	 \
	#

# We don't expect that something is broken.
grep '^#define.*BROKEN' config.h && exit 1

# SMP-incompatible.
make MODDIR=/usr/lib
make -C Etc

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: compaudit problem

[Bart Schaefer]

On Oct 17, 11:05pm, Alexey Tourbin wrote:
}
} On Sat, Aug 19, 2006 at 10:30:33AM -0700, Bart Schaefer wrote:
} > 
} >         * 22557: Doc/Zsh/options.yo, Misc/globtests, Src/options.c,
} >         Src/pattern.c, Test/D02glob.ztst, Test/D07multibyte.ztst:
} >         Turn on multibyte option by default for MULTIBYTE_SUPPORT and fix
} >         tests and patterns.
} 
} git-bisect says the problem started with change #22557.

Given that, according to your build script, you compiled both the before-
and after-22557 sources with --enable-multibyte, the only change in 22557
that could possibly be related is this one:

--- Src/pattern.c	25 Jul 2006 09:25:27 -0000	1.37
+++ Src/pattern.c	25 Jul 2006 18:06:48 -0000
@@ -343,7 +343,7 @@
     /* Error.  Treat as single byte. */
     /* Reset the shift state for next time. */
     memset(&shiftstate, 0, sizeof(shiftstate));
-    return (wchar_t) *(*x)++;
+    return (wchar_t) STOUC(*(*x)++);
 }
 
 #else

Does backing out that STOUC fix the problem?  What underlying integer
type is wchar_t on the machine where you're building zsh?

Re: compaudit problem

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 1318 bytes --]

On Tue, Oct 17, 2006 at 08:41:50PM -0700, Bart Schaefer wrote:
> On Oct 17, 11:05pm, Alexey Tourbin wrote: > }
> } On Sat, Aug 19, 2006 at 10:30:33AM -0700, Bart Schaefer wrote:
> } > 
> } >         * 22557: Doc/Zsh/options.yo, Misc/globtests, Src/options.c,
> } >         Src/pattern.c, Test/D02glob.ztst, Test/D07multibyte.ztst:
> } >         Turn on multibyte option by default for MULTIBYTE_SUPPORT and fix
> } >         tests and patterns.
> } 
> } git-bisect says the problem started with change #22557.
> 
> Given that, according to your build script, you compiled both the before-
> and after-22557 sources with --enable-multibyte, the only change in 22557
> that could possibly be related is this one:
> 
> --- Src/pattern.c	25 Jul 2006 09:25:27 -0000	1.37
> +++ Src/pattern.c	25 Jul 2006 18:06:48 -0000
> @@ -343,7 +343,7 @@
>      /* Error.  Treat as single byte. */
>      /* Reset the shift state for next time. */
>      memset(&shiftstate, 0, sizeof(shiftstate));
> -    return (wchar_t) *(*x)++;
> +    return (wchar_t) STOUC(*(*x)++);
>  }
>  
>  #else
> 
> Does backing out that STOUC fix the problem?  What underlying integer
> type is wchar_t on the machine where you're building zsh?

Nope.  But reverting Src/options.c part fixes the problem.
I'm kind of puzzled.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: compaudit problem

[Peter Stephenson]

Alexey Tourbin <at@altlinux.ru> wrote:
> > Given that, according to your build script, you compiled both the before-
> > and after-22557 sources with --enable-multibyte, the only change in 22557
> > that could possibly be related is this one:
>
> Nope.  But reverting Src/options.c part fixes the problem.
> I'm kind of puzzled.

That part turns on the MULTIBYTE option by default.  It sounds like
the real problem was in an earlier patch, but to find it you need
to "setopt multibyte" in .zshrc.

-- 
Peter Stephenson <pws@csr.com>                  Software Engineer
CSR PLC, Churchill House, Cambridge Business Park, Cowley Road
Cambridge, CB4 0WZ, UK                          Tel: +44 (0)1223 692070


To access the latest news from CSR copy this link into a web browser:  http://www.csr.com/email_sig.php

Re: compaudit problem

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 651 bytes --]

On Wed, Oct 18, 2006 at 02:31:13PM +0100, Peter Stephenson wrote:
> Alexey Tourbin <at@altlinux.ru> wrote:
> > > Given that, according to your build script, you compiled both the before-
> > > and after-22557 sources with --enable-multibyte, the only change in 22557
> > > that could possibly be related is this one:
> >
> > Nope.  But reverting Src/options.c part fixes the problem.
> > I'm kind of puzzled.
> 
> That part turns on the MULTIBYTE option by default.  It sounds like
> the real problem was in an earlier patch, but to find it you need
> to "setopt multibyte" in .zshrc.

Thanks for the clue.  git-bisect now blames 22544.

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: compaudit problem

[Alexey Tourbin]

[-- Attachment #1: Type: text/plain, Size: 3662 bytes --]

On Wed, Oct 18, 2006 at 06:20:19PM +0100, Peter Stephenson wrote:
> Alexey Tourbin <at@altlinux.ru> wrote:
> > Thanks for the clue.  git-bisect now blames 22544.
> 
> That patch made the shell smarter about finding the end of
> special types of string known to the shell (identifiers in particular),
> using the multibyte code.
> 
> I wonder if it's part of the problem Andrey noted?  At some points the
> string we apply this too may contain tokenized characters, which
> aren't valid multibyte characters.  Since the string must be metafied,
> these are easy to detect.
> 
> The simplest fix is just to ensure we don't try to handle these as
> mulitbyte characters, telling the caller they're invalid.  Most callers
> will just handle it as a single-byte character and move on, which
> is the right thing to do; some callers which really need valid characters
> will abort, but they shouldn't be getting a tokenized string.  So
> this might actually work.  If not, we need to be smarter, but probably at a
> higher level.
> 
> We need some fix like this even if it isn't the root of the present
> problem.  (If I could reproduce that it ought now to be easy to trace.)
> 
> Index: Src/utils.c
> ===================================================================
> RCS file: /cvsroot/zsh/zsh/Src/utils.c,v
> retrieving revision 1.142
> diff -u -r1.142 utils.c
> --- Src/utils.c	10 Oct 2006 09:37:19 -0000	1.142
> +++ Src/utils.c	18 Oct 2006 17:09:16 -0000
> @@ -4003,6 +4003,21 @@
>  	    *wcp = (wint_t)(*s == Meta ? s[1] ^ 32 : *s);
>  	return 1 + (*s == Meta);
>      }
> +    /*
> +     * We have to handle tokens here, since we may be looking
> +     * through a tokenized input.  Obviously this isn't
> +     * a valid multibyte character, so just return WEOF
> +     * and let the caller handle it as a single character.
> +     *
> +     * TODO: I've a sneaking suspicion we could do more here
> +     * to prevent the caller always needing to handle invalid
> +     * characters specially, but sometimes it may need to know.
> +     */
> +    if (itok(*s)) {
> +	if (wcp)
> +	    *wcp = EOF;
> +	return 1;
> +    }
>  
>      ret = MB_INVALID;
>      for (ptr = s; *ptr; ) {

Thanks Peter!  This patch resolves the problem.

(I quote the whole message because apparently it was not CC'ed to
zsh-wokers.)

Unfortunately I don't quite understand unicode issues in zsh.  I build
zsh rpm package because I use it (and a few others use it, too).  The
latest stable 4.2 release had problems in utf8 console, so I decided
to move to then-current cvs snapshot.  I got my first decently working
utf8-enabled zsh with 20050926 snapshot.

So as for now there's just about the only thing I can provide is feedback.
This will change as I grok zsh code.

BTW, git archive is available at
git://git.altlinux.org/people/at/packages/zsh.git
The 'master' branch is for my own cooking, but "cvs" branch, as well
as "zsh-4_0-patches" and "zsh-4_2-patches" have pristine zsh sources.
I verified "cvs" branch against checkout, and it's almost zero-diff
(the only exception is that there's very old Completion/Core/_closequotes
is in there, but is not in checkout).  I used Keith Packard's "parsecvs"
(with my changes, some of which already merged into mainline).

> -- 
> Peter Stephenson <pws@csr.com>                  Software Engineer
> CSR PLC, Churchill House, Cambridge Business Park, Cowley Road
> Cambridge, CB4 0WZ, UK                          Tel: +44 (0)1223 692070
> 
> 
> To access the latest news from CSR copy this link into a web browser:  http://www.csr.com/email_sig.php

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: compaudit problem

[Peter Stephenson]

Alexey Tourbin wrote:
> Thanks Peter!  This patch resolves the problem.

Excellent.  I was betting it wouldn't be that simple.

> (I quote the whole message because apparently it was not CC'ed to
> zsh-wokers.)

This time I was so convinced I'd sent it to the list I was waiting
for it to come back before comitting it.  I'll do that now.

-- 
Peter Stephenson <pws@csr.com>                  Software Engineer
CSR PLC, Churchill House, Cambridge Business Park, Cowley Road
Cambridge, CB4 0WZ, UK                          Tel: +44 (0)1223 692070


To access the latest news from CSR copy this link into a web browser:  http://www.csr.com/email_sig.php