zsh-workers
 help / color / mirror / code / Atom feed
From: Martijn Dekker <martijn@inlv.org>
To: Peter Stephenson <p.w.stephenson@ntlworld.com>,
	Zsh hackers list <zsh-workers@zsh.org>
Subject: Re: 5.4 almost released
Date: Tue, 8 Aug 2017 11:26:47 +0100	[thread overview]
Message-ID: <9581a3ad-bdf9-18cb-dc84-9d6fb34e93d5@inlv.org> (raw)
In-Reply-To: <20170807193412.3ff8651c@ntlworld.com>

Op 07-08-17 om 19:34 schreef Peter Stephenson:
> On Mon, 7 Aug 2017 14:25:35 -0400
> Eric Cook <llua@gmx.com> wrote:
>> On 08/07/2017 02:03 PM, Peter Stephenson wrote:
>>> THe files for version 5.4 are now in www.zsh.org/dev.  If a few people
>>> get a chance to check this is OK I'll announce it and tidy up.
>>>
>>> Thanks
>>> pws
>>>
>>
>> pardon the typo, 5.4 is on http://www.zsh.org/pub
> 
> Er, yes, thank you.

The recent POSIX_STRINGS-related change (41499, 2eacbef) introduced at
least one serious bug. I think you should probably revert that commit
before the 5.4 release so it can have more testing before 5.4.1 or
whatever the release after this one is.

$* concatenated, IFS is space:

        emulate sh
        set " abc " " def ghi " " jkl "
        IFS=' '
        set xx$*yy
	echo "$#,$1|$2|$3|$4|$5|$6"

Actual output: 5,xx|abc|def|ghi|jklyy|
Expected output: 6,xx|abc|def|ghi|jkl|yy

The 'yy' is joined to the final 'jkl' instead of becoming a separate
argument.

The same bug happens with $@ (which, when unquoted, acts identically to $*).

Thanks,

- M.


  reply	other threads:[~2017-08-08 10:26 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20170807135641epcas3p4bc0a64f832ae7fcd76051ac198722045@epcas3p4.samsung.com>
2017-08-07 13:55 ` parsing empty alternatives: case foo|) :;; Daniel Shahaf
2017-08-07 14:26   ` Peter Stephenson
2017-08-07 14:44     ` Bart Schaefer
2017-08-07 14:58       ` Peter Stephenson
2017-08-07 18:03         ` 5.4 almost released Peter Stephenson
2017-08-07 18:25           ` Eric Cook
2017-08-07 18:34             ` Peter Stephenson
2017-08-08 10:26               ` Martijn Dekker [this message]
2017-08-08 11:00                 ` Peter Stephenson
2017-08-07 18:21         ` parsing empty alternatives: case foo|) :;; Peter Stephenson
2017-08-07 18:56           ` Peter Stephenson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=9581a3ad-bdf9-18cb-dc84-9d6fb34e93d5@inlv.org \
    --to=martijn@inlv.org \
    --cc=p.w.stephenson@ntlworld.com \
    --cc=zsh-workers@zsh.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).