Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix documentation build #2

Open
wants to merge 1 commit into
base: epiphany-cgen-1.1
Choose a base branch
from

Conversation

carletes
Copy link

My version of makeinfo (5.2) fails to build the documentation when the @subsection tag removed in this patch is present:

$ make info
restore=: && backupdir=".am$$" && \
        rm -rf $backupdir && mkdir $backupdir && \
        if (makeinfo --split-size=5000000 --version) >/dev/null 2>&1; then \
          for f in cgen.info cgen.info-[0-9] cgen.info-[0-9][0-9] cgen.i[0-9] cgen.i[0-9][0-9]; do \
            if test -f $f; then mv $f $backupdir; restore=mv; else :; fi; \
          done; \
        else :; fi && \
        if makeinfo --split-size=5000000   -I ../.././cgen/doc \
         -o cgen.info `test -f 'cgen.texi' || echo '../.././cgen/doc/'`cgen.texi; \
        then \
          rc=0; \
        else \
          rc=$?; \
          $restore $backupdir/* `echo "./cgen.info" | sed 's|[^/]*$||'`; \
        fi; \
        rm -rf $backupdir; exit $rc
./running.texi:19: raising the section level of @subsection which is too low
make: *** [cgen.info] Error 1
$

My version of `makeinfo` (5.2) fails to build the documentation when the `@subsection` tag removed in this patch is present:

```
$ make info
restore=: && backupdir=".am$$" && \
        rm -rf $backupdir && mkdir $backupdir && \
        if (makeinfo --split-size=5000000 --version) >/dev/null 2>&1; then \
          for f in cgen.info cgen.info-[0-9] cgen.info-[0-9][0-9] cgen.i[0-9] cgen.i[0-9][0-9]; do \
            if test -f $f; then mv $f $backupdir; restore=mv; else :; fi; \
          done; \
        else :; fi && \
        if makeinfo --split-size=5000000   -I ../.././cgen/doc \
         -o cgen.info `test -f 'cgen.texi' || echo '../.././cgen/doc/'`cgen.texi; \
        then \
          rc=0; \
        else \
          rc=$?; \
          $restore $backupdir/* `echo "./cgen.info" | sed 's|[^/]*$||'`; \
        fi; \
        rm -rf $backupdir; exit $rc
./running.texi:19: raising the section level of @subsection which is too low
make: *** [cgen.info] Error 1
$
```
@jeremybennett
Copy link

carletes,

Thanks for finding this. However it's an upstream bug, so would you consider submitting it to the official CGEN and we can pull it down from there.

Jeremy

@carletes
Copy link
Author

Sure! Where should I submit this, Jeremy? The cgen@sourceware.org mailing list, perhaps?

@jeremybennett
Copy link

Hi carletes,

That's the one. If you let us know the outcome here, I'll make sure it gets pulled in.

(I get the CGEN mailing list, but don't always have time to read it).

carletes added a commit to carletes/epiphany-cgen-build that referenced this pull request Oct 28, 2014
Upstream documentation cannot be built with a modern `makeinfo`
(see adapteva/epiphany-cgen#2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants