search for: file_owner

Linux.file_owner is not used by any other function, so remove it. Linux.is_file_owned is only used when removing kmod-xenpv on old RHEL releases, and so is only required to work for RPM. The old file_owner/is_file_owned functions were completely broken. This replaces them with a simpler, working implementation th...

What was happening in file_owner function did not match the description in the comment. When a path is owned by multiple packages the returned string was in fact a concatenation of the names of all packages that own it. E.g. for `Linux.is_file_owned g inspect "/etc"` the returned value was "filesystemyum" (i.e....

On Wed, 3 Aug 2016 11:14:26 +0100 "Richard W.M. Jones" <rjones@redhat.com> wrote: > On Mon, Aug 01, 2016 at 02:46:14PM +0200, Tomáš Golembiovský wrote: > > What was happening in file_owner function did not match the description > > in the comment. When a path is owned by multiple packages the returned > > string was in fact a concatenation of the names of all packages that own > > it. E.g. for `Linux.is_file_owned g inspect "/etc"` the returned value >...

...ed rather then hidden. Signed-off-by: Tomáš Golembiovský <tgolembi@redhat.com> --- v2v/linux.ml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/v2v/linux.ml b/v2v/linux.ml index e57dad6..46cb3ba 100644 --- a/v2v/linux.ml +++ b/v2v/linux.ml @@ -109,8 +109,8 @@ let rec file_owner (g : G.guestfs) inspect path = raise Not_found else raise exn - | Invalid_argument msg -> - raise Not_found + | Invalid_argument "index out of bounds" -> + error (f_"internal error: file_owner: rpm command returned n...

On Mon, Aug 01, 2016 at 02:46:14PM +0200, Tomáš Golembiovský wrote: > What was happening in file_owner function did not match the description > in the comment. When a path is owned by multiple packages the returned > string was in fact a concatenation of the names of all packages that own > it. E.g. for `Linux.is_file_owned g inspect "/etc"` the returned value > was "files...

Implement the 'remove', 'file_list_of_package', and 'file_owner' methods of the Linux module for the "deb" package manager (dpkg basically, on Debian and derived distributions). Also allow it for the main conversion code. --- v2v/convert_linux.ml | 2 +- v2v/linux.ml | 33 +++++++++++++++++++++++++++++++++ 2 files changed, 34 insertions...

On Mon, 8 Aug 2016 18:38:49 +0200 Pino Toscano <ptoscano@redhat.com> wrote: > Implement the 'remove', 'file_list_of_package', and 'file_owner' methods > of the Linux module for the "deb" package manager (dpkg basically, on > Debian and derived distributions). > > Also allow it for the main conversion code. > --- > v2v/convert_linux.ml | 2 +- > v2v/linux.ml | 23 +++++++++++++++++++++++ >...

v1 was here: https://www.redhat.com/archives/libguestfs/2017-October/msg00183.html v2: - Adds back the Debian test, but simplified. - Adds tests on Fedora & Debian. Rich.

Hi, this series implements the first bits in v2v to convert Debian/Ubuntu (and derived) guests. The series does not complete the support (see known issues below), but all the patches here should be fit for review and inclusion. The series does not enable the conversion, yet. Known issues: * there is no grubby nor Bootloader::Tools Perl module available in Debian, so there is no way to know

...o@redhat.com> wrote: > On Tuesday, 9 August 2016 13:55:36 CEST Tomáš Golembiovský wrote: > > On Mon, 8 Aug 2016 18:38:49 +0200 > > Pino Toscano <ptoscano@redhat.com> wrote: > > > > > Implement the 'remove', 'file_list_of_package', and 'file_owner' methods > > > of the Linux module for the "deb" package manager (dpkg basically, on > > > Debian and derived distributions). > > > > > > Also allow it for the main conversion code. > > > --- > > > v2v/convert_linux.ml | 2 +- &g...

Implement the 'remove', 'file_list_of_package', and 'file_owner' methods of the Linux module for the "deb" package manager (dpkg basically, on Debian and derived distributions). Also allow it for the main conversion code. --- v2v/convert_linux.ml | 2 +- v2v/linux.ml | 23 +++++++++++++++++++++++ 2 files changed, 24 insertions(+), 1 del...

On Tuesday, 9 August 2016 13:55:36 CEST Tomáš Golembiovský wrote: > On Mon, 8 Aug 2016 18:38:49 +0200 > Pino Toscano <ptoscano@redhat.com> wrote: > > > Implement the 'remove', 'file_list_of_package', and 'file_owner' methods > > of the Linux module for the "deb" package manager (dpkg basically, on > > Debian and derived distributions). > > > > Also allow it for the main conversion code. > > --- > > v2v/convert_linux.ml | 2 +- > > v2v/linux.ml...

...Golembiovský <tgolembi@redhat.com> > --- > v2v/linux.ml | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/v2v/linux.ml b/v2v/linux.ml > index e57dad6..46cb3ba 100644 > --- a/v2v/linux.ml > +++ b/v2v/linux.ml > @@ -109,8 +109,8 @@ let rec file_owner (g : G.guestfs) inspect path = > raise Not_found > else > raise exn > - | Invalid_argument msg -> > - raise Not_found > + | Invalid_argument "index out of bounds" -> > + error (f_"internal error:...

Hi, this series implements the first bits in v2v to convert Debian/Ubuntu (and derived) guests. The series does not complete the support (see known issues below), but all the patches here should be fit for review and inclusion. The series does not enable the conversion, yet. Known issues: * there is no grubby nor Bootloader::Tools Perl module available in Debian, so there is no way to know

...ed") - | Failure "int_of_string" -> () + | Failure _ -> () ) done; List.nth roots (!i - 1) diff --git a/v2v/linux.ml b/v2v/linux.ml index d449e10..729bb5d 100644 --- a/v2v/linux.ml +++ b/v2v/linux.ml @@ -145,7 +145,7 @@ let rec file_owner (g : G.guestfs) inspect path = raise Not_found else raise exn - | Invalid_argument "index out of bounds" -> + | Invalid_argument _ (* pkgs.(0) raises index out of bounds *) -> error (f_"internal error: file_owner: rpm comm...

...2 are very kind... Mr. Kelley suggest me to investigate on the winbind and Mr. Fraille (that runs into the same problem but solved it!) suggest me to add this line in smbfax-html.pl (that is the script that create the 2 files described up...) but seems that on my system all is ignored (why?): sub file_owner { my $file = shift; my @statent = stat($file) or return undef; my @pwent = getpwuid($statent[4]); # Put username in uppercase $pwent[0] =~ tr/[a-z]?/[A-Z]?/; ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ return $pwent[0]; } Sorry for the long mail, but its necessary to better explain my p...

Hallo Everyone, Please answer to: http://lists.samba.org/pipermail/samba/2002-November/084692.html Thanks a lot in advance, Gianluigi Di Vaio

...es = g#command_lines cmd in let files = Array.to_list files in List.sort compare files @@ -155,7 +155,7 @@ let file_list_of_package verbose (g : Guestfs.guestfs) inspect app = error (f_"don't know how to get list of files from package using %s") format -let rec file_owner verbose g inspect path = +let rec file_owner g inspect path = let package_format = inspect.i_package_format in match package_format with | "rpm" -> @@ -163,7 +163,7 @@ let rec file_owner verbose g inspect path = * a file, this deliberately only returns one package....

.. and a lot of refactoring. https://bugzilla.redhat.com/show_bug.cgi?id=1167623 Rich.

https://bugzilla.redhat.com/show_bug.cgi?id=1167623