<div dir="ltr"><div dir="ltr">Hi<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Sep 9, 2020 at 6:45 PM Jakub Janku <<a href="mailto:jjanku@redhat.com">jjanku@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, Sep 9, 2020 at 4:36 PM Frediano Ziglio <<a href="mailto:fziglio@redhat.com" target="_blank">fziglio@redhat.com</a>> wrote:<br>
><br>
> > On Wed, Sep 9, 2020 at 4:16 PM Frediano Ziglio <<a href="mailto:fziglio@redhat.com" target="_blank">fziglio@redhat.com</a>> wrote:<br>
> > ><br>
> > > > > commit 4b9092b96b8da946ff3d17922b0fcf225c5dc81f<br>
> > > ><br>
> > > > > Author: Jakub Janků < <a href="mailto:jjanku@redhat.com" target="_blank">jjanku@redhat.com</a> ><br>
> > > ><br>
> > > > > Date: Sat May 23 16:28:52 2020 +0200<br>
> > > ><br>
> > ><br>
> > > > > session: make spice_session_get_webdav_server() public<br>
> > > ><br>
> > ><br>
> > > > > It will be necessary to access the webdav server from<br>
> > > > > spice-gtk-session.c<br>
> > > ><br>
> > > > > which isn't compiled with spice-session-priv.h, so make<br>
> > > ><br>
> > > > > spice_session_get_webdav_server() public.<br>
> > > ><br>
> > ><br>
> > > > I haven't looked at the whole series. Wouldn't it make sense to make it a<br>
> > > > read-only property instead?<br>
> > ><br>
> > > It sounds reasonable for me.<br>
> > > Jakub ?<br>
> > ><br>
> ><br>
> > I agree.<br>
> ><br>
> > Revert the commits please. I'll reopen the merge request once I have it<br>
> > ready.<br>
> ><br>
> > Cheers,<br>
> > Jakub<br>
> ><br>
><br>
> To be honest I don't see the need to revert commits, it's just a change<br>
> from public to private.<br>
<br>
Ok, so should I open a separate MR?<br>
<br>
To make sure that I didn't misunderstand it: the suggestion is to keep<br>
spice_session_get_webdav_server() private and install a new<br>
SpiceSession read-only property "webdav", correct?<br><br></blockquote><div><br></div><div>yes (the main motivation is to avoid adding new library symbols, and properties can be looked up at runtime, which may avoid bumping dependencies in some cases)<br></div></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature">Marc-André Lureau<br></div></div>