Hi,
On Tue, May 16, 2017 at 05:27:01PM +0200, Christophe Fergeau wrote:
> Took me a few seconds to understand what this is about ;) This adds a
> #define to avoid hardcoding the path in 2 different places. At first I
> thought you were forcing a (new) default value.
>
> Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
Indeed. I've included your comment "This adds a #define to avoid
hardcoding the path in 2 different places." in the commit log.
Thanks,
>
> Christophe
>
> On Fri, May 12, 2017 at 01:19:37PM +0200, Victor Toso wrote:
> > From: Victor Toso <me@xxxxxxxxxxxxxx>
> >
> > ---
> > src/vdagent/vdagent.c | 2 +-
> > src/vdagentd-proto.h | 2 ++
> > src/vdagentd/vdagentd.c | 2 +-
> > 3 files changed, 4 insertions(+), 2 deletions(-)
> >
> > diff --git a/src/vdagent/vdagent.c b/src/vdagent/vdagent.c
> > index 6f025e8..9900303 100644
> > --- a/src/vdagent/vdagent.c
> > +++ b/src/vdagent/vdagent.c
> > @@ -44,7 +44,7 @@
> > #include "x11.h"
> > #include "file-xfers.h"
> >
> > -static const char *portdev = "/dev/virtio-ports/com.redhat.spice.0";
> > +static const char *portdev = DEFAULT_VIRTIO_PORT_PATH;
> > static const char *vdagentd_socket = VDAGENTD_SOCKET;
> > static int debug = 0;
> > static const char *fx_dir = NULL;
> > diff --git a/src/vdagentd-proto.h b/src/vdagentd-proto.h
> > index 9815488..f72a890 100644
> > --- a/src/vdagentd-proto.h
> > +++ b/src/vdagentd-proto.h
> > @@ -25,6 +25,8 @@
> >
> > #define VDAGENTD_SOCKET "/var/run/spice-vdagentd/spice-vdagent-sock"
> >
> > +#define DEFAULT_VIRTIO_PORT_PATH "/dev/virtio-ports/com.redhat.spice.0"
> > +
> > enum {
> > VDAGENTD_GUEST_XORG_RESOLUTION, /* client -> daemon, arg1: overall width,
> > arg2: overall height, data: array of
> > diff --git a/src/vdagentd/vdagentd.c b/src/vdagentd/vdagentd.c
> > index f3ac606..800a09c 100644
> > --- a/src/vdagentd/vdagentd.c
> > +++ b/src/vdagentd/vdagentd.c
> > @@ -54,7 +54,7 @@ struct agent_data {
> >
> > /* variables */
> > static const char *pidfilename = "/var/run/spice-vdagentd/spice-vdagentd.pid";
> > -static const char *portdev = "/dev/virtio-ports/com.redhat.spice.0";
> > +static const char *portdev = DEFAULT_VIRTIO_PORT_PATH;
> > static const char *vdagentd_socket = VDAGENTD_SOCKET;
> > static const char *uinput_device = "/dev/uinput";
> > static int debug = 0;
> > --
> > 2.12.2
> >
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> > https://lists.freedesktop.org/mailman/listinfo/spice-devel
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel