<div dir="ltr"><div>Hi, do you want to keep installing spicy-screenshot and spicy-stats? They are also tools just for testing. </div><div> </div>Pavel </div><div class="gmail_extra"> <div class="gmail_quote">2017-08-10 11:37 GMT+02:00 Marc-André Lureau <<a href="mailto:marcandre.lureau@redhat.com" target="_blank">marcandre.lureau@redhat.com</a>>: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi ----- Original Message ----- > Hi, > > On Wed, Aug 09, 2017 at 10:48:58PM +0200, <a href="mailto:marcandre.lureau@redhat.com">marcandre.lureau@redhat.com</a> wrote: > > From: Marc-André Lureau <<a href="mailto:marcandre.lureau@redhat.com">marcandre.lureau@redhat.com</a>> > > > > spicy is only a test UI. A proper end-user UI is provided by > > virt-viewer, vinagre or Boxes for example. > > > > Signed-off-by: Marc-André Lureau <<a href="mailto:marcandre.lureau@redhat.com">marcandre.lureau@redhat.com</a>> > > I don't see why a testing tool should not be installed... Because users expect it to be a fully-featured client, which is not. Imho, as a testing tool, it's fine to have it only in the build directory. You may install it manually, but it shouldn't be there by default. We may want to have --enable-installed-tests, that would be a different patch though <a href="https://wiki.gnome.org/Initiatives/GnomeGoals/InstalledTests" rel="noreferrer" target="_blank">https://wiki.gnome.org/Initiatives/GnomeGoals/InstalledTests</a> <div class="HOEnZb"><div class="h5"> > > > --- > > tools/Makefile.am | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/tools/Makefile.am b/tools/Makefile.am > > index 7c2d90e..5b7c4de 100644 > > --- a/tools/Makefile.am > > +++ b/tools/Makefile.am > > @@ -13,7 +13,7 @@ TOOLS_CPPFLAGS = \ > > $(NULL) > > > > if WITH_GTK > > -bin_PROGRAMS += spicy > > +noinst_PROGRAMS = spicy > > TOOLS_CPPFLAGS += $(GTK_CFLAGS) > > endif > > > > -- > > 2.14.0.1.geff633fa0 > > > > _______________________________________________ > > Spice-devel mailing list > > <a href="mailto:Spice-devel@lists.freedesktop.org">Spice-devel@lists.freedesktop.org</a> > > <a href="https://lists.freedesktop.org/mailman/listinfo/spice-devel" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/spice-devel</a> > _______________________________________________ Spice-devel mailing list <a href="mailto:Spice-devel@lists.freedesktop.org">Spice-devel@lists.freedesktop.org</a> <a href="https://lists.freedesktop.org/mailman/listinfo/spice-devel" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/spice-devel</a> </div></div></blockquote></div> </div>