<div dir="ltr"><br><div class="gmail_quote"><div dir="ltr">On Wed, Sep 27, 2017 at 1:57 PM Frediano Ziglio <<a href="mailto:fziglio@redhat.com">fziglio@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">><br>
> Hi,<br>
><br>
> On Wed, Sep 27, 2017 at 11:40:40AM +0000, Frediano Ziglio wrote:<br>
> > ><br>
> > > Hi,<br>
> > ><br>
> > > On Wed, Sep 27, 2017 at 10:24:46AM +0000, Frediano Ziglio wrote:<br>
> > > > ><br>
> > > > > ---<br>
> > > > >  src/vdagent/vdagent.c | 17 ++++++-----------<br>
> > > > >  1 file changed, 6 insertions(+), 11 deletions(-)<br>
> > > > ><br>
> > > > > diff --git a/src/vdagent/vdagent.c b/src/vdagent/vdagent.c<br>
> > > > > index dfb9284..6b4c127 100644<br>
> > > > > --- a/src/vdagent/vdagent.c<br>
> > > > > +++ b/src/vdagent/vdagent.c<br>
> > > > > @@ -168,7 +168,7 @@ static void usage(FILE *fp)<br>
> > > > >        "  -s <port>                         set virtio serial port\n"<br>
> > > > >        "  -S <filename>                     set udcs socket\n"<br>
> > > > >        "  -x                                don't daemonize\n"<br>
> > > > > -      "  -f <dir|xdg-desktop|xdg-download> file xfer save dir\n"<br>
> > > > > +      "  -f <dir>                          file xfer save dir\n"<br>
> > > > >        "  -o <0|1>                          open dir on file xfer<br>
> > > > >        completion\n",<br>
> > > > >        VERSION);<br>
> > > > >  }<br>
> > > > > @@ -313,18 +313,13 @@ reconnect:<br>
> > > > >          return 1;<br>
> > > > >      }<br>
> > > > ><br>
> > > > > -    if (!fx_dir) {<br>
> > > > > -        if (vdagent_x11_has_icons_on_desktop(x11))<br>
> > > > > -            fx_dir = "xdg-desktop";<br>
> > > > > -        else<br>
> > > > > -            fx_dir = "xdg-download";<br>
> > > > > -    }<br>
> > > > > +    if (!fx_dir)<br>
> > > > > +        fx_dir =<br>
> > > > > g_get_user_special_dir(vdagent_x11_has_icons_on_desktop(x11) ?<br>
> > > > > +                                        G_USER_DIRECTORY_DESKTOP :<br>
> > > > > +                                        G_USER_DIRECTORY_DOWNLOAD);<br>
> > > > >      if (fx_open_dir == -1)<br>
> > > > >          fx_open_dir = !vdagent_x11_has_icons_on_desktop(x11);<br>
> > > > > -    if (!strcmp(fx_dir, "xdg-desktop"))<br>
> > > > > -        fx_dir = g_get_user_special_dir(G_USER_DIRECTORY_DESKTOP);<br>
> > > > > -    else if (!strcmp(fx_dir, "xdg-download"))<br>
> > > > > -        fx_dir = g_get_user_special_dir(G_USER_DIRECTORY_DOWNLOAD);<br>
> > > > > +<br>
> > > > >      if (fx_dir) {<br>
> > > > >          vdagent_file_xfers = vdagent_file_xfers_create(client,<br>
> > > > >          fx_dir,<br>
> > > > >                                                         fx_open_dir,<br>
> > > > >                                                         debug);<br>
> > > ><br>
> > > > This will break updated if some people configured these values.<br>
> > > > I think we can have GLib and the current behaviour.<br>
> > ><br>
> > > If we do a release before this changes, we can deprecate this options in<br>
> > > the release and remove them after the release.<br>
> > ><br>
> > > Personally, I don't see much reason to keep this extra options around.<br>
> > ><br>
> > > Cheers,<br>
> > >     Victor<br>
> > ><br>
> ><br>
> > The rule we agree on is one release to announce the deprecation and<br>
> > remove the feature after 2 releases that is on the current schema 0.20.0.<br>
> ><br>
> > Frediano<br>
><br>
> For all components? One thing is spice-server and spice-gtk which are<br>
> libraries for QEMU and virt-viewer, etc. The agent is not a library and<br>
> we are breaking command line options...<br>
><br>
> Cheers,<br>
>     toso<br>
><br>
><br>
<br>
Command line options for an application is the public interface.<br>
I would follow the rule for all public interfaces.<br>
<br>
Frediano<br></blockquote><div><br></div><div>I don't mind keeping this option, although it's imho kinda useless.</div><div>Should it be marked as deprecated?</div><div><br></div><div>Cheers,</div><div>Jakub</div></div></div>