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