On Thu, Sep 13, 2018 at 06:51:16AM +0200, Rafael Sadowski wrote:
> On Mon Sep 10, 2018 at 03:10:09PM +0200, Pierre-Emmanuel Andre wrote:
> > On Sat, Sep 08, 2018 at 02:20:49PM -0500, Matthew Martin wrote:
> > > On Sat, Sep 08, 2018 at 08:36:29AM +0200, Rafael Sadowski wrote:
> > > > Hi All.
> > > > 
> > > > please find below a simple diff to update ZSH to the latest stable
> > > > version. This release includes security fixes: CVE-2018-0502,
> > > > CVE-2018-13259.
> > > 
> > > zsh 5.6.1 will be out soon (it's tagged and built; just not announced).
> > > It fixes the error which (I suspect) is why you switched to gmake.
> > > 
> > > > Notable port changes:
> > > > 
> > > > - Use configure elements instead of post-install tasks.
> > > > -- Avoid version in path name. I don't see any reasons for that, I only
> > > >    provide one ZSH version.
> > > 
> > > This is the upstream default, and I don't see a compelling reason to
> > > deviate from it. 
> > > 
> > > Patch for 5.6.1 below. It was rolled a bit over an hour ago, so not well
> > > tested yet. Obviously we should wait until it's officially announced.
> > > There's one test error over what 5.5.1 had which I haven't had time to
> > > look at yet.
> > >
> > 
> > 
> > I agree with Matthew. I prefer to wait for 5.6.1.
> > 
> 
> I also agree with Matthew. Thank you for your feedback, Matthew.
> 
> Matthew's and my diff below. Feedback? Ok?
>


Thanks. There is a regress failure:

./A05execution.ztst: starting.
Pattern match failed, line 1:
-<*tstcmd-interp-too-long: bad interpreter: x*xn: no such file or directory
 <status 127
 +>(eval):2: 
/usr/ports/pobj/zsh-5.6.1/zsh-5.6.1/Test/command.tmp//tstcmd-interp-too-long: 
bad interpreter: 
xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx$
 xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxn: exec format error
  >status 126
  Test ./A05execution.ztst failed: output differs from expected as shown above 
for:
    path=($shpath $echopath ${ZTST_testdir}/command.tmp/)
      tstcmd-interp-too-long 2>&1; echo "status $?"
        path=($storepath)
        Was testing: path (5)
        ./A05execution.ztst: test failed.


This test success with the 5.5.1.
I will look at it.

Reply via email to