Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 14 Jul 2020 08:31:02 -0700
From:      Adrian Chadd <adrian@freebsd.org>
To:        Jung-uk Kim <jkim@freebsd.org>
Cc:        src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>,  svn-src-head <svn-src-head@freebsd.org>, Kyle Evans <kevans@freebsd.org>
Subject:   Re: svn commit: r363171 - head/contrib/flex/src
Message-ID:  <CAJ-VmokQmuNWs7nOG5ZsE3-uA4tG_GhLyaoYvLxrNJFMM1MsEQ@mail.gmail.com>
In-Reply-To: <953fd975-215b-2d53-23e6-58faa6af9c96@FreeBSD.org>
References:  <202007140500.06E508mV016928@repo.freebsd.org> <953fd975-215b-2d53-23e6-58faa6af9c96@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Hi!

On Tue, 14 Jul 2020 at 08:00, Jung-uk Kim <jkim@freebsd.org> wrote:

> On 20. 7. 14., Adrian Chadd wrote:
> > Author: adrian
> > Date: Tue Jul 14 05:00:08 2020
> > New Revision: 363171
> > URL: https://svnweb.freebsd.org/changeset/base/363171
> >
> > Log:
> >   [flex] Fix compilation issue under mips-gcc-6.4
> >
> >   This was triggering a pointer-losing-constness error.
>


> It was discussed in this ML.
>
> https://lists.freebsd.org/pipermail/svn-src-head/2020-June/137432.html
>
> kevans even filed an upstream pull request.
>
> https://github.com/westes/flex/pull/449


Oh good!


>
>
> Why did you commit it without asking us?
>

Oh I'm sorry! I didn't realise you were already doing the work to get it
fixed upstream. I just saw things broken when building my test wifi setups
here at home and saw they were small fixes that could be submitted upstream.

I'm personally not a huge fan of sitting on patches that
fix builds/functionality whilst waiting for upstream - this was one of the
many reasons for migrating from CVS to SVN in the long distant past! - so I
figured it would be easy to commit.

I'll make sure I ask around a little more next time!



-adrian


> Jung-uk Kim
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-VmokQmuNWs7nOG5ZsE3-uA4tG_GhLyaoYvLxrNJFMM1MsEQ>