[Libguestfs] [PATCHx7] Misc patches
Jim Meyering
jim at meyering.net
Thu Aug 13 14:55:28 UTC 2009
Richard W.M. Jones wrote:
>>From 80c479162db74c461d671128237975aecc982a5e Mon Sep 17 00:00:00 2001
> From: Richard Jones <rjones at trick.home.annexia.org>
> Date: Thu, 13 Aug 2009 14:39:31 +0100
> Subject: [PATCH 2/7] cp/cp-a/mv parameters marked as Pathname.
>
> ---
> daemon/cpmv.c | 4 ----
> src/generator.ml | 6 +++---
> 2 files changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/daemon/cpmv.c b/daemon/cpmv.c
> index 1007203..47b8aa2 100644
> --- a/daemon/cpmv.c
> +++ b/daemon/cpmv.c
> @@ -52,10 +52,6 @@ cpmv_cmd (const char *cmd, const char *flags, const char *src, const char *dest)
> char *err;
> int r;
>
> - NEED_ROOT (return -1);
> - ABS_PATH (src, return -1);
> - ABS_PATH (dest, return -1);
> -
> srcbuf = sysroot_path (src);
> if (srcbuf == NULL) {
> reply_with_perror ("malloc");
> diff --git a/src/generator.ml b/src/generator.ml
> index b4dd4cd..1278afd 100755
> --- a/src/generator.ml
> +++ b/src/generator.ml
> @@ -2170,7 +2170,7 @@ See also: C<guestfs_scrub_device>.");
> This command installs GRUB (the Grand Unified Bootloader) on
> C<device>, with the root directory being C<root>.");
>
> - ("cp", (RErr, [String "src"; String "dest"]), 87, [],
> + ("cp", (RErr, [Pathname "src"; Pathname "dest"]), 87, [],
> [InitBasicFS, Always, TestOutput (
> [["write_file"; "/old"; "file content"; "0"];
> ["cp"; "/old"; "/new"];
> @@ -2189,7 +2189,7 @@ C<device>, with the root directory being C<root>.");
> This copies a file from C<src> to C<dest> where C<dest> is
> either a destination filename or destination directory.");
>
> - ("cp_a", (RErr, [String "src"; String "dest"]), 88, [],
> + ("cp_a", (RErr, [Pathname "src"; Pathname "dest"]), 88, [],
> [InitBasicFS, Always, TestOutput (
> [["mkdir"; "/olddir"];
> ["mkdir"; "/newdir"];
> @@ -2201,7 +2201,7 @@ either a destination filename or destination directory.");
> This copies a file or directory from C<src> to C<dest>
> recursively using the C<cp -a> command.");
>
> - ("mv", (RErr, [String "src"; String "dest"]), 89, [],
> + ("mv", (RErr, [Pathname "src"; Pathname "dest"]), 89, [],
> [InitBasicFS, Always, TestOutput (
> [["write_file"; "/old"; "file content"; "0"];
> ["mv"; "/old"; "/new"];
More of the same. Looks fine.
ACK.
More information about the Libguestfs
mailing list