From: Amit Shah <amit.shah@redhat.com>
To: Eric Blake <eblake@redhat.com>
Cc: "Juan Quintela" <quintela@redhat.com>,
"qemu list" <qemu-devel@nongnu.org>,
"Markus Armbruster" <armbru@redhat.com>,
"Alexander Graf" <agraf@suse.de>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Andreas Färber" <afaerber@suse.de>,
"Dr. David Alan Gilbert" <dgilbert@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v3 00/18] migration: add static analysis tool to check vmstate compat
Date: Tue, 17 Jun 2014 19:29:17 +0530 [thread overview]
Message-ID: <20140617135917.GA24644@grmbl.mre> (raw)
In-Reply-To: <53A043CF.9030800@redhat.com>
On (Tue) 17 Jun 2014 [07:34:07], Eric Blake wrote:
>
>
> On 06/17/2014 07:19 AM, Amit Shah wrote:
> > Hello,
> >
> > v3:
> > - Python script returns an error code: 0 if no errors, positive for
> > the number of errors identified.
>
> Returning the error count as $? is unreliable - if you have exactly 256
> errors, that gives the same status as if you had success. Does your
> code guarantee non-zero return in that case?
No; any good ideas for this, then? Just limit to 255, and never bump
upwards?
I'm only introducing this so that future integration with 'make check'
can be noticed by scripts.
Amit
next prev parent reply other threads:[~2014-06-17 13:59 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-17 13:19 [Qemu-devel] [PATCH v3 00/18] migration: add static analysis tool to check vmstate compat Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 01/18] migration: dump vmstate info as a json file for static analysis Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 02/18] vmstate-static-checker: script to validate vmstate changes Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 03/18] tests: vmstate static checker: add dump1 and dump2 files Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 04/18] tests: vmstate static checker: incompat machine types Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 05/18] tests: vmstate static checker: add version error in main section Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 06/18] tests: vmstate static checker: version mismatch inside a Description Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 07/18] tests: vmstate static checker: minimum_version_id check Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 08/18] tests: vmstate static checker: remove a section Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 09/18] tests: vmstate static checker: remove a field Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 10/18] tests: vmstate static checker: remove last field in a struct Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 11/18] tests: vmstate static checker: change description name Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 12/18] tests: vmstate static checker: remove Fields Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 13/18] tests: vmstate static checker: remove Description Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 14/18] tests: vmstate static checker: remove Description inside Fields Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 15/18] tests: vmstate static checker: remove a subsection Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 16/18] tests: vmstate static checker: remove Subsections Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 17/18] tests: vmstate static checker: add substructure for usb-kbd for hid section Amit Shah
2014-06-17 13:19 ` [Qemu-devel] [PATCH v3 18/18] tests: vmstate static checker: add size mismatch inside substructure Amit Shah
2014-06-17 13:34 ` [Qemu-devel] [PATCH v3 00/18] migration: add static analysis tool to check vmstate compat Eric Blake
2014-06-17 13:59 ` Amit Shah [this message]
2014-06-17 15:25 ` Eric Blake
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140617135917.GA24644@grmbl.mre \
--to=amit.shah@redhat.com \
--cc=afaerber@suse.de \
--cc=agraf@suse.de \
--cc=armbru@redhat.com \
--cc=dgilbert@redhat.com \
--cc=eblake@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=quintela@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).