<div dir="ltr"><div>So I did refresh the bulk-ocamlbuild-update PR on top of Louis' changes, and the result is indeed much nicer.<br><br>The updated PR is still at<br>  <a href="https://github.com/ocaml/opam-repository/pull/5140">https://github.com/ocaml/opam-repository/pull/5140</a><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 27, 2015 at 10:53 AM, Gabriel Scherer <span dir="ltr"><<a href="mailto:gabriel.scherer@gmail.com" target="_blank">gabriel.scherer@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Perfect, thanks! I plan to do the forward-porting and refresh my PR<br>
this week-end; if you haven't gotten to adding the quoting option by<br>
then, no worries, I'll just hack it as a temporary patch in my local<br>
branch.<br>
<div class="HOEnZb"><div class="h5"><br>
On Fri, Nov 27, 2015 at 10:46 AM, Louis Gesbert<br>
<<a href="mailto:louis.gesbert@ocamlpro.com">louis.gesbert@ocamlpro.com</a>> wrote:<br>
>> With this solution, you still lose comments on the modified fields<br>
>> (we've seen in the examples above that for examples there are comments<br>
>> inside the depext or depopt fields). The general guarantee you provide<br>
>> to users is still "best effort, comments may disappear".<br>
><br>
> Indeed.<br>
><br>
>> Do you plan to cherry-pick the format-preserving-printer on top of<br>
>> some 1.2 branch? I'm fine with porting my own script back to trunk,<br>
>> but I wonder whether I will have the same incompatible-formatting<br>
>> issues as last time (where those clear bugs that you fixed, or just a<br>
>> change in expected format?).<br>
><br>
> No, it's not reasonably doable since parsing/printing has been largely<br>
> rewritten and the whole format-preserving stuff is based on functions that<br>
> didn't exist.<br>
><br>
> The '"""' quoted strings are the only format update in the parser/printer, so<br>
> I'll add an option to disable their automatic use when printing and use that<br>
> in the rewriting scripts (it wasn't strictly required, but we wanted to be<br>
> able do include the package descriptions in the opam file, and it makes<br>
> escaping much less painful). With that fix you should be good on trunk.<br>
</div></div></blockquote></div><br></div>