[libvirt] [PATCH 2/2] docs: hacking: Add 'Code coverage reports' section

Martin Kletzander mkletzan at redhat.com
Thu Mar 14 08:59:26 UTC 2019


On Wed, Mar 13, 2019 at 01:11:32PM -0400, Cole Robinson wrote:
>Signed-off-by: Cole Robinson <crobinso at redhat.com>
>---
> docs/hacking.html.in | 29 +++++++++++++++++++++++++++++
> 1 file changed, 29 insertions(+)
>
>diff --git a/docs/hacking.html.in b/docs/hacking.html.in
>index f99d143b7b..56608fbc9e 100644
>--- a/docs/hacking.html.in
>+++ b/docs/hacking.html.in
>@@ -1412,5 +1412,34 @@ int foo()
>         in the same way, but still make sure they get reviewed if non-trivial.
>       </li>
>     </ul>
>+    <h2><a id="coverage">Code coverage reports</a></h2>
>+
>+    <p>
>+      Code coverage HTML reports can be generated with:
>+    </p>
>+
>+<pre>
>+  make coverage
>+</pre>
>+
>+    <p>
>+      Reports will be generated in the <code>cov/</code> directory. Point a
>+      web browser at <code>cov/index.html</code> for the full report.
>+    </p>
>+
>+    <p>
>+      The <code>make coverage</code> target is provided by <code>gnulib</code>.
>+      It is a convenience helper for calling the following 3 targets in order.
>+      It may be useful to occasionally call these directly.
>+
>+    <ul>
>+      <li><code>make init-coverage</code>: run <code>make clean</code> and
>+          remove all code coverage counter files (*.gcno, etc)</li>

s/etc/etc./

>+      <li><code>make build-coverage</code>: run <code>make</code> and
>+          <code>make check</code> with <code>CFLAGS</code> filled in with
>+          necessary coverage flags.</li>

remove the full stop at the end so that it is the same for all three lines

With the above fixed:

Reviewed-by: Martin Kletzander <mkletzan at redhat.com>

>+      <li><code>make gen-coverage</code>: generate the HTML report</li>
>+    </ul>
>+    </p>
>   </body>
> </html>
>-- 
>2.20.1
>
>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20190314/3b00144d/attachment-0001.sig>


More information about the libvir-list mailing list