Re: [PATCH v3] docs: Use make invocation's -j argument for parallelism

2019-10-15 Thread Rasmus Villemoes
On 06/10/2019 21.33, Rasmus Villemoes wrote: > On 04/10/2019 18.08, Kees Cook wrote: >> The best solution would be to teach sphinx-build about the Make >> jobserver, though I expect that would be weird. Another idea would be to >> hold the reservation until sphinx-build finishes and THEN return th

Re: [PATCH v3] docs: Use make invocation's -j argument for parallelism

2019-10-06 Thread Rasmus Villemoes
On 04/10/2019 18.08, Kees Cook wrote: > On Fri, Oct 04, 2019 at 11:15:46AM +0200, Rasmus Villemoes wrote: >> On 25/09/2019 01.29, Kees Cook wrote: >>> +# Extract and prepare jobserver file descriptors from envirnoment. >> >> Ah, reading more carefully you set O_NONBLOCK explicitly. Well, older >> M

Re: [PATCH v3] docs: Use make invocation's -j argument for parallelism

2019-10-04 Thread Kees Cook
On Fri, Oct 04, 2019 at 11:15:46AM +0200, Rasmus Villemoes wrote: > On 25/09/2019 01.29, Kees Cook wrote: > > +# Extract and prepare jobserver file descriptors from envirnoment. > > +try: > > + # Fetch the make environment options. > > + flags = os.environ['MAKEFLAGS'] > > + > > + # Look for

Re: [PATCH v3] docs: Use make invocation's -j argument for parallelism

2019-10-04 Thread Christian Borntraeger
On 04.10.19 10:04, Christian Borntraeger wrote: > > On 25.09.19 01:29, Kees Cook wrote: >> While sphinx 1.7 and later supports "-jauto" for parallelism, this >> effectively ignores the "-j" flag used in the "make" invocation, which >> may cause confusion for build systems. Instead, extract the

Re: [PATCH v3] docs: Use make invocation's -j argument for parallelism

2019-10-04 Thread Rasmus Villemoes
On 25/09/2019 01.29, Kees Cook wrote: > > # User-friendly check for pdflatex and latexmk > HAVE_PDFLATEX := $(shell if which $(PDFLATEX) >/dev/null 2>&1; then echo 1; > else echo 0; fi) > @@ -68,6 +68,7 @@ quiet_cmd_sphinx = SPHINX $@ --> file://$(abspath > $(BUILDDIR)/$3/$4) > PYTHOND

Re: [PATCH v3] docs: Use make invocation's -j argument for parallelism

2019-10-04 Thread Christian Borntraeger
On 25.09.19 01:29, Kees Cook wrote: > While sphinx 1.7 and later supports "-jauto" for parallelism, this > effectively ignores the "-j" flag used in the "make" invocation, which > may cause confusion for build systems. Instead, extract the available > parallelism from "make"'s job server (since i

Re: [PATCH v3] docs: Use make invocation's -j argument for parallelism

2019-10-01 Thread Jonathan Corbet
On Tue, 24 Sep 2019 16:29:58 -0700 Kees Cook wrote: > While sphinx 1.7 and later supports "-jauto" for parallelism, this > effectively ignores the "-j" flag used in the "make" invocation, which > may cause confusion for build systems. Instead, extract the available > parallelism from "make"'s job