Re: [PATCH] Spell VMware with a lowercase w

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

 



On Tue, Feb 16, 2016 at 12:33:54PM +0100, Michal Privoznik wrote:
> On 15.02.2016 15:38, Ján Tomko wrote:
> > Replace all occurrences of VMWare outside the news.
> > ---
> >  docs/formatdomain.html.in | 2 +-
> >  docs/migration.html.in    | 2 +-
> >  docs/storage.html.in      | 2 +-
> >  libvirt.spec.in           | 2 +-
> >  src/util/virerror.c       | 2 +-
> >  src/util/virstoragefile.c | 2 +-
> >  tools/virsh.c             | 2 +-
> >  7 files changed, 7 insertions(+), 7 deletions(-)
> > 
> 
> 
> > diff --git a/tools/virsh.c b/tools/virsh.c
> > index b96dbda..eb84dd0 100644
> > --- a/tools/virsh.c
> > +++ b/tools/virsh.c
> > @@ -550,7 +550,7 @@ virshShowVersion(vshControl *ctl ATTRIBUTE_UNUSED)
> >      vshPrint(ctl, " OpenVZ");
> >  #endif
> >  #ifdef WITH_VMWARE
> > -    vshPrint(ctl, " VMWare");
> > +    vshPrint(ctl, " VMware");
> >  #endif
> >  #ifdef WITH_PHYP
> >      vshPrint(ctl, " PHYP");
> > 
> 
> I'm not sure we can do this. What if there's somebody relying to see
> VMWare? e.g.

This output is meant for human consumption and should not change
unless you do the compilation by hand. I don't see a reason to check
it automatically.

> 
> if [ -z "`virsh -V | grep VMWare`" ]; then
>   destroy_the_world();
> fi
> 

Then it's the end of the world as we know it.

> But maybe I'm too late since you've pushed this.
> 

Changes can be undone too.

Jan

Attachment: signature.asc
Description: Digital 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]