On 02/26/2013 03:16 AM, Viktor Mihajlovski wrote:
> Can we please revert the following commits before rc2;
> 
> 24aa7f8d11054b7b2e643cf3cd5c80a199764af0 (S390: Documentation for  CCW
> address type)
> 0bbbd42c30543d8251536c2fa11166834c886ada (S390: domain_conf support for
> CCW)
> 
> this is for two reasons:
> 1. They don't make sense without the rest of the series which is still
> pending review (and probably needs a rebase meanwhile) and the result
> already shows up on libvirt.org
> 2. I received feedback from someone more platform-savvy than I am, that
> one of the XML attribute names (schid) is misleading

Yes, avoiding the trap of releasing XML early, only to force ourselves
into supporting that design forever, seems like a reasonable reason to
revert while waiting for the more complete design to get a thorough review.

> 
> Reverting gives me the opportunity to provide a new series on top of
> 1.0.3 which can be hopefully applied in time for the next release.

Thanks for bringing up the question; I've reverted those patches as
requested.  It also lends a bit more weight to the question about DHCP
<option>, which is another XML addition where we are not sure if the
final design is ready for 1.0.3.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to