On Thu, Dec 11, 2008 at 12:44:32PM +0100, Daniel Veillard wrote:
> On Wed, Dec 10, 2008 at 06:06:51AM -0500, Itamar Heim wrote:
> > Sure - probably not a good idea to send patches at such a late hour anyway
> > :)
> > Attached diff.
> >
> > Some more details:
> > For those using SDL ("for some crazy reason" as Berrange put it in the
> > original patch), there is a use for the fullscreen option, causing the
> > qemu window to open in full screen, change resolution, etc.
> > This patch accepts an additional attribute in the graphics element:
> > <graphics type='sdl' display=':0.1' xauth='/root/.Xauthority'
> > fullscreen='true'/>
> >
> > If fullscreen is true, "-full-screen" is added to the qemu command line
> > (unlike the display and auth which are added as environment variables)
>
> Okay, to avoid having this patch forgotten in a nitpicking discussion
> (I'm guilty I started :-) I commited the following which integrate the
> associated changes and keep the final code in line with the existing one:
> - make the check at parse time
> - use an int to store the data
> - use yes/no as the accepted values
> - error out if the attribute value does not follow
> - drop all bool usage
> - fix docs/libvirt.rng to detail the new optional attribute and its 2
> values
> - fix docs/formatdomain.html[.in] to describe the attributes for the
> sdl display type.
> - fix the test to use the new value 'yes' in the xml
>
> I guess this covers most of the changes needed for this patch, now you
> can blame me if I forgot something :-) but this passes the tests (well
> there is errors on the uml tests I didn't figured out yet but looks
> unrelated.
ACK, the committed patch looks good to me.
Daniel
--
|: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :|
|: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|
--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list