Richard W.M. Jones
2011-Mar-15 16:51 UTC
[Libguestfs] [PATCH] perl: Binding and test for guestfs_last_errno (RHBZ#672491).
-- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones virt-p2v converts physical machines to virtual machines. Boot with a live CD or over the network (PXE) and turn machines into Xen guests. http://et.redhat.com/~rjones/virt-p2v -------------- next part -------------->From 883390fb2e8cdaf81223eb41c87f245571e3d63d Mon Sep 17 00:00:00 2001From: Richard W.M. Jones <rjones at redhat.com> Date: Tue, 15 Mar 2011 16:46:00 +0000 Subject: [PATCH] perl: Binding and test for guestfs_last_errno (RHBZ#672491). --- generator/generator_perl.ml | 34 +++++++++++++++++++ perl/t/810-mkdir-eexist.t | 76 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 110 insertions(+), 0 deletions(-) create mode 100644 perl/t/810-mkdir-eexist.t diff --git a/generator/generator_perl.ml b/generator/generator_perl.ml index 09bf20f..c832469 100644 --- a/generator/generator_perl.ml +++ b/generator/generator_perl.ml @@ -262,6 +262,17 @@ PREINIT: guestfs_delete_event_callback (g, event_handle); +SV * +last_errno (g) + guestfs_h *g; +PREINIT: + int errnum; + CODE: + errnum = guestfs_last_errno (g); + RETVAL = newSViv (errnum); + OUTPUT: + RETVAL + "; List.iter ( @@ -625,6 +636,9 @@ with libvirt. All errors turn into calls to C<croak> (see L<Carp(3)>). +The error string from libguestfs is directly available from +C<$@>. Use the C<last_errno> method if you want to get the errno. + =head1 METHODS =over 4 @@ -738,6 +752,26 @@ this function. This removes the callback which was previously registered using C<set_event_callback>. +=item $errnum = $h->last_errno (); + +This returns the last error number (errno) that happened on the +handle C<$h>. + +If successful, an errno integer not equal to zero is returned. + +If no error number is available, this returns 0. +See L<guestfs(3)/guestfs_last_errno> for more details of why +this can happen. + +You can use the standard Perl module L<Errno(3)> to compare +the numeric error returned from this call with symbolic +errnos: + + $h->mkdir (\"/foo\"); + if ($h->last_errno() == Errno::EEXIST()) { + # mkdir failed because the directory exists already. + } + =cut "; diff --git a/perl/t/810-mkdir-eexist.t b/perl/t/810-mkdir-eexist.t new file mode 100644 index 0000000..86b4e0e --- /dev/null +++ b/perl/t/810-mkdir-eexist.t @@ -0,0 +1,76 @@ +# libguestfs Perl bindings -*- perl -*- +# Copyright (C) 2011 Red Hat Inc. +# +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 2 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program; if not, write to the Free Software +# Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. + +# Test $g->last_errno (RHBZ#672491). + +use strict; +use warnings; +use Test::More tests => 15; + +use Errno; + +use Sys::Guestfs; + +my $g = Sys::Guestfs->new (); +ok ($g); + +open FILE, ">test.img"; +truncate FILE, 500*1024*1024; +close FILE; +ok (1); + +$g->add_drive_opts ("test.img", format => "raw"); +ok (1); + +$g->launch (); +ok (1); + +$g->part_disk ("/dev/sda", "mbr"); +ok (1); +$g->mkfs ("ext2", "/dev/sda1"); +ok (1); + +$g->mount_options ("", "/dev/sda1", "/"); +ok (1); + +# Directory doesn't exist, so this mkdir should succeed. +$g->mkdir ("/foo"); +ok (1); + +# Directory exists, we should be able to recover gracefully. +eval { + $g->mkdir ("/foo"); +}; +ok ($@); +my $err = $g->last_errno (); +ok ($err > 0); +ok ($err == Errno::EEXIST()); + +# Can't create subdirectories with missing parents; this should +# be a different errno. +eval { + $g->mkdir ("/bar/baz"); +}; +ok ($@); +$err = $g->last_errno (); +ok ($err > 0); +ok ($err != Errno::EEXIST()); + +undef $g; +ok (1); + +unlink ("test.img"); -- 1.7.4
Maybe Matching Threads
- [PATCH] perl: Add explicit close() method (RHBZ#602592).
- [PATCH] daemon: Work around udevsettle issue (RHBZ#548121).
- [PATCH] Add a prefix to output when tracing (RHBZ#673479).
- [PATCH] guestfish: Use xstrtol to parse integers (RHBZ#557655).
- [PATCH] Unify guestfish and guestmount options processing (RHBZ#642932).