Date: Thu, 26 Apr 2018 10:00:12 -0600 From: Warner Losh <imp@bsdimp.com> To: Ed Maste <emaste@freebsd.org> Cc: src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org, Warner Losh <imp@freebsd.org> Subject: Re: svn commit: r333017 - head Message-ID: <CANCZdfo=Co5abvTeGUf-6JzcvrRMtc26T1QA9Av0=GfR6XPriA@mail.gmail.com> In-Reply-To: <CAPyFy2AmCOvXKeH8BkiJ2t2OP0wj%2BKi1gMRDy-PzL_qUSS3enQ@mail.gmail.com> References: <201804261318.w3QDIPNu069198@repo.freebsd.org> <CAPyFy2AmCOvXKeH8BkiJ2t2OP0wj%2BKi1gMRDy-PzL_qUSS3enQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Apr 26, 2018 at 7:47 AM, Ed Maste <emaste@freebsd.org> wrote: > On 26 April 2018 at 09:18, Ed Maste <emaste@freebsd.org> wrote: > > > > Log: > > Correct r307825 BOOTSTRAPPING test for unsupported upgrade > > > > __FreeBSD_version has five digits after the major number, but the min > > versions in the test had an extra 0 (so the test was always false). > > With the test adjusted I believe we should now set the minimum > supported rel to 10.3. No 9.x release contains r286035, so if > reporting a minimum release (as opposed to SVN rev) then 10.3 is the > first release with the fix (r286033). > > That said, is it time to just bump the minimum and remove the special > cases? > I added the original case so that up-to-date stable-9 systems that were at (Juniper?) could work. The bug was such that we don't know that it wasn't being used. It would never trigger on those systems, so people could be using it and we'll not know it. Basically, we should ask Juniper first before removing it since they specifically asked me to allow it. Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfo=Co5abvTeGUf-6JzcvrRMtc26T1QA9Av0=GfR6XPriA>