git: 0006530aa14b - main - include: Remove symlink installation

Emmanuel Vadot manu at bidouilliste.com
Tue Mar 16 07:39:59 UTC 2021


On Tue, 16 Mar 2021 09:17:36 +0200
Konstantin Belousov <kostikbel at gmail.com> wrote:

> On Tue, Mar 16, 2021 at 08:12:14AM +0100, Emmanuel Vadot wrote:
> > On Tue, 16 Mar 2021 08:52:07 +0200
> > Konstantin Belousov <kostikbel at gmail.com> wrote:
> > 
> > > On Tue, Mar 16, 2021 at 06:13:21AM +0000, Emmanuel Vadot wrote:
> > > > The branch main has been updated by manu:
> > > > 
> > > > URL: https://cgit.FreeBSD.org/src/commit/?id=0006530aa14b9df56f88df7d819fae89b115d865
> > > > 
> > > > commit 0006530aa14b9df56f88df7d819fae89b115d865
> > > > Author:     Emmanuel Vadot <manu at FreeBSD.org>
> > > > AuthorDate: 2021-03-16 06:12:53 +0000
> > > > Commit:     Emmanuel Vadot <manu at FreeBSD.org>
> > > > CommitDate: 2021-03-16 06:12:53 +0000
> > > > 
> > > >     include: Remove symlink installation
> > > >     
> > > >     headers could be installed as symlink to the source tree instead of copies.
> > > >     Remove the possibility to do that.
> > > >     This make the makefile easier to read and to maintain and also don't duplicate
> > > >     code.
> > > >     
> > > >     While here remove some directories from LSBUDIRS as we already install them using
> > > >     the INCS stuff.
> > > 
> > > This might make Makefiles easier to read, but also it makes practically
> > > very hard to develop rtld/libc/libthr. After any system header
> > > modification, full buildworld or at least kernel-toolchain run is
> > > required to get buildenv populated with the new headers.
> > 
> >  Can you share what you did before so I can cook a patch that restore
> > this behavior in a better way than it was done before ?
> 
> I did
> $ make buildworld <- this was done relatively irregularly
> $ make buildenv
> [Edit whatever I needed, typically in sys/sys and libexec/rtld-elf]
> <nested env sh>$ cd libexec/rtld-elf
> <nested env sh>$ make DEBUG_FLAGS=-g WITHOUT_TESTS=yes all
> [Check built ld-elf.so.1, more edits]
> <nested env sh>$ make ...
> 
> Now, if I modify anything in sys/, I have to repopuate staged headers in
> the obj directory of buildworld.

 Right ok, I've missed that SHARED=symlinks was used in Makefile.inc1,
I'll revert and redo a better patch. 

-- 
Emmanuel Vadot <manu at bidouilliste.com> <manu at FreeBSD.org>


More information about the dev-commits-src-all mailing list