Re: [PATCH v2] tools: add configure --with-dumpdir=DIR option

[Wei Liu <wei.liu2@citrix.com>]

On Wed, May 06, 2015 at 07:08:44AM +0000, Olaf Hering wrote:
> The current base directory /var/xen for domU dumps will be patched to
> /var/lib/xen by most distros. Provide a configure option to avoid
> patching the source.
> 
> If the option is not specified the default remains /var/xen/dump.
> 
> Please rerun autogen.sh after applying this patch.
> 
> Signed-off-by: Olaf Hering <olaf@aepfle.de>
> Cc: Ian Jackson <ian.jackson@eu.citrix.com>
> Cc: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
> Cc: Ian Campbell <ian.campbell@citrix.com>
> Cc: Wei Liu <wei.liu2@citrix.com>
> Acked-by: Wei Liu <wei.liu2@citrix.com>
> ---
> 
> v2:
>  - fix "conigure" typo in Subject line
> 
>  Config.mk                | 2 +-
>  INSTALL                  | 4 ++++
>  config/Paths.mk.in       | 1 +
>  m4/paths.m4              | 9 +++++++++
>  tools/Makefile           | 2 +-
>  tools/libxl/xl_cmdimpl.c | 2 +-
>  6 files changed, 17 insertions(+), 3 deletions(-)

You have not patched docs. I think it still references /var/xen/dump.

I discovered this when I read the manpage of xl.cfg.

Sorry for not noticing this earlier.

Wei.