On Thu, Oct 12, 2023 at 03:27:05PM +0200, Theo Buehler wrote:
> This randomly doesn't package with
>
> makeinfo: unknown option --
> set-customization-variable=SECTION_NAME_IN_TITLE=true
Our current Makefile.in patch addresses this and works for the current
version regardless of MAKE_JOBS, but with bison 3.8.2 and MAKE_JOBS=1
(tb's dpb build job) the patch has no effect whereas >1 (my machine) works.
I don't fully understand the make job race inside this GNU auto*/make code,
but here's the same update with a simpler Makefile.in patch that reliably
packages bison regardless of MAKE_JOBS.
> checking for xsltproc... no
> checking for valgrind... no
> checking Valgrind suppression file...
The new diff also prevents picking up xsltproc and valgrind if present
(my machine) to avoid potential future failure and explicitly record
that bison does not depend on those ports.
> /usr/bin/make all-recursive
> Making all in po
> Making all in runtime-po
> Making all in gnulib-po
> Making all in .
> restore=: && backupdir=".am$$" && am__cwd=`pwd` &&
> CDPATH="${ZSH_VERSION+.}:" && cd . && rm -rf $backupdir && mkdir $backupdir
> && if (/bin/sh '/tmp/pobj/bison-3.8.2/bison-3.8.2/build-aux/missing'
> makeinfo --version) >/dev/null 2>&1; then for f in ./doc/bison.info
> ./doc/bison.info-[0-9] ./doc/bison.info-[0-9][0-9] ./doc/bison.i[0-9]
> ./doc/bison.i[0-9][0-9]; do if test -f $f; then mv $f $backupdir;
> restore=mv; else :; fi; done; else :; fi && cd "$am__cwd"; if /bin/sh
> '/tmp/pobj/bison-3.8.2/bison-3.8.2/build-aux/missing' makeinfo --no-split
>
> --set-customization-variable=SECTION_NAME_IN_TITLE=true
> --set-customization-variable=AVOID_MENU_REDUNDANCY=true
> --set-customization-variable=ICONS=true -I doc -I ./doc -o ./doc/bison.info
> ./doc/bison.texi; then rc=0; CDPATH="${ZSH_VERSION+.}:" && cd .; else
> rc=$?; CDPATH="${ZSH_VERSION+.}:" && cd . && $restore $backupdir/* `echo
> "././doc/bison.info" | sed 's|[^/]*$||'`; fi; rm -rf $backupdir; exit $rc
> makeinfo: unknown option --
> set-customization-variable=SECTION_NAME_IN_TITLE=true
> Try `makeinfo --help' for more information.
> *** Error 1 in . (Makefile:9601 './doc/bison.info')
> *** Error 1 in . (Makefile:10402 'all-recursive': @fail=; if
> (target_option=k; case ${target_option-} in ?) ;; *) echo
> "am__make_running_...)
> *** Error 2 in /tmp/pobj/bison-3.8.2/bison-3.8.2 (Makefile:3866 'all')
> *** Error 2 in devel/bison (/usr/ports/infrastructure/mk/bsd.port.mk:3034
> '/tmp/pobj/bison-3.8.2/.build_done': @cd /tmp/pobj/bison-3.8.2/bis...)
> *** Error 2 in devel/bison (/usr/ports/infrastructure/mk/bsd.port.mk:2677
> 'build': @lock=bison-3.8.2; export _LOCKS_HELD=" bison-3.8.2"; ...)
Index: Makefile
===================================================================
RCS file: /cvs/ports/devel/bison/Makefile,v
retrieving revision 1.66
diff -u -p -r1.66 Makefile
--- Makefile 21 Sep 2023 09:49:49 -0000 1.66
+++ Makefile 12 Oct 2023 16:32:23 -0000
@@ -1,6 +1,6 @@
COMMENT= GNU parser generator
-DISTNAME= bison-3.7.6
+DISTNAME= bison-3.8.2
CATEGORIES= devel
SITES= ${SITE_GNU:=bison/}
EXTRACT_SUFX= .tar.xz
@@ -19,6 +19,8 @@ BUILD_DEPENDS= devel/gettext,-tools \
RUN_DEPENDS= devel/m4
CONFIGURE_STYLE=gnu
+CONFIGURE_ENV= ac_cv_prog_VALGRIND=no \
+ ac_cv_path_XSLTPROC=no
CONFIGURE_ARGS= --disable-yacc
# Avoid crashes because of bad interactions between bison, libtextstyle
# from devel/gettext and ncurses
Index: distinfo
===================================================================
RCS file: /cvs/ports/devel/bison/distinfo,v
retrieving revision 1.14
diff -u -p -r1.14 distinfo
--- distinfo 17 May 2021 16:41:44 -0000 1.14
+++ distinfo 12 Oct 2023 16:32:23 -0000
@@ -1,2 +1,2 @@
-SHA256 (bison-3.7.6.tar.xz) = Z9aM4eIhkgUFJWQ/wKeiIpdXZoK+9qXFFEaQP1ru888=
-SIZE (bison-3.7.6.tar.xz) = 2627180
+SHA256 (bison-3.8.2.tar.xz) = m7oCFMz38QecXVkhAEUie89hlRmEDr+oDNOEnP9aW/I=
+SIZE (bison-3.8.2.tar.xz) = 2817324
Index: patches/patch-Makefile_in
===================================================================
RCS file: /cvs/ports/devel/bison/patches/patch-Makefile_in,v
retrieving revision 1.2
diff -u -p -r1.2 patch-Makefile_in
--- patches/patch-Makefile_in 11 Mar 2022 18:49:44 -0000 1.2
+++ patches/patch-Makefile_in 12 Oct 2023 18:19:20 -0000
@@ -1,14 +1,14 @@
-Break the "bison.help: bison" dependency to avoid rebuilding bison.info.
+Breaking the "bison.help: bison" dependency is non-trivial with MAKE_JOBS>1;
+instead make the problematic target a NOOP.
Index: Makefile.in
--- Makefile.in.orig
+++ Makefile.in
-@@ -10432,7 +10432,7 @@ $(srcdir)/$(doc_bison).info.bak: $(srcdir)/$(doc_bison
+@@ -9579,6 +9579,7 @@ doc/$(am__dirstamp):
+ @: > doc/$(am__dirstamp)
- doc/refcard.pdf: doc/refcard.tex
- $(AM_V_GEN) cd doc && pdftex $(abs_top_srcdir)/doc/refcard.tex
--@CROSS_COMPILING_FALSE@$(top_srcdir)/doc/bison.help: src/bison$(EXEEXT)
-+@CROSS_COMPILING_FALSE@$(top_srcdir)/doc/bison.help: #src/bison$(EXEEXT)
- @CROSS_COMPILING_FALSE@ $(AM_V_GEN)$(MKDIR_P) doc
- @CROSS_COMPILING_FALSE@ $(AM_V_at) LC_ALL=C tests/bison --version
>doc/bison.help.tmp
- @CROSS_COMPILING_FALSE@ $(AM_V_at) LC_ALL=C tests/bison --help | \
+ $(srcdir)/doc/bison.info: doc/bison.texi $(srcdir)/doc/version.texi
$(doc_bison_TEXINFOS)
++neutered-bison-info:
+ $(AM_V_MAKEINFO)restore=: && backupdir="$(am__leading_dot)am$$$$" && \
+ am__cwd=`pwd` && $(am__cd) $(srcdir) && \
+ rm -rf $$backupdir && mkdir $$backupdir && \
Index: pkg/PLIST
===================================================================
RCS file: /cvs/ports/devel/bison/pkg/PLIST,v
retrieving revision 1.15
diff -u -p -r1.15 PLIST
--- pkg/PLIST 11 Mar 2022 18:49:45 -0000 1.15
+++ pkg/PLIST 12 Oct 2023 16:32:23 -0000
@@ -11,7 +11,6 @@ share/bison/m4sugar/
share/bison/m4sugar/foreach.m4
share/bison/m4sugar/m4sugar.m4
share/bison/skeletons/
-share/bison/skeletons/README-D.txt
share/bison/skeletons/bison.m4
share/bison/skeletons/c++-skel.m4
share/bison/skeletons/c++.m4
@@ -22,6 +21,7 @@ share/bison/skeletons/d-skel.m4
share/bison/skeletons/d.m4
share/bison/skeletons/glr.c
share/bison/skeletons/glr.cc
+share/bison/skeletons/glr2.cc
share/bison/skeletons/java-skel.m4
share/bison/skeletons/java.m4
share/bison/skeletons/lalr1.cc
@@ -70,6 +70,10 @@ share/doc/bison/examples/c/calc/
share/doc/bison/examples/c/calc/Makefile
share/doc/bison/examples/c/calc/README.md
share/doc/bison/examples/c/calc/calc.y
+share/doc/bison/examples/c/glr/
+share/doc/bison/examples/c/glr/Makefile
+share/doc/bison/examples/c/glr/README.md
+share/doc/bison/examples/c/glr/c++-types.y
share/doc/bison/examples/c/lexcalc/
share/doc/bison/examples/c/lexcalc/Makefile
share/doc/bison/examples/c/lexcalc/README.md
@@ -92,9 +96,13 @@ share/doc/bison/examples/c/rpcalc/
share/doc/bison/examples/c/rpcalc/Makefile
share/doc/bison/examples/c/rpcalc/rpcalc.y
share/doc/bison/examples/d/
-share/doc/bison/examples/d/Makefile
share/doc/bison/examples/d/README.md
-share/doc/bison/examples/d/calc.y
+share/doc/bison/examples/d/calc/
+share/doc/bison/examples/d/calc/Makefile
+share/doc/bison/examples/d/calc/calc.y
+share/doc/bison/examples/d/simple/
+share/doc/bison/examples/d/simple/Makefile
+share/doc/bison/examples/d/simple/calc.y
share/doc/bison/examples/java/
share/doc/bison/examples/java/README.md
share/doc/bison/examples/java/calc/