Hi, On Sun, Sep 17, 2017 at 02:13:34PM +0200, Thomas Schwinge wrote: > Hi! > > On Tue, 24 Jan 2017 15:30:34 -0500, David Malcolm <dmalc...@redhat.com> wrote: > > On Tue, 2017-01-24 at 13:52 +0100, Martin Jambor wrote: > > > [...] I have just > > > committed the BRIG FE as revision 244867. > > In a build with that enabled, I just happened to "make html" in "gcc/", > and ran into: > > [...] > makeinfo --split-size=5000000 --html -I [...]/source-gcc/gcc/doc -I > [...]/source-gcc/gcc/doc/include \ > -I [...]/source-gcc/gcc/brig -o > [...]/build-gcc/gcc/HTML/gcc-8.0.0/brig > makeinfo: missing file argument. > Try `makeinfo --help' for more information. > [...]/source-gcc/gcc/brig/Make-lang.in:117: recipe for target > '[...]/build-gcc/gcc/HTML/gcc-8.0.0/brig/index.html' failed > make: *** [[...]/build-gcc/gcc/HTML/gcc-8.0.0/brig/index.html] Error 255 > > > A deps issue for the docs I noticed when glancing through the commit: > > > > diff --git a/gcc/brig/Make-lang.in b/gcc/brig/Make-lang.in > > new file mode 100644 (file) > > index 0000000..b85b1b0 > > --- /dev/null > > +++ b/gcc/brig/Make-lang.in > > > > [...snip...] > > > > +# Documentation. > > + > > +GO_TEXI_FILES = \ > > + brig/gccbrig.texi \ > > + $(gcc_docdir)/include/fdl.texi \ > > + $(gcc_docdir)/include/gpl_v3.texi \ > > + $(gcc_docdir)/include/gcc-common.texi \ > > + gcc-vers.texi > > > > Presumably this should be BRIG_TEXI_FILES, rather than GO_TEXI_FILES? > > > > +# doc/gccbrig.info: $(BRIG_TEXI_FILES) > > +# if test "x$(BUILD_INFO)" = xinfo; then \ > > +# rm -f doc/gccbrig.info*; \ > > +# $(MAKEINFO) $(MAKEINFOFLAGS) -I $(gcc_docdir) \ > > +# -I $(gcc_docdir)/include -o $@ $<; \ > > +# else true; fi > > + > > +# doc/gccbrig.dvi: $(BRIG_TEXI_FILES) > > +# $(TEXI2DVI) -I $(abs_docdir) -I $(abs_docdir)/include -o $@ $< > > + > > +# doc/gccbrig.pdf: $(BRIG_TEXI_FILES) > > +# $(TEXI2PDF) -I $(abs_docdir) -I $(abs_docdir)/include -o $@ $< > > + > > +$(build_htmldir)/brig/index.html: $(BRIG_TEXI_FILES) > > + $(mkinstalldirs) $(@D) > > + rm -f $(@D)/* > > + $(TEXI2HTML) -I $(gcc_docdir) -I $(gcc_docdir)/include \ > > + -I $(srcdir)/brig -o $(@D) $< > > > > ...for use in describing the deps of the above. > > ..., so that still needs to be fixed. Alas, that won't help: the > "gccbrig.texi" file doesn't actually exist. ;-) >
I see, I always only check "make info" when verifying documentation changes and so missed this. Thanks for providing the interim fix, me and/or Pekka will add some basic content by the time next gcc 7 is released (IIRC, it is supposed to come out at the end of this or the beginning of next year). Martin
