From owner-svn-src-all@freebsd.org Wed May 23 20:21:05 2018 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 3C067EB0816; Wed, 23 May 2018 20:21:05 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2610:1c1:1:6074::16:84]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "freefall.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id CB4667CBC5; Wed, 23 May 2018 20:21:04 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (unknown [127.0.1.132]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by freefall.freebsd.org (Postfix) with ESMTPS id 95C25D865; Wed, 23 May 2018 20:21:04 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (localhost [172.31.3.2]) by mail.xzibition.com (Postfix) with ESMTP id 67B5656F9; Wed, 23 May 2018 20:21:03 +0000 (UTC) X-Virus-Scanned: amavisd-new at mail.xzibition.com Received: from mail.xzibition.com ([172.31.3.2]) by mail.xzibition.com (mail.xzibition.com [172.31.3.2]) (amavisd-new, port 10026) with LMTP id eVT27xm-CJ3x; Wed, 23 May 2018 20:21:00 +0000 (UTC) Subject: Re: svn commit: r334008 - head/bin/sh DKIM-Filter: OpenDKIM Filter v2.10.3 mail.xzibition.com 8CC1C56F4 From: Bryan Drewery To: "O. Hartmann" , Cy Schubert , David Wolfskill Cc: Jilles Tjoelker , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org References: <20180522061339.2149763e@freyja.zeit4.iv.bundesimmobilien.de> <201805220445.w4M4jroR019550@slippy.cwsent.com> <20180522101737.52e76c0f@freyja.zeit4.iv.bundesimmobilien.de> <20180522202259.GA44110@stack.nl> Openpgp: id=F9173CB2C3AAEA7A5C8A1F0935D771BB6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc Autocrypt: addr=bdrewery@FreeBSD.org; prefer-encrypt=mutual; keydata= xsBNBFJphmsBCADiFgmS4bIzwZijrS31SjEMzg+n5zNellgM+HkShwehpqCiyhXdWrvH6dTZ a6u50pbUIX7doTR7W7PQHCjCTqtpwvcj0eulZva+iHFp+XrbgSFHn+VVXgkYP2MFySyZRFab D2qqzJBEJofhpv4HvY6uQI5K99pMqKr1Z/lHqsijYYu4RH2OfwB5PinId7xeldzWEonVoCr+ rfxzO/UrgA6v/3layGZcKNHFjmc3NqoN1DXtdaEHqtjIozzbndVkH6lkFvIpIrI6i5ox8pwp VxsxLCr/4Musd5CWgHiet5kSw2SzNeA8FbxdLYCpXNVu+uBACEbCUP+CSNy3NVfEUxsBABEB AAHNJEJyeWFuIERyZXdlcnkgPGJkcmV3ZXJ5QEZyZWVCU0Qub3JnPsLAgAQTAQoAKgIbAwUL CQgHAwUVCgkICwUWAwIBAAIeAQIXgAIZAQUCWujOIgUJCmB7NwAKCRA113G7bkaXz/xpB/9b /UWIPbieY1IeIuHF2pyYPE7Hytkh3HVsxMA0F5Ma2AYQsXZZeKNKWrF7RPyDyDwUklLHJkhm k3EfClBbHxf08kMIm1vWCJRtgxic9knY/bzYGiWMpHjg3cSd1XfrYH1autYqTZAjDwIkgOjU dR//Tbn4V36sY7y2jz+kdMVWvK53U32aZqiwBbCn4DPe1wSZcUs17mV/0uZdIoGdj74B1orN A/0py5vHYo6HcbBNoaR8pKRLf5VZNRsxqGIMhTucx4SJWcHpuRBWYyvJSFzwvxdK4ZD4Yqoc kFGPVtOXktVMai9exrLvP3G77fKMu8DI6j4QRU4wCesnHuIfRPFuzsBNBFJphmsBCACiVFPf kNfaFtUSuY0395ueo/rMyHPGPQ2iwvERFCpeFGSQSgagpenNHLpFQKTg/dl6FOoST5tqyxMq fyHGHDzzU51bvA/IfaGoNi/BIhTe/toZNMRvpcI3PLjiGcnJnuwCCbAVOAGdb+t5cZtpNdOI cKYmrYG3u9RiBpe6dTF+qLrD/8Bs1wjhduQ8fcNNgnkXu8xDH4ZxY0lIc3QgvYWp9vimlQe6 iKjUd2/DX28ETZcD5h6pYV331KMPTrEI0p0yvFijUZce8c1XHFyL1j9sBAha5qpszJl6Uq5i LolhKRcGfcdmtD72vHQjUYglUyudSJUVyo2gMYjdbiFKzJulABEBAAHCwGUEGAEKAA8CGwwF AlrozigFCQpgez0ACgkQNddxu25Gl8+m5Af/R3VEdxNMAcDIes9ADhQyofj20SPV3eCJ3HYR OebTSuNdOudGt4AAyA8Ks94u9hiIp5IGsc6RDsT9W7O2vgXhd6eV3eiY5Oif5xLIYrIDVu1Y 1GyRxRrPEn/QOqDN6uFZCPwK1aOapGcYCrO9lB0gMuTVfgHanU61rgC9tMX0OoAOyRd+V3/M 8lDNhjJdF/IpO3SdYzKfkwduy4qamw4Gphcx/RfYQvYLq/eDkP8d50PphWdboqWBwNRHayro W/07OGzfxM5fJ5mBsXPQcO2QcRjkyHf6xCM6Hi1qQL4OnXMNE/ZTX0lnOj1/pH93TlzSHZMP TaiiA/MBD3vGsXBmBg== Organization: FreeBSD Message-ID: <1e2b1d35-d3ac-34b6-cb68-5e52605e2a31@FreeBSD.org> Date: Wed, 23 May 2018 13:20:58 -0700 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="r0SjQLVt0aVOsWRU8pTqlNVwklafpmlAH" X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 23 May 2018 20:21:05 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --r0SjQLVt0aVOsWRU8pTqlNVwklafpmlAH Content-Type: multipart/mixed; boundary="m4hjhAuQR2Z5ZyU2z6tcTDiTQu04EKQ2E"; protected-headers="v1" From: Bryan Drewery To: "O. Hartmann" , Cy Schubert , David Wolfskill Cc: Jilles Tjoelker , src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Message-ID: <1e2b1d35-d3ac-34b6-cb68-5e52605e2a31@FreeBSD.org> Subject: Re: svn commit: r334008 - head/bin/sh References: <20180522061339.2149763e@freyja.zeit4.iv.bundesimmobilien.de> <201805220445.w4M4jroR019550@slippy.cwsent.com> <20180522101737.52e76c0f@freyja.zeit4.iv.bundesimmobilien.de> <20180522202259.GA44110@stack.nl> In-Reply-To: --m4hjhAuQR2Z5ZyU2z6tcTDiTQu04EKQ2E Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable On 5/22/2018 5:30 PM, Bryan Drewery wrote: > On 5/22/2018 1:22 PM, Jilles Tjoelker wrote: >> On Tue, May 22, 2018 at 10:17:41AM +0200, O. Hartmann wrote: >>> On Mon, 21 May 2018 21:45:53 -0700 >>> Cy Schubert wrote: >>>> In message <20180522061339.2149763e@freyja.zeit4.iv.bundesimmobilien= =2Ede> >>>> , "O. H >>>> artmann" writes: >>>>> On Mon, 21 May 2018 21:52:48 +0000 (UTC) >>>>> Jilles Tjoelker wrote: >> >>>>>> Author: jilles >>>>>> Date: Mon May 21 21:52:48 2018 >>>>>> New Revision: 334008 >>>>>> URL: https://svnweb.freebsd.org/changeset/base/334008 >> >>>>>> Log: >>>>>> sh: Split CNL syntax category to avoid a check on state[level].s= yntax >> >>>>>> No functional change is intended. >> >>>>>> Modified: >>>>>> head/bin/sh/mksyntax.c >>>>>> head/bin/sh/parser.c >>> [snip] >> >>>>> Have this been tested? Doesn't compile for me: >> >>>>> [...] >>>>> Building /usr/obj/usr/src/amd64.amd64/kerberos5/libexec/hprop/hprop= >>>>> --- all_subdir_rescue --- >>>>> --- parser.o --- >>>>> /usr/src/bin/sh/parser.c:1440:9: error: use of undeclared identifie= r 'CQNL' >>>>> case CQNL: >>>>> ^ >>>>> --- all_subdir_gnu --- >>>>> Building /usr/obj/usr/src/amd64.amd64/gnu/usr.bin/gdb/libgdb/amd64b= sd-nat.o >>>>> --- all_subdir_rescue --- >>>>> 1 error generated. >>>>> *** [parser.o] Error code 1 >> >>>>> make[6]: stopped in /usr/src/bin/sh >> >>>> CQNL is defined in /usr/obj/opt/src/svn-current/amd64.amd64/bin/sh/s= ynta >>>> x.h, generated by mksyntax. >> >>>> slippy$ ag -s CQNL /export/obj/opt/src/svn-current/amd64.amd64/bin/s= h/*. >>>> h >>>> /export/obj/opt/src/svn-current/amd64.amd64/bin/sh/syntax.h >>>> 11:#define CQNL 2 /* newline character in quotes */ >>>> slippy$=20 >> >>>> Remove the file if it's not defined in your syntax.h. >> >>>> Just out of interest, do you use meta mode? >> >>> I think such a question is of common interest if errors/bugs like tha= t occur: >>> Yes, I use/compile world/kernel with META mode. >> >> The change itself is fine. It built for me and for Jenkins >> (ci.freebsd.org). What is not fine is an incremental build with meta >> mode. Apparently, the syntax.h: .NOMETA rule added in r301285 causes= >> bmake to build some files against the old syntax.h, even though syntax= =2Ec >> and syntax.h will be rebuilt. >> >> To fix this, it may be possible to generate a meta file for syntax.h >> based on the one for syntax.c. The same would be done for builtins.[ch= ] >> and nodes.[ch]. >> >> Conceptually simpler is accepting what make would like: one command >> generates one file only. This is not really new with meta mode since a= >=20 > Yeah bmake (both with and without meta mode) is lacking in properly > handling 1 target generating multiple files. It's a big frustration of= > mine as every pattern I've seen does not do the right thing. I'll look= > into this case more. For now just remove the syntax.h file from the > objdir or remove the bin/sh dir. >=20 >> somewhat ugly .ORDER declaration had been necessary before. The .c >> content can go inside a #ifdef in the .h file so the .c file need not = be >> autogenerated, or the tools can be run twice, once to generate the .c >> file and once to generate the .h file. In both cases, the tools will b= e >> somewhat uglier in order to simplify the build system. >> >=20 >=20 The problem is the rescue build is doing 'make foo.o bar.o' rather than 'make all', so the implicit 'make depend' phase is skipped. Switching rescue (crunchgen) to use 'make all' fixes it. I am testing a full build of that now and will commit it soon. The .NOMETA does certainly cause the problem but 'make all' ensures the implicit 'make depend' will always run for all modes as intended. --=20 Regards, Bryan Drewery --m4hjhAuQR2Z5ZyU2z6tcTDiTQu04EKQ2E-- --r0SjQLVt0aVOsWRU8pTqlNVwklafpmlAH Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBAgAGBQJbBc0qAAoJEDXXcbtuRpfPsKsH/1ruDtJNevoxWNcs3yxOdEZc IJocanEV2gL8mUficsUWwui6tf/UjR2saaiSCrbDssRA0+J/KEC3t1SoPB/n/qNJ JZ6lgfugIfGVVlmZg7Jfqh4J9J+4ZQu/ip3helR3NUTW401Z6Ves84x0bsZmQO9Q DeBS1uOYp98/O9huiDTR8XWQQqmqR/tU/dhVcMxzNUenQkI7D9cmgYCLzZ/HZ8s3 tMCSoM4x94T5+cJ3kW4t8Er/2SluSRWbTE2u23OThzppiF1thcwoWk+SnAPBTKpX dLVTKSuW1K+BR5LQNHkdwReHwUIBxAl5qAfsLWC1mstoNzexy6cRHqrWqNzVv4E= =lwBA -----END PGP SIGNATURE----- --r0SjQLVt0aVOsWRU8pTqlNVwklafpmlAH--