From owner-dev-commits-src-all@freebsd.org Sat Jul 31 15:37:58 2021 Return-Path: Delivered-To: dev-commits-src-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id AD43D65FF50; Sat, 31 Jul 2021 15:37:58 +0000 (UTC) (envelope-from kp@FreeBSD.org) Received: from smtp.freebsd.org (smtp.freebsd.org [96.47.72.83]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4GcT1L4Gl0z4g3T; Sat, 31 Jul 2021 15:37:58 +0000 (UTC) (envelope-from kp@FreeBSD.org) Received: from venus.codepro.be (venus.codepro.be [5.9.86.228]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "mx1.codepro.be", Issuer "R3" (verified OK)) (Authenticated sender: kp) by smtp.freebsd.org (Postfix) with ESMTPSA id 6184A7A8D; Sat, 31 Jul 2021 15:37:58 +0000 (UTC) (envelope-from kp@FreeBSD.org) Received: by venus.codepro.be (Postfix, authenticated sender kp) id AAFA8154AC; Sat, 31 Jul 2021 17:37:56 +0200 (CEST) From: "Kristof Provost" To: "Warner Losh" Cc: "Warner Losh" , src-committers , dev-commits-src-all@freebsd.org, dev-commits-src-branches@freebsd.org Subject: Re: git: 6edf5082bab7 - stable/12 - one-true-awk: import 20210221 (1e4bc42c53a1) which fixes a number of bugs Date: Sat, 31 Jul 2021 17:37:55 +0200 X-Mailer: MailMate (1.13.2r5673) Message-ID: <8300183E-DD11-4C2F-AE5A-0B6FDE6E9E8F@FreeBSD.org> In-Reply-To: References: <202107101712.16AHCwoE088447@gitrepo.freebsd.org> <4AD3AB8C-A960-43BE-A89B-5D9C4CD077DD@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: quoted-printable X-BeenThere: dev-commits-src-all@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for all branches of the src repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 31 Jul 2021 15:37:58 -0000 On 31 Jul 2021, at 16:20, Warner Losh wrote: > On Sat, Jul 31, 2021 at 8:07 AM Warner Losh wrote: > >> >> >> On Sat, Jul 31, 2021 at 3:11 AM Kristof Provost = >> wrote: >> >>> Hi Warner, >>> >>> On 10 Jul 2021, at 19:12, Warner Losh wrote: >>> >>> The branch stable/12 has been updated by imp: >>> >>> URL: >>> https://cgit.FreeBSD.org/src/commit/?id=3D6edf5082bab71cf923efff9f18e= 38efe5b83b0ec >>> >>> commit 6edf5082bab71cf923efff9f18e38efe5b83b0ec >>> Author: Warner Losh >>> AuthorDate: 2021-07-07 23:30:35 +0000 >>> Commit: Warner Losh >>> CommitDate: 2021-07-10 17:11:15 +0000 >>> >>> one-true-awk: import 20210221 (1e4bc42c53a1) which fixes a number of = >>> bugs >>> >>> Import the latest bsd-features branch of the one-true-awk upstream: >>> >>> o Move to bison for $YACC >>> o Set close-on-exec flag for file and pipe redirects that aren't = >>> std* >>> o lots of little fixes to modernize ocde base >>> o free sval member before setting it >>> o fix a bug where a{0,3} could match aaaa >>> o pull in systime and strftime from NetBSD awk >>> o pull in fixes from {Net,Free,Open}BSD (normalized our code with = >>> them) >>> o add BSD extensions and, or, xor, compl, lsheift, rshift (mostly a = >>> nop) >>> >>> Also revert a few of the trivial FreeBSD changes that were done = >>> slightly >>> differently in the upstreaming process. Also, our PR database may = >>> have >>> been mined by upstream for these fixes, and Mikolaj Golub may = >>> deserve >>> credit for some of the fixes in this update. >>> >>> Suggested by: Mikolaj Golub >>> PR: 143363,143365,143368,143369,143373,143375,214782 >>> Sponsored by: Netflix >>> >>> (cherry picked from commit f39dd6a9784467f0db5886012b3f4b13899be6b8) >>> >>> I *think* this is the relevant commit, but I=E2=80=99m not 100% sure.= >>> >>> Anyway, I=E2=80=99ve recently started seeing issues building pfSense = for = >>> aarch64. >>> I=E2=80=99ve finally narrowed it down, and replicated the problem on = base = >>> FreeBSD >>> (stable/12, but I assume other branches are affected too). >>> >>> It fails to build proctab.c. This file is generated by maketab, a = >>> tool >>> built as part of the build process. In my case this tool is built = >>> for >>> aarch64, which makes executing it on amd64 less successful than = >>> we=E2=80=99d like. >>> >>> This only happens when building with META_MODE though. I do not >>> understand the build system sufficiently to debug why. Hopefully = >>> you=E2=80=99ll >>> have a better idea. >>> >> Generally I do, but this one is a bit weird. Also, the amount of = >> context >> you've included omits the actual error :( Though I'm guessing it's = >> wrong >> arch for the build tool... >> >> I rarely use meta mode, and understand it not at all, but awk follows = >> the >> same patterns as /bin/sh and /bin/csh does >> > > [[ stupid gmail ]] > > I've just pushed removal of proctab.c. That may fix this, though I'm = > not > hopeful. Unfortunately that didn=E2=80=99t help. > I've rarely used metamode, and not for quite some time, but this > an extra file that would confuse things so that maketab might not be = > built > during build-tools and instead is built later. Please give that a try, = > and > I'll start a meta build, but that takes a while, not least because I = > have > to re-leran how to do that :) > I=E2=80=99ve posted my build log and the proctab.c.meta and maketab.meta = files = on https://people.freebsd.org/~kp/meta/ I=E2=80=99m a little confused by maketab.meta, because what=E2=80=99s in = the meta = file doesn=E2=80=99t seem to match the build command seen in the metamode= =2Elog = file. Best regards, Kristof