Re: [PATCH v2 1/5] storage: adds with_storage_vstorage buils option

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jan 27, 2017 at 12:31:03 +0300, Olga Krishtal wrote:
> On 27/01/17 12:09, Peter Krempa wrote:
> > On Tue, Jan 17, 2017 at 17:10:55 +0300, Olga Krishtal wrote:
> > > This patch only adds  --with-storage-vstorage build option.
> > > In order to use vstorage as a backend for storage pool
> > > vstorage tools should be installed.
> > > 
> > > Signed-off-by: Olga Krishtal <okrishtal@xxxxxxxxxxxxx>
> > > ---
> > >   configure.ac                |  4 +++
> > >   m4/virt-storage-vstorage.m4 | 73 +++++++++++++++++++++++++++++++++++++++++++++
> > >   2 files changed, 77 insertions(+)
> > >   create mode 100644 m4/virt-storage-vstorage.m4
> > [...]
> > 
> > > @@ -0,0 +1,73 @@
> > > +dnl The storage vstorage check
> > > +dnl
> > > +dnl Copyright (C) 2016  Parallels IP Holdings GmbH, Inc.
> > This does not make sense. Is this company a GmbH or an Inc ?
> 
> Parallels IP Holdings GmbH - this is correct

By the way, copying a file and than changing copyright information
actually breaks the copyright.

This is the case for this file, since there's a typo left from the 'fs'
storage code that I'm going to send a patch for in a while. Also the
leftover "Inc" from changing Red Hat to Parallels .. is a second clue.

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux