On Thu, 22 Jan 2026 02:49:40 GMT, SendaoYan <[email protected]> wrote:

>> Hi all,
>> 
>> The sctp tests need libsctp.so.1 to be installed, otherwise those tests will 
>> report skipped. Seems that there is no help information to descript how to 
>> run those tests. I think it's meaningful to add help information how to run 
>> those tests.
>
> SendaoYan has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Regenerate testing.html by "make update-build-docs"

doc/testing.html line 634:

> 632: <pre><code>sudo dnf install -y lksctp-tools
> 633: sudo modprobe sctp
> 634: lsmod | grep sctp</code></pre>

This is not what my diff ends up being when running `make update-build-docs`. I 
get line breaks at <80 chars (don't know exactly, but that's what it looks 
like). We use pandoc 2.19.2, which version are you using?

Here is the diff when I run `make update-build-docs` in this branch:


diff --git a/doc/testing.html b/doc/testing.html
index 6267df13109..77bd2cfbee0 100644
--- a/doc/testing.html
+++ b/doc/testing.html
@@ -623,12 +623,17 @@ <h3 id="pkcs11-tests">PKCS11 Tests</h3>
 <p>For more notes about the PKCS11 tests, please refer to
 test/jdk/sun/security/pkcs11/README.</p>
 <h3 id="sctp-tests">SCTP Tests</h3>
-<p>The SCTP tests require the SCTP runtime library, which is often not 
installed by default in popular Linux distributions. Without this library,the 
SCTP tests will be skipped. If you want to enable the SCTP tests, you should 
install the SCTP library before running the tests.</p>
-<p>For distributions using the .deb packaging format and the apt tool (such as 
Debian, Ubuntu, etc.), try this:</p>
+<p>The SCTP tests require the SCTP runtime library, which is often not
+installed by default in popular Linux distributions. Without this
+library,the SCTP tests will be skipped. If you want to enable the SCTP
+tests, you should install the SCTP library before running the tests.</p>
+<p>For distributions using the .deb packaging format and the apt tool
+(such as Debian, Ubuntu, etc.), try this:</p>
 <pre><code>sudo apt install libsctp1
 sudo modprobe sctp
 lsmod | grep sctp</code></pre>
-<p>For distributions using the .rpm packaging format and the dnf tool (such as 
Fedora, Red Hat, etc.), try this:</p>
+<p>For distributions using the .rpm packaging format and the dnf tool
+(such as Fedora, Red Hat, etc.), try this:</p>
 <pre><code>sudo dnf install -y lksctp-tools
 sudo modprobe sctp
 lsmod | grep sctp</code></pre>

doc/testing.md line 647:

> 645: 
> 646: The SCTP tests require the SCTP runtime library, which is often not 
> installed
> 647: by default in popular Linux distributions. Without this library,the SCTP 
> tests

Suggestion:

by default in popular Linux distributions. Without this library, the SCTP tests

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/29320#discussion_r2717910363
PR Review Comment: https://git.openjdk.org/jdk/pull/29320#discussion_r2717122913

Reply via email to