> On March 2, 2016 at 8:45 PM Wolfgang Link <w.l...@proxmox.com> wrote:
>
>
> syscmd use run_command with noout what return the only exitcode.
OK, but why do you use
> eval { syscmd ("/sbin/vgcreate pve $lvmdev") };
then? I am confused now.
> > Dietmar Maurer <diet...@proxmox.com> hat am 2. März 2016 um 17:11
> > geschrieben:
> >
> >
> > comments inline:
> >
> > > diff --git a/proxinstall b/proxinstall
> > > index ec15477..7a67623 100755
> > > --- a/proxinstall
> > > +++ b/proxinstall
> > > @@ -834,8 +834,11 @@ sub create_lvm_volumes {
> > > # so pe_start is aligned on a 128k boundary (advantage for SSDs)
> > > syscmd ("/sbin/pvcreate --metadatasize 250k -y -ff $lvmdev") == 0 ||
> > > die "unable to initialize physical volume $lvmdev";
> > > - syscmd ("/sbin/vgcreate pve $lvmdev") == 0 ||
> >
> > seems we use this pattern everywhere - but syscmd() return
> > the output from stdout, not the exit code.
> >
> > something is wrong with that?!
> >
> > > - die "unable to create volume group";
> > > +
> > > + eval { syscmd ("/sbin/vgcreate pve $lvmdev") };
> > > + if (my $err = $@) {
> > > + die "unable to create volume group: $err";
> > > + }
> > >
> > > my $hdgb = int($os_size/(1024*1024));
> > > my $space = (($hdgb > 128) ? 16 : ($hdgb/8))*1024*1024;
> > > --
> > > 2.1.4
> > >
> > >
> > > _______________________________________________
> > > pve-devel mailing list
> > > pve-devel@pve.proxmox.com
> > > http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> > >
_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
