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

Remove section numbers #4

Open
wants to merge 2 commits into
base: master-with-README
Choose a base branch
from

Conversation

lefessan
Copy link
Member

On top of #2

@GitMensch
Copy link
Collaborator

Nice, this also fixes all the wrong numbers like 8.1.11 and the formatting looks a bit cleaner (more space after the number) :-)
I guess the old-menu can be dropped, right?

Can you check to auto-add the numbers of the chapters in the PDF structure?

The old version had
grafik
while the new one has
grafik

The old one (apart of the wrong trailing periods which are now fixed overall) was similar to the standard:
grafik

Hm, the GC manual has a "mix", ideally we can get all to use the same:
grafik

* Rename @anchoridx{Data} to @anchoridx{DateTypes} in section 13 (duplicate
  with the same index in section 7)
* Remove @validatemenus not available after version 6.8 of texinfo
* Rename section "cobc - The GnuCOBOL Compiler - Runtime options" to
  "cobc - The GnuCOBOL Compiler"
* Add @Iftex around menu of section 7.8, as it makes makeinfo segfault and
  is not useful for HTML
* Small additional fixes for info generation to work
As a result, most numbers are not changed, but appendices have
different numbers, especially C1..C4
@GitMensch
Copy link
Collaborator

This new one LGTM now. @vbcoen I think you're fine with that and with the result? This should fix https://sourceforge.net/p/gnucobol/feature-requests/421/ and hopefully also explain more, what my babbling meant.

Question: that's quite a technical change - should there still be an adjustment to update.texi?

Note @lefessan: as this is a technical change there should be a separate commit in svn for it (referencing the implemented FR, possibly in the commit log).

The other minor updates from the other PRs could go in as another one together.

@lefessan
Copy link
Member Author

I tried to understand how to add numbers in the structure, but I couldn't find anything. Since gnucobol.pdf has them, I compared the files, but the only difference I could find is that the GnuCOBOL manual uses a more recent version of texinfo.tex. I tried to use that the same one, but it runs into errors that I don't understand...

@GitMensch
Copy link
Collaborator

Updating texinfo and fixing the errors is then a thing we can do later (and should track somewhere, as well as getting the numbers back). I think the benefits of "numbered correctly" and "no need to manually renumber everything when something is moved/inserted/removed" are enough to live with the numbers that should be added back later.

lefessan added a commit that referenced this pull request Jan 15, 2024
Remove section numbers and let texinfo auto-generate numbers. This
should ease refactorings of the manuals, such as adding new sections
and moving sections.

See PR on Github: #4



git-svn-id: svn+ssh://svn.code.sf.net/p/gnucobol/code/external-doc@5179 ed166372-6744-4ac0-a67f-bb1ae9efa102
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants