docs: compiling: mention build dir

[libvirt PATCH] docs: compiling: mention build dir

Ján Tomko posted 1 patch 3 years, 2 months ago

Download mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/ff0016e88a5b86fb7d8c75ed345c09c903d1317d.1611926978.git.jtomko@redhat.com

docs/compiling.html.in | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)

Expand all Fold all

[libvirt PATCH] docs: compiling: mention build dir

Posted by Ján Tomko 3 years, 2 months ago

Our docs have not been fully updated to reflect the separate
build directory.

Suggested-by: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Ján Tomko <jtomko@redhat.com>
---
 docs/compiling.html.in | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/docs/compiling.html.in b/docs/compiling.html.in
index c2c9d9f0ed..df6074780f 100644
--- a/docs/compiling.html.in
+++ b/docs/compiling.html.in
@@ -98,15 +98,17 @@ $ ninja -C build
     <pre>
 $ su -
 # service libvirtd stop  (or systemctl stop libvirtd.service)
-# /home/to/your/checkout/src/libvirtd
+# /home/to/your/checkout/build/src/libvirtd
     </pre>
 
     <p>
-      It is also possible to run virsh directly from the source tree
+      It is also possible to run virsh directly from the build tree
       using the ./run script (which sets some environment variables):
     </p>
 
     <pre>
+$ pwd
+/home/to/your/checkout/build
 $ ./run ./tools/virsh ....
     </pre>
   </body>
-- 
2.29.2

Re: [libvirt PATCH] docs: compiling: mention build dir

Posted by Michal Privoznik 3 years, 2 months ago

On 1/29/21 2:29 PM, Ján Tomko wrote:
> Our docs have not been fully updated to reflect the separate
> build directory.
> 
> Suggested-by: Laszlo Ersek <lersek@redhat.com>
> Signed-off-by: Ján Tomko <jtomko@redhat.com>
> ---
>   docs/compiling.html.in | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal