From: Avi Kivity <avi@redhat.com>
To: Chris Lalancette <clalance@redhat.com>
Cc: kvm@vger.kernel.org
Subject: Re: [PATCH]: Always use KVM_VERSION to build version number
Date: Sun, 09 Aug 2009 12:58:01 +0300 [thread overview]
Message-ID: <4A7E9DA9.3060204@redhat.com> (raw)
In-Reply-To: <4A7C1E96.3060200@redhat.com>
On 08/07/2009 03:31 PM, Chris Lalancette wrote:
> Avi,
> Trying to use libvirt with development snapshots of qemu-kvm is a bit
> problematic. The trouble is that for all development snapshots, the value that
> gets placed into this string:
>
> QEMU PC emulator version 0.10.0 (kvm-devel), Copyright (c) 2003-2008
>
> Is always kvm-devel. That means we can't tell if this is a kvm development
> snapshot built yesterday, or 6 months ago, which means that in turn we can't
> tell what features are available. While we can always tell people building
> their own qemu to force it by echoing a value into KVM_VERSION, it would be much
> better if this were done by default. Something like kvm-88-devel, which would
> signify that this the development happening after kvm-88, leading towards
> kvm-89. Would you accept something like the patch below, which would require
> you to edit the KVM_VERSION file twice during a release (once right before the
> release, to change it to kvm-89, and once right after the release to change it
> back to kvm-89-devel)?
>
>
This is problematic in two ways. One is that I am basically guaranteed
to forget to edit the file (which is why the release scripts generate
the name based on the tag).
On fix is to use git describe --match to find out what's the closest
release. But this is still quite bad as it doesn't account for branches
and forks.
How about adding 'qemu -describe-features' which will output, one line
per feature, what's supported (and limits where applicable)? I
understand libvirt already does this for some features using -help; this
is simply a formalization of that hack.
--
error compiling committee.c: too many arguments to function
next prev parent reply other threads:[~2009-08-09 9:52 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-07 12:31 [PATCH]: Always use KVM_VERSION to build version number Chris Lalancette
2009-08-09 9:58 ` Avi Kivity [this message]
2009-08-10 9:19 ` Mark McLoughlin
2009-08-10 9:39 ` [PATCH] configure: make default KVM_VERSION qemu-kvm-devel Mark McLoughlin
2009-08-10 9:47 ` Avi Kivity
2009-08-10 9:46 ` [PATCH]: Always use KVM_VERSION to build version number Avi Kivity
2009-08-10 9:52 ` Mark McLoughlin
2009-08-10 10:12 ` Avi Kivity
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=4A7E9DA9.3060204@redhat.com \
--to=avi@redhat.com \
--cc=clalance@redhat.com \
--cc=kvm@vger.kernel.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.