[PATCH] configure: fix handling of --docdir parameter

Bruce Rogers posted 1 patch 4 days ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201015190233.270296-1-brogers@suse.com
configure | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH] configure: fix handling of --docdir parameter

Posted by Bruce Rogers 4 days ago
Commit ca8c0909f01 changed qemu_docdir to be docdir, then later uses the
qemu_docdir name in the final assignment. Unfortunately, one instance of
qemu_docdir was missed: the one which comes from the --docdir parameter.
This patch restores the proper handling of the --docdir parameter.

Fixes: ca8c0909f01 ("configure: build docdir like other suffixed
directories")

Signed-off-by: Bruce Rogers <brogers@suse.com>
---
 configure | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/configure b/configure
index f839c2a557..07acd01a25 100755
--- a/configure
+++ b/configure
@@ -971,7 +971,7 @@ for opt do
   ;;
   --with-suffix=*) qemu_suffix="$optarg"
   ;;
-  --docdir=*) qemu_docdir="$optarg"
+  --docdir=*) docdir="$optarg"
   ;;
   --sysconfdir=*) sysconfdir="$optarg"
   ;;
-- 
2.28.0


Re: [PATCH] configure: fix handling of --docdir parameter

Posted by Bruce Rogers 4 days ago
On Thu, 2020-10-15 at 13:02 -0600, Bruce Rogers wrote:
> Commit ca8c0909f01 changed qemu_docdir to be docdir, then later uses
> the
> qemu_docdir name in the final assignment. Unfortunately, one instance
> of
> qemu_docdir was missed: the one which comes from the --docdir
> parameter.
> This patch restores the proper handling of the --docdir parameter.
> 
> Fixes: ca8c0909f01 ("configure: build docdir like other suffixed
> directories")
> 
> Signed-off-by: Bruce Rogers <brogers@suse.com>
> ---
>  configure | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure b/configure
> index f839c2a557..07acd01a25 100755
> --- a/configure
> +++ b/configure
> @@ -971,7 +971,7 @@ for opt do
>    ;;
>    --with-suffix=*) qemu_suffix="$optarg"
>    ;;
> -  --docdir=*) qemu_docdir="$optarg"
> +  --docdir=*) docdir="$optarg"
>    ;;
>    --sysconfdir=*) sysconfdir="$optarg"
>    ;;

Oops! I got the wrong CC's put on this. Let me resend with the right
interested parties. Sorry for the noise.

- Bruce