Re: include-what-you-use: ship docs, use python 3, other nits

2019-06-17 Thread Klemens Nanni
On Wed, Jun 12, 2019 at 06:28:13PM +0100, Stuart Henderson wrote: > I think it's correct to remove those extras. > > libexecinfo is an extra specially annoying case because many ports are only > prepared to deal with Linux (execinfo.h header but the backtrace functions are > in libc). I'm not

Re: include-what-you-use: ship docs, use python 3, other nits

2019-06-12 Thread Stuart Henderson
On 2019/06/11 16:16, Jonathan Gray wrote: > On Mon, Jun 10, 2019 at 10:34:21PM +0200, Klemens Nanni wrote: > > On Mon, Jun 10, 2019 at 08:10:58PM +0100, Stuart Henderson wrote: > > > Removing execinfo from WANTLIB nullifies the devel/libexecinfo > > > LIB_DEPENDS .. > > That... is a valid point.

Re: include-what-you-use: ship docs, use python 3, other nits

2019-06-11 Thread Daniel Jakots
On Tue, 11 Jun 2019 16:16:04 +1000, Jonathan Gray wrote: > Is there a push to move everything to python3 now? I don't think we will be able to suddenly flip a switch and look at what breaks as breakage will probably be at runtime. I think it's easier to do what kn is doing, switching port by

Re: include-what-you-use: ship docs, use python 3, other nits

2019-06-11 Thread Klemens Nanni
On Tue, Jun 11, 2019 at 04:16:04PM +1000, Jonathan Gray wrote: > I don't see why you are removing libcurses and libz from wantlib Because even after Stuart's reply and my revised diff, I rushed things without looking around... `make port-lib-depends-check` lists those libs as extra - I should

Re: include-what-you-use: ship docs, use python 3, other nits

2019-06-11 Thread Jonathan Gray
On Mon, Jun 10, 2019 at 10:34:21PM +0200, Klemens Nanni wrote: > On Mon, Jun 10, 2019 at 08:10:58PM +0100, Stuart Henderson wrote: > > Removing execinfo from WANTLIB nullifies the devel/libexecinfo LIB_DEPENDS > > .. > That... is a valid point. I'll leave it in and double check tooling's >

Re: include-what-you-use: ship docs, use python 3, other nits

2019-06-10 Thread Klemens Nanni
On Tue, Jun 11, 2019 at 12:11:37AM +0200, Ingo Schwarze wrote: > Why does this have an unconditional dependency on ports LLVM? > For example, on amd64, it actually uses base LLVM unless i read > the build logs incorrectly. > > I realize this is unrelated to your changes, just wondering... I

Re: include-what-you-use: ship docs, use python 3, other nits

2019-06-10 Thread Ingo Schwarze
Hi Klemens, the authors of this tool absolutely do not understand how documentation works. That said, i agree it is better to install whatever piss-poor and disorganized musings they do provide rather than installing no documentation whatsoever. See below in-line for two questions, none of

Re: include-what-you-use: ship docs, use python 3, other nits

2019-06-10 Thread Klemens Nanni
On Mon, Jun 10, 2019 at 08:10:58PM +0100, Stuart Henderson wrote: > Removing execinfo from WANTLIB nullifies the devel/libexecinfo LIB_DEPENDS .. That... is a valid point. I'll leave it in and double check tooling's output before cooking diffs, thanks. Index: Makefile

Re: include-what-you-use: ship docs, use python 3, other nits

2019-06-10 Thread Stuart Henderson
On 2019/06/10 18:47, Klemens Nanni wrote: > > -WANTLIB= c curses execinfo form m pthread z > +WANTLIB= ${COMPILER_LIBCXX} c m pthread > LLVM_V= 7.0.1 > BUILD_DEPENDS= devel/llvm>=${LLVM_V} > RUN_DEPENDS= devel/llvm>=${LLVM_V} > LIB_DEPENDS= devel/libexecinfo

include-what-you-use: ship docs, use python 3, other nits

2019-06-10 Thread Klemens Nanni
Came here to ship docs so I don't have to browse the web to lern how to use it, but improved a bit more while here. - derive PKGNAME from DISTNAME so future bumps require only one change - sync WANTLIB according to `port-lib-depends-check` - PERMIT_PACKAGE - use TLS - use Python3 - Python is RDEP