Stuart Henderson wrote: > On 2021/09/14 14:06, Stefan Hagen wrote: > > Stuart Henderson wrote: > > > On 2021/09/14 07:43, Stefan Hagen wrote: > > > > X11/ion is currently broken. > > > > > > Hmm? It does build, though it does either need a BUILD_DEPENDS on groff > > > or your change for mandoc to produce consistent results (i.e. including > > > the manpage contents in welcome.txt etc). > > I've committed an alternative for the nroff fix. (Generally prefer to > use patch rather than sed unless there are a large number of files). > > > > $ ls -l */ion-* > > > -rw-r--r-- 1 root wheel 760147 Sep 13 07:16 aarch64/ion-20090110p4.tgz > > > -rw-r--r-- 1 root wheel 846233 Sep 13 07:03 amd64/ion-20090110p4.tgz > > > -rw-r--r-- 2 root wheel 756225 Jun 2 08:13 arm/ion-20090110p4.tgz > > > -rw-r--r-- 1 root wheel 760814 Sep 6 07:23 i386/ion-20090110p4.tgz > > > -rw-r--r-- 1 root wheel 869767 Aug 22 07:40 mips64/ion-20090110p4.tgz > > > -rw-r--r-- 1 root wheel 843277 Jun 25 07:55 mips64el/ion-20090110p4.tgz > > > -rw-r--r-- 1 root wheel 849572 Sep 9 09:59 powerpc/ion-20090110p4.tgz > > > -rw-r--r-- 1 root wheel 835022 Sep 10 07:12 > > > powerpc64/ion-20090110p4.tgz > > > -rw-r--r-- 1 root wheel 810101 Sep 11 09:30 sparc64/ion-20090110p4.tgz > > > > > > What problem are you seeing with the various exports.h files? Is there > > > something out of the ordinary with your lua installation? > > > > You're right. Once luajit is installed, it doesn't compile anymore. > > If I deinstall luajit, it works again. > > > > With the diff it works in both cases. > > Still strange as I don't see that here, and don't recall ever seeing a > problem in bulk builds with this. Have just tested with both in-tree > luajit and a local build of luajit 2.1. And I don't see any mechanism in > system.mk that ought to trigger the problem by having luajit installed > really.. > > I'm not seeing a good reason to remove it at this point, even with a > successor. > > > If we want to keep it, I'll look closer to find the root cause for the > > behavior change. > > > AFAIR it worked even with luajit a few months ago. > > > > This is the error I'm seeing: > > [...] > > Scanning mplex.c for exports. > > The command line running libextl-mkexports might give clues. Here's the > full build log from mine. Typical invocations are like this: > > `which lua51` ../libextl/libextl-mkexports -module mod_tiling -o exports.c -h > exports.h \ > main.c tiling.c placement.c split.c split-stdisp.c splitfloat.c panehandle.c > ops.c
I found the cause and also why it's only happening here. It has nothing to do with luajit. This was an accidental correlation. Setting MAKE_JOBS>1 breaks it. When not using MAKE_JOBS, it works every time. According to README.internals MAKE_JOBS is used by dpb, so I wonder why this didn't come up in bulk builds. Let's not touch it any further, besides the mandoc bit you committed already. Best regards, Stefan
