Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 12 Feb 2021 15:00:41 -0500
From:      Mark Johnston <markj@freebsd.org>
To:        Jung-uk Kim <jkim@freebsd.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r362333 - in head: contrib/flex contrib/flex/src usr.bin/lex usr.bin/lex/lib
Message-ID:  <YCbeabF%2BBSrpQep0@raichu>
In-Reply-To: <202006181809.05II9G8p054025@repo.freebsd.org>
References:  <202006181809.05II9G8p054025@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Jun 18, 2020 at 06:09:16PM +0000, Jung-uk Kim wrote:
> Author: jkim
> Date: Thu Jun 18 18:09:16 2020
> New Revision: 362333
> URL: https://svnweb.freebsd.org/changeset/base/362333
> 
> Log:
>   MFV:	r362286
>   
>   Merge flex 2.6.4.

This seems to have introduced a regression: input() now returns 0
instead of EOF to indicate that the end of input was reached.  This has
been reported in a few places:

https://github.com/westes/flex/issues/448
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=911415

It breaks the scanner used by libdtrace, and as a result dtrace is
unable to resolve some probe argument types:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=253440

I have a small patch to libdtrace which works around this.  The upstream
commit that introduced the change is fairly old at this point, and
upstream seems somewhat unresponsive, so I'm not sure it'll get
reverted.  Some other scanners in the tree look like they'd be affected,
like crypto/heimdal/lib/asn1/lex.l or
sys/contrib/dev/acpica/compiler/aslsupport.l, so I'm wondering if we
should revert this particular change for 13.0.  Certainly we should fix
the DTrace bug one way or another for 13.0.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?YCbeabF%2BBSrpQep0>