[Engine-devel] CodeQuality/Automated Checking
Alon Bar-Lev
alonbl at redhat.com
Thu Dec 12 11:15:17 UTC 2013
----- Original Message -----
> From: "Eli Mesika" <emesika at redhat.com>
> To: "Sven Kieske" <S.Kieske at mittwald.de>
> Cc: engine-devel at ovirt.org
> Sent: Thursday, December 12, 2013 1:10:20 PM
> Subject: Re: [Engine-devel] CodeQuality/Automated Checking
>
>
>
> ----- Original Message -----
> > From: "Sven Kieske" <S.Kieske at mittwald.de>
> > To: engine-devel at ovirt.org
> > Sent: Thursday, December 12, 2013 10:14:16 AM
> > Subject: [Engine-devel] CodeQuality/Automated Checking
> >
> > Hi,
> >
> > I just stumbled upon this:
> >
> > http://gerrit.ovirt.org/#/c/22310/1/vdsm.spec.in
> >
> > such errors could be prevented easily.
> > either by using bash -n $scriptname.sh before commit
> > or by automated syntaxchecking via jenkins ?
>
> +1
first, using bash is not a good idea... better to use POSIX compliant shell.
second, these changes should be part of Makefile just like we validate python scripts.
I can workout some patch to do this.
>
> >
> > what do you think?
> > --
> > Mit freundlichen Grüßen / Regards
> >
> > Sven Kieske
> >
> > Systemadministrator
> > Mittwald CM Service GmbH & Co. KG
> > Königsberger Straße 6
> > 32339 Espelkamp
> > T: +49-5772-293-100
> > F: +49-5772-293-333
> > https://www.mittwald.de
> > Geschäftsführer: Robert Meyer
> > St.Nr.: 331/5721/1033, USt-IdNr.: DE814773217, HRA 6640, AG Bad Oeynhausen
> > Komplementärin: Robert Meyer Verwaltungs GmbH, HRB 13260, AG Bad Oeynhausen
> > _______________________________________________
> > Engine-devel mailing list
> > Engine-devel at ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/engine-devel
> >
> _______________________________________________
> Engine-devel mailing list
> Engine-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel
>
More information about the Engine-devel
mailing list