From owner-dev-commits-src-branches@freebsd.org Sat Jul 31 14:07:56 2021 Return-Path: Delivered-To: dev-commits-src-branches@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 B338365EA1D for ; Sat, 31 Jul 2021 14:07:56 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-qk1-x736.google.com (mail-qk1-x736.google.com [IPv6:2607:f8b0:4864:20::736]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4GcR1S42S3z4Yln for ; Sat, 31 Jul 2021 14:07:53 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by mail-qk1-x736.google.com with SMTP id az7so12314431qkb.5 for ; Sat, 31 Jul 2021 07:07:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=WChS9ED6kXrkCAMIJGH1fgBBPx5XGUxS+t+cL48J2p0=; b=DZD1rrtv2W34F8p0Wmj2tv5YGNMGsnaUnaVZioxZZ34r46rby62rAQi3fhZ8tLWo+B ZDKLm1qPxM7q9LiP+yTInjBJjVBRDVhvBjuJ/6kjME1aPD9UnZFDF+zGLAXvP+rS5/dL O/oV7+f8ANq5wjDfw3SlEsZ72KUmxlggPeu6BnXjCpgM3Q8DB0s4JAqN0AuLv9HxXfwm M1d17NLiul7k1buANmrvh56PKEU9+DLqxJw/7mnBpqNhPeH0xw+984G1Rehi4fQo/GOr BjEI1OOF1p+SXG/nDGzdPyZ/Jq3BbIFoNes9/tKT0mpUKG49T1A/qTjwp21bCDB9W5sP oOQw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=WChS9ED6kXrkCAMIJGH1fgBBPx5XGUxS+t+cL48J2p0=; b=HoEh4aCDCBaaGcEBKv6pzk2Tn3AxYMXE4X6Ma0rXE7uZQAvmnpIFC9AQ9ArwL5UZf3 A6LrPKLyEn6E6p74z7GymXNfnBUefKrLXaVXMrpyr4YR+Zr7nTr607wPkQaZF9M3nQZY RiQYGEvDc4DIlpmTUgeUTLQ8bDIcFhqU1XhJroGO5EVy/M0uLYB6ht723fSAfA2p41LR NH1/7HogS5b3TUdckpBoLK11eAlp5245GO04FsSpgo7G6pPVu4JoTPTvAlMhOtGPgwiz 5UcHELieIO26Rea4K27xg8EVYnL/s/g6c4anl2Om6ZdMpemmmAHlfKr+YigjVK1MgH5O +yZA== X-Gm-Message-State: AOAM5306NGRsUCYTPSOavdDWG4T4jszvG0L5sE/qhAuWZjTDO68JN2O2 PpRqxxPsEOIqtA8ObQ6gy+uUEBTSIgJc2iIDYzMkXg== X-Google-Smtp-Source: ABdhPJwsPLqp5FLdWccCIayLPUi+O5jvZzjsowpftPugCrRdOt58B7ykeqhSD/1tQxRVG9f/vZuh7Vs18UtUT2EW4GU= X-Received: by 2002:a05:620a:45:: with SMTP id t5mr6862413qkt.206.1627740472992; Sat, 31 Jul 2021 07:07:52 -0700 (PDT) MIME-Version: 1.0 References: <202107101712.16AHCwoE088447@gitrepo.freebsd.org> <4AD3AB8C-A960-43BE-A89B-5D9C4CD077DD@FreeBSD.org> In-Reply-To: <4AD3AB8C-A960-43BE-A89B-5D9C4CD077DD@FreeBSD.org> From: Warner Losh Date: Sat, 31 Jul 2021 08:07:42 -0600 Message-ID: Subject: Re: git: 6edf5082bab7 - stable/12 - one-true-awk: import 20210221 (1e4bc42c53a1) which fixes a number of bugs To: Kristof Provost Cc: Warner Losh , src-committers , "" , dev-commits-src-branches@freebsd.org X-Rspamd-Queue-Id: 4GcR1S42S3z4Yln X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.34 X-BeenThere: dev-commits-src-branches@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commits to the stable branches of the FreeBSD src repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 31 Jul 2021 14:07:56 -0000 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=3D6edf5082bab71cf923efff9f18e38ef= e5b83b0ec > > 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 understan= d > 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. Warner This triggers the build failure: > > % cat /usr/src/src_env.conf > WITH_META_MODE=3Dyes > % env SRC_ENV_CONF=3D/usr/src/src_env.conf TARGET_ARCH=3Daarch64 make -j = 8 buildworld > > =E2=80=A6 > --- proctab.c --- > *** [proctab.c] Error code 255 > > make[4]: stopped in /usr/src/usr.bin/awk > .ERROR_TARGET=3D'proctab.c' > .ERROR_META_FILE=3D'/usr/obj/usr/src/arm64.aarch64/usr.bin/awk/proctab.c.= meta' > .MAKE.LEVEL=3D'4' > MAKEFILE=3D'' > .MAKE.MODE=3D'meta missing-filemon=3Dyes missing-meta=3Dyes silent=3Dyes = verbose' > _ERROR_CMD=3D'./maketab awkgram.h > proctab.c;' > .CURDIR=3D'/usr/src/usr.bin/awk' > .MAKE=3D'make' > .OBJDIR=3D'/usr/obj/usr/src/arm64.aarch64/usr.bin/awk' > .TARGETS=3D'all' > DESTDIR=3D'/usr/obj/usr/src/arm64.aarch64/tmp' > LD_LIBRARY_PATH=3D'' > MACHINE=3D'arm64' > MACHINE_ARCH=3D'aarch64' > MAKEOBJDIRPREFIX=3D'' > MAKESYSPATH=3D'/usr/src/share/mk' > MAKE_VERSION=3D'20200710' > > Further up in the build output: > > --- maketab --- > cc -target aarch64-unknown-freebsd12.2 --sysroot=3D/usr/obj/usr/src/arm64= .aarch64/tmp -B/usr/obj/usr/src/arm64.aarch64/tmp/usr/bin -O2 -pipe -fno-co= mmon -DHAS_ISBLANK -I. -I/usr/src/contrib/one-true-awk -DFOPEN_MAX=3D64 -= g -std=3Dgnu99 -Wno-format-zero-length -fstack-protector-strong -Wsystem-he= aders -Werror -Wno-pointer-sign -Wno-empty-body -Wno-string-plus-int -Wno-u= nused-const-variable -Wno-tautological-compare -Wno-unused-value -Wno-paren= theses-equality -Wno-unused-function -Wno-enum-conversion -Wno-unused-local= -typedef -Wno-address-of-packed-member -Wno-switch -Wno-switch-enum -Wno-kn= r-promoted-parameter -Wno-parentheses -Qunused-arguments /usr/src/contr= ib/one-true-awk/maketab.c -o maketab > > Without META_MODE maketab is built as an amd64 binary, and the entire > build succeeds. > > Best regards, > Kristof >