<div dir="ltr"><div><div>Thanks, I'll give it a try tonight.<br><br></div>Cheers,<br></div>André Miranda<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 21, 2017 at 8:18 AM, Vladimír Chren <span dir="ltr"><<a href="mailto:vlado.chren@inmail.sk" target="_blank">vlado.chren@inmail.sk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi<br>
<br>
On Po, 2017-03-20 at 21:54 -0300, André Miranda wrote:<br>
> Hi Vladimír,<br>
> Sorry for the delay.<br>
<br>
No problem.<br>
<br>
> I wasn't able to apply the patches against current master (4feb1d4),<br>
> if possible, upload a GitHub fork with your commits.<br>
<br>
You've probably tried to apply all the patches at once, but only<br>
"cumulative" one is needed. I split this one patch for easy review<br>
purposes to smaller ones. Reading it together would be too tiresome,<br>
cause is too long and repetitive, but as separated and smaller changes,<br>
it can take up to 10-15 minutes to review. I should have send them as<br>
diffs instead. (you can apply cumulative OR all other patch files)<br>
<br>
I made one addition to previous though, to make sure there are no<br>
futher changes which can brake what is now prepared, I added CFLAGS="-<br>
DGSEAL_ENABLE $CFLAGS" to <a href="http://configure.ac.in" rel="noreferrer" target="_blank">configure.ac.in</a>. This can be removed after<br>
porting is finished.<br>
<br>
I also uploaded it to github<br>
<a href="https://github.com/vc-01/thunar" rel="noreferrer" target="_blank">https://github.com/vc-01/<wbr>thunar</a><br>
<br>
<br>
<br>
<br>
Best regards<br>
<br>
Vladimir<br>
<br>
<br>
><br>
> Cheers,<br>
> André Miranda<br>
><br>
> On 03/10/2017 05:31 AM, Vladimír Chren wrote:<br>
><br>
> > Hello.<br>
> ><br>
> > I'm sending patches as attachments - separated as mentioned before<br>
> > for<br>
> > easy reviewing, also including cumulative one.<br>
> ><br>
> > This work (surely not entertaining) solves this one thing<br>
> >  $ make CFLAGS+="-DGSEAL_ENABLE"<br>
> ><br>
> > This is all current and can be a base for further gtk3 porting.<br>
> ><br>
> ><br>
> > Greets<br>
> ><br>
> > --<br>
> > vlado<br>
> ><br>
> ><br>
> > On Ut, 2017-03-07 at 22:53 -0300, André Miranda wrote:<br>
> > > Hi Vladmir,<br>
> > > Every help in the porting effort is more than welcome!<br>
> > > Currently there is unmaintained work[1][2], you can setup a<br>
> > > personal<br>
> > > repository on GitHub and use them as a start point.<br>
> > > Many components and panel plugins have been ported, they are also<br>
> > > good references.<br>
> > > However I suggest you talk with devs on #xfce-dev so they can<br>
> > > give<br>
> > > you better hints than me.<br>
> > ><br>
> > > 1 - <a href="https://wiki.xfce.org/releng/4.14/roadmap/thunar" rel="noreferrer" target="_blank">https://wiki.xfce.org/releng/<wbr>4.14/roadmap/thunar</a><br>
> > > 2 - <a href="https://github.com/rgcjonas/thunar-gtk3" rel="noreferrer" target="_blank">https://github.com/rgcjonas/<wbr>thunar-gtk3</a><br>
> > ><br>
> > > Cheers,<br>
> > > André Miranda<br>
> > ><br>
> > > On Mon, Mar 6, 2017 at 4:54 AM, Vladimír Chren <vlado.chren@inmai<br>
> > > <a href="http://l.sk" rel="noreferrer" target="_blank">l.sk</a><br>
> > > > wrote:<br>
> > > > Hello All.<br>
> > > ><br>
> > > > I don't know details about Thunar gtk3 porting status but it<br>
> > > > looks<br>
> > > > that<br>
> > > > there is no big progress.<br>
> > > ><br>
> > > > Would you be interested in patches solving one preparation step<br>
> > > > of<br>
> > > > porting Thunar to gtk3 according to manual<br>
> > > >  <a href="https://developer.gnome.org/gtk3/stable/gtk-migrating-2-to-3.h" rel="noreferrer" target="_blank">https://developer.gnome.org/<wbr>gtk3/stable/gtk-migrating-2-<wbr>to-3.h</a><br>
> > > > tml#<br>
> > > > id-1<br>
> > > > .6.3.3.5<br>
> > > ><br>
> > > > Chapter "Use accessor functions instead of direct access"<br>
> > > ><br>
> > > > Patches would look mostly like this<br>
> > > >  -  GTK_BIN (standard_view)->child<br>
> > > >  +  gtk_bin_get_child (GTK_BIN (standard_view))<br>
> > > ><br>
> > > > and like this<br>
> > > >  - renderer->is_expanded<br>
> > > >  + g_object_get (renderer, "is-expanded", &is_expanded, NULL);<br>
> > > ><br>
> > > > I'll separate each type of change to different patch.<br>
> > > ><br>
> > > > This could be then applied against master (i think) because<br>
> > > > it's<br>
> > > > still<br>
> > > > gtk2 (2.24.0) and nothing changes functionally. I'll reuse what<br>
> > > > was<br>
> > > > already done and update it to current master.<br>
> > > ><br>
> > > > Patches would be longer but simple to review.<br>
> > > ><br>
> > > > I'm asking first if it's necessary or useful to do it.<br>
> > > ><br>
> > > ><br>
> > > > Thank you<br>
> > > ><br>
> > > > --<br>
> > > > vlado<br>
> > > ><br>
> > > ><br>
> > > > ______________________________<wbr>_________________<br>
> > > > Xfce4-dev mailing list<br>
> > > > <a href="mailto:Xfce4-dev@xfce.org">Xfce4-dev@xfce.org</a><br>
> > > > <a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" rel="noreferrer" target="_blank">https://mail.xfce.org/mailman/<wbr>listinfo/xfce4-dev</a><br>
> > > ______________________________<wbr>_________________<br>
> > > Xfce4-dev mailing list<br>
> > > <a href="mailto:Xfce4-dev@xfce.org">Xfce4-dev@xfce.org</a><br>
> > > <a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" rel="noreferrer" target="_blank">https://mail.xfce.org/mailman/<wbr>listinfo/xfce4-dev</a><br>
> > ><br>
> > ><br>
> > > ______________________________<wbr>_________________<br>
> > > Xfce4-dev mailing list<br>
> > > <a href="mailto:Xfce4-dev@xfce.org">Xfce4-dev@xfce.org</a><br>
> > > <a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" rel="noreferrer" target="_blank">https://mail.xfce.org/mailman/<wbr>listinfo/xfce4-dev</a><br>
>  <br>
> ______________________________<wbr>_________________<br>
> Xfce4-dev mailing list<br>
> <a href="mailto:Xfce4-dev@xfce.org">Xfce4-dev@xfce.org</a><br>
> <a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" rel="noreferrer" target="_blank">https://mail.xfce.org/mailman/<wbr>listinfo/xfce4-dev</a><br>______________________________<wbr>_________________<br>
Xfce4-dev mailing list<br>
<a href="mailto:Xfce4-dev@xfce.org">Xfce4-dev@xfce.org</a><br>
<a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" rel="noreferrer" target="_blank">https://mail.xfce.org/mailman/<wbr>listinfo/xfce4-dev</a><br></blockquote></div><br></div></div>