Date: Tue, 22 Jan 2013 12:13:48 -0600 From: Brooks Davis <brooks@FreeBSD.org> To: Chris Rees <crees@FreeBSD.org> Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org Subject: Re: svn commit: r245752 - in head: etc share/mk Message-ID: <20130122181348.GA1662@lor.one-eyed-alien.net> In-Reply-To: <CADLo83-9-oSZkUn6u=rWuj8oh3fDEVL5oaNp-RsBJdUbFQg0cQ@mail.gmail.com> References: <201301212240.r0LMedcF088022@svn.freebsd.org> <CADLo83-9-oSZkUn6u=rWuj8oh3fDEVL5oaNp-RsBJdUbFQg0cQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--J2SCkAp4GZ/dPZZf Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jan 22, 2013 at 05:55:31PM +0000, Chris Rees wrote: > On 21 January 2013 22:40, Brooks Davis <brooks@freebsd.org> wrote: > > Author: brooks > > Date: Mon Jan 21 22:40:39 2013 > > New Revision: 245752 > > URL: http://svnweb.freebsd.org/changeset/base/245752 > > > > Log: > > Replace all known uses of ln in the build process with appropriate > > install -l invocations via new INSTALL_LINK and INSTALL_SYMLINK > > variables. > > > > Sponsored by: DARPA, AFRL > > Reviewed by: ian, ray, rpaulo > > > > Modified: > > head/etc/Makefile > > head/share/mk/bsd.incs.mk > > head/share/mk/bsd.info.mk > > head/share/mk/bsd.lib.mk > > head/share/mk/bsd.links.mk > > head/share/mk/bsd.man.mk > > head/share/mk/bsd.own.mk > > > > Modified: head/etc/Makefile > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D > > --- head/etc/Makefile Mon Jan 21 22:32:00 2013 (r245751) > > +++ head/etc/Makefile Mon Jan 21 22:40:39 2013 (r245752) > > @@ -324,26 +324,29 @@ distrib-dirs: > > -f $$m -p $$d; \ > > ${MTREE_CMD} -deU ${MTREE_FOLLOWS_SYMLINKS} -f $$m -p $= $d; \ > > done; true > > - ln -sfh usr/src/sys ${DESTDIR}/sys > > + ${INSTALL_SYMLINK} usr/src/sys ${DESTDIR}/sys >=20 > This broke installworld for me, because of the different behaviour. >=20 > ln -sf overwrites existing symlinks, whereas install -l a b puts a > symlink to a inside itself... which fails on a read-only src (NFS > mounted). >=20 > Perhaps it's install misbehaving? I can't work out if it's > bootstrapped properly; we should be using new install(1), are we? Sorry about that. It's fixed. -- Brooks --J2SCkAp4GZ/dPZZf Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (FreeBSD) iD8DBQFQ/tbcXY6L6fI4GtQRAiFLAKCxbdqoSTJF8dvxyoc5NrCMlnqtGQCeMxCO nGEQ58sX2BfUUQ0ST9rgBtk= =vDFI -----END PGP SIGNATURE----- --J2SCkAp4GZ/dPZZf--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20130122181348.GA1662>