[Libguestfs] [PATCH] sysprep: mention globbing in help for --delete

[Pino Toscano]

On Monday 09 December 2013 17:58:37 Richard W.M. Jones wrote:
> On Mon, Dec 09, 2013 at 06:17:00PM +0100, Pino Toscano wrote:
> > Followup of ed4bcb119cb908e98ecc1107dcd8b740ee2f484f.
> > ---
> > 
> >  sysprep/sysprep_operation_delete.ml | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> > 
> > diff --git a/sysprep/sysprep_operation_delete.ml
> > b/sysprep/sysprep_operation_delete.ml index 59d5485..a08d4d0 100644
> > --- a/sysprep/sysprep_operation_delete.ml
> > +++ b/sysprep/sysprep_operation_delete.ml
> > @@ -40,7 +40,10 @@ let op = {
> > 
> >      pod_description = Some (s_"\
> >  
> >  Delete specified files or directories.
> > 
> > -Use the I<--delete> option to specify a path to remove.");
> > +Use the I<--delete> option to specify a path to remove.
> > +
> > +You can use shell glob characters in the specified path, i.e.
> > +C</some/dir/prefix*>.");
> 
> How about /var/log/*.log as the example?  Also I think we should tell
> people to quote the arg to defend it from the shell.  So it could look
> like this:
> 
> -Use the I<--delete> option to specify a path to remove.");
> +Use the I<--delete> option to specify a path to remove.
> +
> +You can use shell glob characters in the specified path,
> +for example:
> +
> + virt-sysprep --delete '/var/log/*.log'");
> 
> (Putting a space in front makes verbatim text in POD)

Good starting point, I've spelt the escaping need explicitly; updated 
patch coming.

-- 
Pino Toscano