From owner-svn-src-head@freebsd.org Tue Jan 19 16:14:01 2016 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 1EA4BA88C49; Tue, 19 Jan 2016 16:14:01 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by mx1.freebsd.org (Postfix) with ESMTP id 0A31618B3; Tue, 19 Jan 2016 16:14:01 +0000 (UTC) (envelope-from bdrewery@FreeBSD.org) Received: from mail.xzibition.com (localhost [IPv6:::1]) by freefall.freebsd.org (Postfix) with ESMTP id F1F3D1597; Tue, 19 Jan 2016 16:14:00 +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 945CA15CA9; Tue, 19 Jan 2016 16:14:00 +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 96hP1_lQLUZ5; Tue, 19 Jan 2016 16:13:53 +0000 (UTC) Subject: Re: svn commit: r291941 - in head: lib/libpam/modules/pam_ssh secure/lib/libssh secure/libexec/sftp-server secure/libexec/ssh-keysign secure/libexec/ssh-pkcs11-helper secure/usr.bin/scp secure/usr.bin/... DKIM-Filter: OpenDKIM Filter v2.9.2 mail.xzibition.com D598615CA4 To: =?UTF-8?Q?Dag-Erling_Sm=c3=b8rgrav?= References: <201512071608.tB7G89BZ095366@repo.freebsd.org> <86vb6phdug.fsf@desk.des.no> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org From: Bryan Drewery Openpgp: id=F9173CB2C3AAEA7A5C8A1F0935D771BB6E4697CF; url=http://www.shatow.net/bryan/bryan2.asc X-Enigmail-Draft-Status: N1110 Organization: FreeBSD Message-ID: <569E60BA.6050007@FreeBSD.org> Date: Tue, 19 Jan 2016 08:13:46 -0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <86vb6phdug.fsf@desk.des.no> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="ImOUi8qoVMxdCLw0lxfNegh6WQNX2pVxe" X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Jan 2016 16:14:01 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --ImOUi8qoVMxdCLw0lxfNegh6WQNX2pVxe Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 1/19/2016 5:56 AM, Dag-Erling Sm=C3=B8rgrav wrote: > Bryan Drewery writes: >> Log: >> Replace unneeded manual dependency on header by adding it to SRCS. >> =20 >> bsd.lib.mk and bsd.prog.mk already depend all objs on headers in SRC= S if >> there is not yet a depend file. The headers in SRCS are never built= or >> installed. After 'make depend' the header was already added as a pr= oper >> dependency on the objects where needed. >=20 > This doesn't work. Try the following: >=20 > $ cd /usr/src/secure/lib/libssh > $ make depend && make > $ touch /usr/src/crypto/openssh/ssh_namespace.h > $ make Well, it sort of works with WITH_FAST_DEPEND which will be default very shortly. > ~/git/freebsd/secure/lib/libssh # make -j15 > ~/git/freebsd/secure/lib/libssh # touch ../../../crypto/openssh/ssh_nam= espace.h > ~/git/freebsd/secure/lib/libssh # make > /usr/local/bin/ccache cc -O2 -pipe -DHAVE_LDNS=3D1 -I/root/git/freeb= sd/secure/lib/libssh/../../../contrib/ldns -I/root/git/freebsd/secure/lib= /libssh/../../../crypto/openssh -include ssh_namespace.h -include krb5_co= nfig.h -MD -MP -MF.depend.authfd.o -MTauthfd.o -std=3Dgnu99 -fstack-prote= ctor-strong -Wno-pointer-sign - > -const-variable -Wno-tautological-compare -Wno-unused-value -Wno-parent= heses-equality -Wno-unused-function -Wno-enum-conversion -Wno-unused-loca= l-typedef -Wno-switch -Wno-switch-enum -Wno-knr-promoted-parameter -Wno-p= arentheses -Qunused-arguments -c /root/git/freebsd/secure/lib/libssh/../= =2E./../crypto/openssh/authfd > /usr/local/bin/ccache cc -O2 -pipe -DHAVE_LDNS=3D1 -I/root/git/freeb= sd/secure/lib/libssh/../../../contrib/ldns -I/root/git/freebsd/secure/lib= /libssh/../../../crypto/openssh -include ssh_namespace.h -include krb5_co= nfig.h -MD -MP -MF.depend.authfile.o -MTauthfile.o -std=3Dgnu99 -fstack-p= rotector-strong -Wno-pointer-si > used-const-variable -Wno-tautological-compare -Wno-unused-value -Wno-pa= rentheses-equality -Wno-unused-function -Wno-enum-conversion -Wno-unused-= local-typedef -Wno-switch -Wno-switch-enum -Wno-knr-promoted-parameter -W= no-parentheses -Qunused-arguments -c /root/git/freebsd/secure/lib/libssh= /../../../crypto/openssh/au > /usr/local/bin/ccache cc -O2 -pipe -DHAVE_LDNS=3D1 -I/root/git/freeb= sd/secure/lib/libssh/../../../contrib/ldns -I/root/git/freebsd/secure/lib= /libssh/../../../crypto/openssh -include ssh_namespace.h -include krb5_co= nfig.h -MD -MP -MF.depend.bufaux.o -MTbufaux.o -std=3Dgnu99 -fstack-prote= ctor-strong -Wno-pointer-sign - > -const-variable -Wno-tautological-compare -Wno-unused-value -Wno-parent= heses-equality -Wno-unused-function -Wno-enum-conversion -Wno-unused-loca= l-typedef -Wno-switch -Wno-switch-enum -Wno-knr-promoted-parameter -Wno-p= arentheses -Qunused-arguments -c /root/git/freebsd/secure/lib/libssh/../= =2E./../crypto/openssh/bufaux > /usr/local/bin/ccache cc -O2 -pipe -DHAVE_LDNS=3D1 -I/root/git/freeb= sd/secure/lib/libssh/../../../contrib/ldns -I/root/git/freebsd/secure/lib= /libssh/../../../crypto/openssh -include ssh_namespace.h -include krb5_co= nfig.h -MD -MP -MF.depend.bufbn.o -MTbufbn.o -std=3Dgnu99 -fstack-protect= or-strong -Wno-pointer-sign -Wn > onst-variable -Wno-tautological-compare -Wno-unused-value -Wno-parenthe= ses-equality -Wno-unused-function -Wno-enum-conversion -Wno-unused-local-= typedef -Wno-switch -Wno-switch-enum -Wno-knr-promoted-parameter -Wno-par= entheses -Qunused-arguments -c /root/git/freebsd/secure/lib/libssh/../..= /../crypto/openssh/bufbn.c > /usr/local/bin/ccache cc -O2 -pipe -DHAVE_LDNS=3D1 -I/root/git/freeb= sd/secure/lib/libssh/../../../contrib/ldns -I/root/git/freebsd/secure/lib= /libssh/../../../crypto/openssh -include ssh_namespace.h -include krb5_co= nfig.h -MD -MP -MF.depend.buffer.o -MTbuffer.o -std=3Dgnu99 -fstack-prote= ctor-strong -Wno-pointer-sign - > -const-variable -Wno-tautological-compare -Wno-unused-value -Wno-parent= heses-equality -Wno-unused-function -Wno-enum-conversion -Wno-unused-loca= l-typedef -Wno-switch -Wno-switch-enum -Wno-knr-promoted-parameter -Wno-p= arentheses -Qunused-arguments -c /root/git/freebsd/secure/lib/libssh/../= =2E./../crypto/openssh/buffer > /usr/local/bin/ccache cc -O2 -pipe -DHAVE_LDNS=3D1 -I/root/git/freeb= sd/secure/lib/libssh/../../../contrib/ldns -I/root/git/freebsd/secure/lib= /libssh/../../../crypto/openssh -include ssh_namespace.h -include krb5_co= nfig.h -MD -MP -MF.depend.canohost.o -MTcanohost.o -std=3Dgnu99 -fstack-p= rotector-strong -Wno-pointer-si > used-const-variable -Wno-tautological-compare -Wno-unused-value -Wno-pa= rentheses-equality -Wno-unused-function -Wno-enum-conversion -Wno-unused-= local-typedef -Wno-switch -Wno-switch-enum -Wno-knr-promoted-parameter -W= no-parentheses -Qunused-arguments -c /root/git/freebsd/secure/lib/libssh= /../../../crypto/openssh/ca > /usr/local/bin/ccache cc -O2 -pipe -DHAVE_LDNS=3D1 -I/root/git/freeb= sd/secure/lib/libssh/../../../contrib/ldns -I/root/git/freebsd/secure/lib= /libssh/../../../crypto/openssh -include ssh_namespace.h -include krb5_co= nfig.h -MD -MP -MF.depend.channels.o -MTchannels.o -std=3Dgnu99 -fstack-p= rotector-strong -Wno-pointer-si > used-const-variable -Wno-tautological-compare -Wno-unused-value -Wno-pa= rentheses-equality -Wno-unused-function -Wno-enum-conversion -Wno-unused-= local-typedef -Wno-switch -Wno-switch-enum -Wno-knr-promoted-parameter -W= no-parentheses -Qunused-arguments -c /root/git/freebsd/secure/lib/libssh= /../../../crypto/openssh/ch > ^Cmake: *** channels.o removed >=20 > The second "make" should rebuild everything, but it doesn't. Also, >=20 > $ grep -c ssh_namespace $(make -V.OBJDIR)/.depend > 0 >=20 That is expected. The dependency is defined in bsd.lib.mk. Granted it is dependent on .depend existing, but I am going to remove that check so it is always defined. --=20 Regards, Bryan Drewery --ImOUi8qoVMxdCLw0lxfNegh6WQNX2pVxe 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 iQEcBAEBAgAGBQJWnmDAAAoJEDXXcbtuRpfPV/8IAMt5x/czMMjby3QdEyh4alir QZzG1q80xHL3SgI6J+p6DKrye7BBqNt3boctFJUHxJ0/SLXjIphjYW6BwkH5R4N8 /p0KDfxLDsrsFPXff7UL5abB35P8v8t/0Dw+YDp5ukjWuVu1AvEbyiWEzOsJCpR/ 7Il8NqufuyVSiAxnAh7YGbsvfM1hStZcIM5VdCk7GkWiYTkHYVazgf+L5ZApcVs4 ZNZdx9X7hrygSDvel8sivsfQlN6xubzQZ7Hwyj8X3HfGaF/mNyxRIxbxqhWi7zb/ vnJG67XM1vJ3yjQvSxtlqtGPTifd1qvhMlmnlq2S6eOrpVcEl+qXr+6Dol12dIU= =3rnz -----END PGP SIGNATURE----- --ImOUi8qoVMxdCLw0lxfNegh6WQNX2pVxe--