Re: [libvirt] virt-viewer, requirement question

2011-11-21 Thread Daniel P. Berrange
On Fri, Nov 18, 2011 at 03:38:07PM -0800, Jason Helfman wrote:
 I see that this was moved to vncconnection.h, however I don't see this patch
 in there. Is this patch still relevant to the code?
 
 http://osdir.com/ml/gtk-vnc-list/2009-11/msg00029.html
 
 Thanks,
 Jason
 
 
 This has already been fixed by FreeBSD Gnome team, and will be updated when
 9 comes out.

Do you have a pointer to the patch they applied to gtk-vnc ?  I would
like to apply it to the upstream GTK-VNC sources, so you don't need to
carry it separately in the future.


Daniel
-- 
|: http://berrange.com  -o-http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org  -o- http://virt-manager.org :|
|: http://autobuild.org   -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org   -o-   http://live.gnome.org/gtk-vnc :|

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


Re: [libvirt] virt-viewer, requirement question

2011-11-18 Thread Jason Helfman

On Fri, Nov 18, 2011 at 10:03:05AM +, Daniel P. Berrange thus spake:

On Thu, Nov 17, 2011 at 05:24:15PM -0800, Jason Helfman wrote:

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256

Hi,

I am the port maintainer for libvirt, and associated tools, in the FreeBSD
Project.

I was wondering if Require SPICE-GTK = 0.6.0 for virt-viewer, is
accurate, or is this optional.


With the latest virt-viewer release, you can disable either SPICE-GTK
or GTK-VNC by passing suitable args to ./configure. Obviously if you
disable them both, the resulting virt-viewer won't be very useful ;-P



Thanks! And had no idea you either wrote gtk-vnc, or contributed to it,
which leads me to my next question :)

I am in the midst of updating it to a current version, to update
virt-viewer, however I was unable to find the source of this patch that was
in FreeBSD. All the other patches have been incorporated into the code, it
seems.

--- src/gvnc.h.orig 2008-03-24 11:39:20.0 -0400
+++ src/gvnc.h  2008-03-24 11:40:22.0 -0400
@@ -3,6 +3,19 @@
 
 #include glib.h

 #include stdint.h
+#ifdef __FreeBSD__
+#include sys/endian.h
+
+#define __BYTE_ORDER   _BYTE_ORDER
+#define __LITTLE_ENDIAN_LITTLE_ENDIAN
+#define __BIG_ENDIAN   _BIG_ENDIAN
+
+#define bswap_16   bswap16
+#define bswap_32   bswap32
+#define bswap_64   bswap64
+#else
+#include endian.h
+#endif
 
 struct gvnc;


Thanks!
Jason

--
Jason Helfman
System Administrator
experts-exchange.com
http://www.experts-exchange.com/M_4830110.html
E4AD 7CF1 1396 27F6 79DD  4342 5E92 AD66 8C8C FBA5

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


Re: [libvirt] virt-viewer, requirement question

2011-11-18 Thread Jason Helfman

On Fri, Nov 18, 2011 at 09:37:11AM -0800, Jason Helfman thus spake:

On Fri, Nov 18, 2011 at 10:03:05AM +, Daniel P. Berrange thus spake:

On Thu, Nov 17, 2011 at 05:24:15PM -0800, Jason Helfman wrote:

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256

Hi,

I am the port maintainer for libvirt, and associated tools, in the FreeBSD
Project.

I was wondering if Require SPICE-GTK = 0.6.0 for virt-viewer, is
accurate, or is this optional.


With the latest virt-viewer release, you can disable either SPICE-GTK
or GTK-VNC by passing suitable args to ./configure. Obviously if you
disable them both, the resulting virt-viewer won't be very useful ;-P



Thanks! And had no idea you either wrote gtk-vnc, or contributed to it,
which leads me to my next question :)

I am in the midst of updating it to a current version, to update
virt-viewer, however I was unable to find the source of this patch that was
in FreeBSD. All the other patches have been incorporated into the code, it
seems.

--- src/gvnc.h.orig 2008-03-24 11:39:20.0 -0400
+++ src/gvnc.h  2008-03-24 11:40:22.0 -0400
@@ -3,6 +3,19 @@

 #include glib.h
 #include stdint.h
+#ifdef __FreeBSD__
+#include sys/endian.h
+
+#define __BYTE_ORDER   _BYTE_ORDER
+#define __LITTLE_ENDIAN_LITTLE_ENDIAN
+#define __BIG_ENDIAN   _BIG_ENDIAN
+
+#define bswap_16   bswap16
+#define bswap_32   bswap32
+#define bswap_64   bswap64
+#else
+#include endian.h
+#endif

 struct gvnc;

Thanks!
Jason



I see that this was moved to vncconnection.h, however I don't see this patch
in there. Is this patch still relevant to the code?

http://osdir.com/ml/gtk-vnc-list/2009-11/msg00029.html

Thanks,
Jason

--
Jason Helfman
System Administrator
experts-exchange.com
http://www.experts-exchange.com/M_4830110.html
E4AD 7CF1 1396 27F6 79DD  4342 5E92 AD66 8C8C FBA5

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


Re: [libvirt] virt-viewer, requirement question

2011-11-18 Thread Jason Helfman

I see that this was moved to vncconnection.h, however I don't see this patch
in there. Is this patch still relevant to the code?

http://osdir.com/ml/gtk-vnc-list/2009-11/msg00029.html

Thanks,
Jason



This has already been fixed by FreeBSD Gnome team, and will be updated when
9 comes out.

-jgh

--
Jason Helfman
System Administrator
experts-exchange.com
http://www.experts-exchange.com/M_4830110.html
E4AD 7CF1 1396 27F6 79DD  4342 5E92 AD66 8C8C FBA5

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