WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] Is Xend versioned?

To: Mark Williamson <mark.williamson@xxxxxxxxxxxx>
Subject: Re: [Xen-devel] Is Xend versioned?
From: Jim Fehlig <jfehlig@xxxxxxxxxx>
Date: Tue, 08 Jan 2008 19:06:04 -0700
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Tue, 08 Jan 2008 18:06:41 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <200801041808.20614.mark.williamson@xxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <47681A9B.3020402@xxxxxxxxxx> <200801041808.20614.mark.williamson@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Thunderbird 1.5.0.8 (X11/20060911)
Mark Williamson wrote:
>> When querying Xend for version this code is invoked in
>> tools/python/xen/xend/XendNode.py
>>
>>     def xen_version(self):
>>         info = self.xc.xeninfo()
>>
>>         try:
>>             from xen import VERSION
>>             info = {'Xen': '%(xen_major)d.%(xen_minor)d' % info,
>>                     'Xend': VERSION}
>>         except (ImportError, AttributeError):
>>             info = {'Xen': '%(xen_major)d.%(xen_minor)d' % info,
>>                     'Xend': '3.0.3'}
>>
>> I cannot find a xen module from which to import VERSION, resulting in
>> the hardcoded one.  I would cook up a patch if I understood how, or even
>> if, it should be versioned.
>>     
>
> You're right, I can't see what this is supposed to achieve...
>
> If you were to add:
>
> VERSION = "blah"
>
> to tools/python/xen/__init__.py
>
> then I imagine that would do the right thing.  As for whether it's necessary, 
> I'm not sure...  But presumably that code ought either to do something or to 
> be removed???
>   

I'm not sure there is any value in versioning xend itself.  Certainly
its interfaces (e.g. XenAPI) should be versioned but the whole daemon? 
I will spin a patch to remove it (or fix it) if there is consensus.

Regards,
Jim


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

<Prev in Thread] Current Thread [Next in Thread>