From owner-svn-src-head@freebsd.org Mon Jan 14 21:14:31 2019 Return-Path: Delivered-To: svn-src-head@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 6721A1487FE1; Mon, 14 Jan 2019 21:14:31 +0000 (UTC) (envelope-from yaneurabeya@gmail.com) Received: from mail-pf1-x434.google.com (mail-pf1-x434.google.com [IPv6:2607:f8b0:4864:20::434]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id C74C176567; Mon, 14 Jan 2019 21:14:30 +0000 (UTC) (envelope-from yaneurabeya@gmail.com) Received: by mail-pf1-x434.google.com with SMTP id h3so218399pfg.1; Mon, 14 Jan 2019 13:14:30 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:message-id:mime-version:subject:date:in-reply-to:cc:to :references; bh=fCSdsPrLH91KcgNFFqNS9d22ksh29iSW6iERDD015pg=; b=liYD0z4TLhbvS8XSZSg7kpfXZlww/AmK69+THLVdZWMdbyIbmebUXl/yW7F3dBUw+f mUwbKF80sxWWshnK/oTKWyFE++XmaLMBH1w1zXUmyfepy7XpjxKHGeX4rC1b15c+1rDP UBCM21HQtrOycNYt/pmUDLuSSFaJo7lxkyXm0V75lzdIwUdxUuu5T9hWsWAHNtZBJ6dx UpVo7IuC+IEJN9Gfx+pLnaNNDjVqU1kQtjQE3a+H7UV5M2Wyv02G2UpncOJsCUaYwBkz PoV9Vk21xxusTaKAUzhO81RRSTdp5RqDx511/K34gw6F9aXE8twQHvwhLlTPCCGok+zO XGCA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:message-id:mime-version:subject:date :in-reply-to:cc:to:references; bh=fCSdsPrLH91KcgNFFqNS9d22ksh29iSW6iERDD015pg=; b=F7RWqNXWwbJCO6YDtJ9nsyZYVHBVXhIKsAw9b6YJPW0CK7hic5txDRpe0+Pydl0ev/ H2+f/XVHKnLH9GaIin681n1+0/7r/fRRaQOcfAUECxwaJc1twsTapGD/y5ar5Cwehl6/ XJ76WTNAUyZkWbOAleVxfN5mXtidfeFf+zLdMlYBftLmMmq9wkl6zCZ4mydfxfUIvFTc 5PKd7xUqQVhwdiKIpOru9pjNx4VQX2SrNhtX+n96ekJ59BEiXYMaNcs5uhnmeau3NFRR oE3kFVx1Le/Kv5hu1ssF+Sno4rHqWzp6z9LvzJXqaEAbKEYYL9gVvWbMd8iAUBM0NFZ7 EtWg== X-Gm-Message-State: AJcUukeMeN6nj625WvN3beA8hYIdIs0Zs+lXKTtOJE3IZOGfW5m9+J30 vcMQ1S81Pyk8fz0JTf4ekei5SL+T X-Google-Smtp-Source: ALg8bN6M4RG+OO7Xu2uNrsDlVQofurU3VuXM0uUzdFi9Y6mGXaJOhZzBS6sU2UhI4EmuicuxIPmbMg== X-Received: by 2002:a62:184e:: with SMTP id 75mr494076pfy.28.1547500469248; Mon, 14 Jan 2019 13:14:29 -0800 (PST) Received: from [192.168.20.7] (c-73-19-52-228.hsd1.wa.comcast.net. [73.19.52.228]) by smtp.gmail.com with ESMTPSA id h79sm1740193pfj.186.2019.01.14.13.14.27 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 14 Jan 2019 13:14:28 -0800 (PST) From: Enji Cooper Message-Id: <67E1A06E-292E-4771-A604-46C60D899277@gmail.com> Content-Type: multipart/signed; boundary="Apple-Mail=_C85D2B5D-7546-4C1A-959C-253E6362636D"; protocol="application/pgp-signature"; micalg=pgp-sha256 Mime-Version: 1.0 (Mac OS X Mail 11.5 \(3445.9.1\)) Subject: Re: svn commit: r343011 - in head/kerberos5: lib/libasn1 lib/libgssapi_krb5 lib/libhdb lib/libheimntlm lib/libhx509 lib/libkadm5clnt lib/libkadm5srv lib/libkdc lib/libkrb5 lib/libwind libexec/digest-se... Date: Mon, 14 Jan 2019 13:14:26 -0800 In-Reply-To: <201901142031.x0EKVQSb036914@pdx.rh.CN85.dnsmgr.net> Cc: Cy Schubert , Takahashi Yoshihiro , src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org To: rgrimes@freebsd.org References: <201901142031.x0EKVQSb036914@pdx.rh.CN85.dnsmgr.net> X-Mailer: Apple Mail (2.3445.9.1) X-Rspamd-Queue-Id: C74C176567 X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-6.95 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-1.000,0]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; REPLY(-4.00)[]; NEURAL_HAM_SHORT(-0.95)[-0.954,0] X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.29 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: Mon, 14 Jan 2019 21:14:31 -0000 --Apple-Mail=_C85D2B5D-7546-4C1A-959C-253E6362636D Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=utf-8 > On Jan 14, 2019, at 12:31 PM, Rodney W. Grimes = wrote: >=20 >> In message <201901141617.x0EGHwqQ035748@pdx.rh.CN85.dnsmgr.net>, >> "Rodney W. Gri >> mes" writes: >>>> Author: nyan >>>> Date: Mon Jan 14 06:34:54 2019 >>>> New Revision: 343011 >>>> URL: https://svnweb.freebsd.org/changeset/base/343011 >>>>=20 >>>> Log: >>>> Use ${SRCTOP}/contrib/com_err/com_err.h instead of the installed = com_err. >>> h. >>>> This fixes build when com_err.h is not installed. >>>=20 >>> Isn't this the general state of affairs for almost all >>> of the source tree? I would rather see this solved >>> in a general way, rather than sprinkly all this >>> -I${SRCTOP} all over in the tree. >>=20 >> Agreed however in practice this is fixed when discovered while = working >> on other issues/problems/projects. >=20 > This adds another special case that should not exist. >=20 >>>=20 >>> It use to be (and maybe still is) required that you run >>> cd $DRCTOP && make includes >>> if you expected new source code to compile. I expect >>> this has changed, but the just of it should work in >>> some aspect. >>=20 >> This fixes the potential bug of requiring a header file to be >> installed. I suspect this to be an oversight at the time. >>=20 >> Good that it's fixed now. >=20 > Something is wrong, as infact com_err.h *IS* > installed as /usr/include/com_err.h >=20 > diff /usr/include/com_err.h /usr/src/contrib/com_err/com_err.h Hi Rod, In the full case, yes. It=E2=80=99s relying on the staging = process for buildworld, which builds and installs everything in one = phase as part of =E2=80=9Cmake libraries=E2=80=9D. The list of kerberos libraries which rely on libcom_err is = potentially incomplete, so there=E2=80=99s a chance that it=E2=80=99s = not installing the headers in time. I=E2=80=99ll leave it as an exercise to the reader (in part, = because I=E2=80=99d rather not spending time chasing down this issue, as = there are other bigger fish to fry), to find the underlying issue in = Makefile.inc1 (in my not so humble opinion, kerberos/lib is broken down = a bit too much in Makefile.inc1, which obfuscates dependencies, makes it = extremely easy to miss them when adding/accounting for libraries, and = also plugs too much knowledge into a separate location in the build = system, duplicating efforts). Cheers, -Enji 2734 _prebuild_libs=3D ${_kerberos5_lib_libasn1} \ 2735 ${_kerberos5_lib_libhdb} \ 2736 ${_kerberos5_lib_libheimbase} \ 2737 ${_kerberos5_lib_libheimntlm} \ 2738 ${_libsqlite3} \ 2739 ${_kerberos5_lib_libheimipcc} \ 2740 ${_kerberos5_lib_libhx509} = ${_kerberos5_lib_libkrb5} \ 2741 ${_kerberos5_lib_libroken} \ 2742 ${_kerberos5_lib_libwind} \ 2743 lib/libbz2 ${_libcom_err} lib/libcrypt \ =E2=80=A6 2878 .if ${MK_KERBEROS} !=3D "no" 2879 kerberos5/lib/libasn1__L: lib/libcom_err__L = kerberos5/lib/libroken__L 2880 kerberos5/lib/libhdb__L: kerberos5/lib/libasn1__L lib/libcom_err__L = \ 2881 kerberos5/lib/libkrb5__L kerberos5/lib/libroken__L \ 2882 kerberos5/lib/libwind__L lib/libsqlite3__L 2883 kerberos5/lib/libheimntlm__L: secure/lib/libcrypto__L = kerberos5/lib/libkrb5__L \ 2884 kerberos5/lib/libroken__L lib/libcom_err__L 2885 kerberos5/lib/libhx509__L: kerberos5/lib/libasn1__L = lib/libcom_err__L \ 2886 secure/lib/libcrypto__L kerberos5/lib/libroken__L = kerberos5/lib/libwind__L 2887 kerberos5/lib/libkrb5__L: kerberos5/lib/libasn1__L = lib/libcom_err__L \ 2888 lib/libcrypt__L secure/lib/libcrypto__L = kerberos5/lib/libhx509__L \ 2889 kerberos5/lib/libroken__L kerberos5/lib/libwind__L \ 2890 kerberos5/lib/libheimbase__L kerberos5/lib/libheimipcc__L 2891 kerberos5/lib/libroken__L: lib/libcrypt__L 2892 kerberos5/lib/libwind__L: kerberos5/lib/libroken__L = lib/libcom_err__L 2893 kerberos5/lib/libheimbase__L: lib/libthr__L 2894 kerberos5/lib/libheimipcc__L: kerberos5/lib/libroken__L = kerberos5/lib/libheimbase__L lib/libthr__L 2895 .endif ... 2903 .if ${MK_KERBEROS} !=3D "no" 2904 _kerberos5_lib=3D kerberos5/lib 2905 _kerberos5_lib_libasn1=3D kerberos5/lib/libasn1 2906 _kerberos5_lib_libhdb=3D kerberos5/lib/libhdb 2907 _kerberos5_lib_libheimbase=3D kerberos5/lib/libheimbase 2908 _kerberos5_lib_libkrb5=3D kerberos5/lib/libkrb5 2909 _kerberos5_lib_libhx509=3D kerberos5/lib/libhx509 2910 _kerberos5_lib_libroken=3D kerberos5/lib/libroken 2911 _kerberos5_lib_libheimntlm=3D kerberos5/lib/libheimntlm 2912 _libsqlite3=3D lib/libsqlite3 2913 _kerberos5_lib_libheimipcc=3D kerberos5/lib/libheimipcc 2914 _kerberos5_lib_libwind=3D kerberos5/lib/libwind 2915 _libcom_err=3D lib/libcom_err 2916 .endif ... 2949 .for _lib in ${_startup_libs} ${_prebuild_libs} ${_generic_libs} 2950 ${_lib}__L: .PHONY .MAKE 2951 .if !defined(_MKSHOWCONFIG) && exists(${.CURDIR}/${_lib}) 2952 ${_+_}@${ECHODIR} "=3D=3D=3D> ${_lib} (obj,all,install)"; \ 2953 cd ${.CURDIR}/${_lib}; \ 2954 if [ -z "${NO_OBJWALK}" ]; then ${MAKE} MK_TESTS=3Dno= DIRPRFX=3D${_lib}/ obj; fi; \ 2955 ${MAKE} MK_TESTS=3Dno DIRPRFX=3D${_lib}/ all; \ 2956 ${MAKE} MK_TESTS=3Dno DIRPRFX=3D${_lib}/ install 2957 .endif 2958 .endfor --Apple-Mail=_C85D2B5D-7546-4C1A-959C-253E6362636D Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=signature.asc Content-Type: application/pgp-signature; name=signature.asc Content-Description: Message signed with OpenPGP -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEtvtxN6kOllEF3nmX5JFNMZeDGN4FAlw8+7IACgkQ5JFNMZeD GN7TwhAAkfVdOVYG1zUazOxs9lajlrH4nMRy4evgPHGToQTQ+7padZnljo6c3rRM 5iUHczEwWCf0JqzvE85uQUD/3KCT+ef09MD4F4avB5Y+K+39PlHJsPp0DINGZtsd O324/CgBJ3ipaYVQsDCks4lRY8itfsM2TctLCu9xcjf6DxFG9WmE7P73NpgAeDms PPzWeHy64uwS8pdMoMUypdjUuZ549OtHwBTbfYkTgbgTPNCfDlmOo3Ln7xr5Q5Gs 1ZzxJ7kvHKbhQHNCoDzeDwwgFLoZlJpFbwWCAjYFxwTgS5+UqUz8R+osbOEeWrI+ IkU0rBb8S1+B4FosOxnaH0PVNYpWHH94ltlhZaQrVZZyBaUA0KoiwCXuHrarYH3R csxb6eYk1mcmOD55UyNGhIhChgUym/YfrW5RmodbdMVNNWcr6R3gjayYBshVMvXw C8lr62dG9mKtO2KoBM0dE6uhC8/41OHlLvIY60mOQhzf15usDfdOy4slwy1oIf7I PKJDaPf30+ZpZ0LFlBuM4GrzRhAAmApDwKR6q4Byjq+D10x5R08bnDMrFNO4FqHE IyfX1ZQe34JiRmczr74TbDejQTcQzbNuiOd6nDRjmum/jX4AgoGfUvY85BPH7pIk 8pVQN/njDQhlTf8A4ZYNQLIcIlpz+AWiIMl4jKO+7hUGLS+gQAQ= =qmlq -----END PGP SIGNATURE----- --Apple-Mail=_C85D2B5D-7546-4C1A-959C-253E6362636D--