Date: Tue, 7 Mar 2017 01:12:31 -0800 From: "Ngie Cooper (yaneurabeya)" <yaneurabeya@gmail.com> To: rgrimes@freebsd.org Cc: src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r314833 - in head: bin/pkill lib/libxo/tests/encoder libexec/rtld-elf share/termcap usr.bin/chpass usr.bin/passwd usr.sbin/bsdinstall/partedit usr.sbin/mailwrapper usr.sbin/nologin Message-ID: <C564389B-E51F-439B-843C-8EFE9EAD9EE4@gmail.com> In-Reply-To: <201703070904.v2794RLY093614@pdx.rh.CN85.dnsmgr.net> References: <201703070904.v2794RLY093614@pdx.rh.CN85.dnsmgr.net>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] > On Mar 7, 2017, at 01:04, Rodney W. Grimes <freebsd@pdx.rh.CN85.dnsmgr.net> wrote: > >>> >>> On Mar 6, 2017, at 21:50, Rodney W. Grimes <freebsd@pdx.rh.CN85.dnsmgr.net> wrote: >>> >>>>> On Mar 6, 2017, at 21:10, Rodney W. Grimes <rgrimes@FreeBSD.org> wrote: >>>>> >>>>> Author: rgrimes >>>>> Date: Tue Mar 7 05:10:38 2017 >>>>> New Revision: 314833 >>>>> URL: https://svnweb.freebsd.org/changeset/base/314833 >>> ... >>> >>>>> >>>>> -SYMLINKS+= ${LIBDIR}/${SHLIB_NAME} /usr/lib/libxo/encoder/test.enc >>>>> +SYMLINKS+= ../../../tests/lib/libxo/${SHLIB_NAME} /usr/lib/libxo/encoder/test.enc >>>> >>>> This change breaks setting TESTSBASE != /usr/tests . Please fix (my employer depends on this being functional). >>>> Thanks, >>> >>> I am just going to revert this until the issue with SYMLINKS vs RSYMLINKS and >>> probably a much better fix in bsd.*.mk can be implemented. I can live with >>> one last nasty wart of an absolute link in DESTDIR until then. The important >>> functional ones are gone now. >>> >>> To fix this without changing SYMLINKS would involve adding a local install: >>> target which would be more mess than having the one absolute link. >> >> The proposed enhancement for bsd.links.mk for RSYMLINKS can be found here: https://reviews.freebsd.org/D9919 . > > I dont think that is what bapt/bdrewery had in mind, I think they wanted to > replicate the SYMLINK= as RSYMLINK= and use that when needed. > > I have thought about this and we do not ever want absolute links in the tree, > so much simpler fix is to just convert SYMLINKS to call INSTALL_RSYMLINKS > instead of INSTALL_SYMLINKS and bam, its all fixed and shall always be fixed, > and as long as no one uses INSTALL_SYMLINKS. SYMLINKS has been around since r245752. I personally don’t know if it’s wise to remove functionality that’s been “in production” for 4+ years. Making the target intelligently use one command over the other might seem ok, but it could have unintended consequences. I think it’s best to have another well-documented variable that uses INSTALL_RSYMLINK vs INSTALL_SYMLINK. Also, in the CR, please note that the SYMLINKS piece doesn’t prefix source targets with ${DESTDIR}, whereas the LINKS (and soon to be RSYMLINKS portion) will need it in order to compute the right paths and execute the right behavior. Otherwise, you’re going to be breaking someone’s use for SYMLINKS for no good reason. Thanks, -Ngie [-- Attachment #2 --] -----BEGIN PGP SIGNATURE----- Comment: GPGTools - https://gpgtools.org iQIcBAEBCgAGBQJYvnl/AAoJEPWDqSZpMIYV9FwP/RMj3HAeTCsRRR2RtR4qQf7Y Kx/SLxc9DMPwCBMja2H8U00SEItld294N+Ze3V0Kxp1amFEhMuhgiDRdwRhLGTMp 8VkN8EIBqBAAfr3i6/0bmyHYxV4HhDRTRsSg4dX3UfxiTXo+cJDSBiCiZp+kr8B6 9kK9wNeKucR8Lu3TsH8vwwLYEq/+wVwYeLPbRjIxbMrYpbWUw/nm+PB9S2gfqDxH uwr9+a4zCWL4GWuc6AuKnw0N8Ixc2YqdreiozETVqcvv6Xv7LElyYVNqD8zsjbO/ cIRpnHjlQykZ75xEl7m4GvTJvH4C81SRmxQLQPjwyNPrJ63T6z1HeOU7uiBtud+d opADMG0phZtT95p+PKZe7HHRQMDit0BAA0C4QZzJW+7zmCHgAMddF6ld7bfujaO5 8vybSn/1KlBpdlNDL9IljDTkwrvy++jp/4FWGgkrBlwn5Z7nLZq3EZMKU/UyGmXe N5QM7uVsFHIeniYDcwtSswAsVdyWQNyXlsuGE0TKN4xCiiR1xIZi081qIACyuwhU mfV5gLLaNjN/J8yVGOWNdXGTqkg5UWWUoRd2o+MLgHxGfQnojY10bJ+Jdf05CI5Z XZcxs6RU54tWiBC4338ImoxrntOFtW5c1C98pocS0PV4ddxMBjhJ9cKMiwp/01sR q5MO6RCRe8/K06RoWZL7 =fZs5 -----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?C564389B-E51F-439B-843C-8EFE9EAD9EE4>