<div dir="ltr">Hey Bastien,<div><br></div><div>On Wed, Mar 9, 2016 at 2:06 AM, Bastien Nocera <span dir="ltr"><<a href="mailto:hadess@hadess.net" target="_blank">hadess@hadess.net</a>></span> wrote:<br></div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">I commented on the xdg-user-dirs patches. It's mostly fine, but still<br>
has the same problem as the first set of patches, which is going to be<br>
about organising conflicts between applications.<br></blockquote><div><br></div><div>Thanks for the comments; I am a bit unclear how you would like to see the current patchset change to address that.</div><div>Could you give me an example? I thought I implemented the suggestion from your initial mail.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">A couple of things that I'd like before merging all this:<br>
- verify that transifex or another system is in place to update and be<br>
reactive to new translations<br></blockquote><div><br></div><div>I see translation commits in master, so I was assuming this worked already.</div><div>Is there anything in my patchset you think would result in a change in that regard?</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
- a test suite, verifying that files get moved properly, get renamed,<br>
etc. as expected.<br></blockquote><div><br></div><div>Definitely; this is something I wanted too and I will work on it next.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Other than that, I'm happy with the changes, even if the man pages are<br>
still on the short side to me.<br></blockquote><div><br></div><div>OK, I will try to expand that too.</div><div><br></div><div>Either way, since the patchset is pretty large as it is, I'd love to be able to merge the refactors/GLib port without the new user directories feature in the meantime. <br></div><div>Another thing that would greatly benefit the code is porting it to GIO. I initially didn't do that to reduce the churn and introduce a dependency on GObject, but in retrospect I don't see why not - practically speaking everyone shipping GLib already also ships GObject/GIO.</div><div><br></div><div>What do you think?</div><div><br></div><div>Thanks,</div><div>Cosimo</div></div></div></div>