Originally reported by @pbokoc:
A listitem inside a varlistentry which begins immediately with an important breaks the whole varlistentry. The previous and next varlistentry-s render fine, but they still should be converted differently. Right now, the varlistentry's term is immediately followed by its listitem; there should be a \n+\n between them.
listitem
varlistentry
important
\n+\n
DocBook:
<varlistentry> <term><parameter>RUNKS=<replaceable>value</replaceable></parameter></term> <listitem> <important> <para> This parameter is deprecated. If you use it in a Kickstart file, it will be ignored. Only the <parameter>inst.ks=</parameter> parameter is necessary to start a Kickstart installation on IBM System z. </para> </important> <para> Where <replaceable>value</replaceable> is defined as <replaceable>1</replaceable> if you want to run the loader automatically on the Linux console without having to log in over the network with SSH. To use <literal>RUNKS=1</literal>, the console must either support full-screen or the <parameter>inst.cmdline</parameter> option (below) should be used. The latter applies for the 3270 terminal under z/VM or the operating system messages console for LPAR. We recommend <literal>RUNKS=1</literal> for fully automatic installations with Kickstart. When <literal>RUNKS=1</literal> is set, the installation program automatically continues in case of parameter errors and does not interrupt unattended installations by prompting for user interaction. </para> <para> Leave out the parameter or specify <literal>RUNKS=0</literal> otherwise. </para> </listitem> </varlistentry>
ADoc
`RUNKS=pass:attributes[{blank}]_value_pass:attributes[{blank}]`:: [IMPORTANT] ==== This parameter is deprecated. If you use it in a Kickstart file, it will be ignored. Only the `inst.ks=` parameter is necessary to start a Kickstart installation on IBM System{nbsp}z. ==== + Where _value_ is defined as _1_ if you want to run the loader automatically on the Linux console without having to log in over the network with SSH. To use `RUNKS=1`, the console must either support full-screen or the `inst.cmdline` option (below) should be used. The latter applies for the 3270 terminal under z/VM or the operating system messages console for LPAR. We recommend `RUNKS=1` for fully automatic installations with Kickstart. When `RUNKS=1` is set, the installation program automatically continues in case of parameter errors and does not interrupt unattended installations by prompting for user interaction. + Leave out the parameter or specify `RUNKS=0` otherwise.
What it SHOULD look like (note newlines and + joiners):
+
`RUNKS=pass:attributes[{blank}]_value_pass:attributes[{blank}]`:: + [IMPORTANT] ==== This parameter is deprecated. If you use it in a Kickstart file, it will be ignored. Only the `inst.ks=` parameter is necessary to start a Kickstart installation on IBM System{nbsp}z. ==== + Where _value_ is defined as _1_ if you want to run the loader automatically on the Linux console without having to log in over the network with SSH. To use `RUNKS=1`, the console must either support full-screen or the `inst.cmdline` option (below) should be used. The latter applies for the 3270 terminal under z/VM or the operating system messages console for LPAR. We recommend `RUNKS=1` for fully automatic installations with Kickstart. When `RUNKS=1` is set, the installation program automatically continues in case of parameter errors and does not interrupt unattended installations by prompting for user interaction. + Leave out the parameter or specify `RUNKS=0` otherwise.
Originally added by @pbokoc:
ALSO: another problem seems to be that varlistentries are sometimes generated with no empty line above them, which also seems to break them in interesting ways, mostly (again) causing + joiners to be rendered.
varlistentries
Example:
[[list-boot-options-sources]] .Specifying the Installation Source indexterm:[boot options,installation source] [option]`inst.repo=`:: + Specifies the installation source - that is, a location where the installation program can find the images and packages it requires. For example: + [subs="quotes, macros"] ---- [option]`inst.repo=cdrom` ----
In the above, moving the [option]`inst.repo=`:: down a line fixes the whole list. (If you want to test that, note that it breaks the table below due to issue #6 )
[option]`inst.repo=`::
(If you want to test that, note that it breaks the table below due to issue #6 )
Oh, and when you do the workaround for issue #6 , the list breaks again, so I think we'll have to open a ccutil issue for that. Except it might not be a ccutil issue, it might be an asciidoctor issue (ccutil uses asciidoctor to convert adoc back to docbook, right?), or it could even be expected behavior. In which case we're screwed if this kind of thing appears in a high number of places.
Login to comment on this ticket.