<div dir="ltr"><div><div>Thanks for all the feedback guys<br><br></div>So rather than speculating about what could and might not break.<br></div><div><div><div><br></div></div></div><div>I am going to test all these different scenarios by creating my own svn server, svn mirror and git mirror etc</div><div>I'll get back to you all with details on what breaks and what doesn't soon with changing the commit msgs</div><div><br></div><div>Many Thanks</div><div>Martell</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 29, 2015 at 7:01 AM, Sean Silva <span dir="ltr"><<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>On Tue, Jul 28, 2015 at 10:42 AM, Martell Malone <span dir="ltr"><<a href="mailto:martellmalone@gmail.com" target="_blank">martellmalone@gmail.com</a>></span> wrote:<br></span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><span><div><div>Hi<br><br>I'm trying to change a commit message that became malformed when pushing to trunk<br><br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">svn propedit svn:log --revprop -r 243434<br>svn: E175002: DAV request failed; it's possible that the repository's pre-revprop-change hook either failed or is non-existent<br>svn: E175002: Repository has not been enabled to accept revision propchanges;<br>ask the administrator to create a pre-revprop-change hook<br></blockquote><br></div>As described here commit messages are not versioned on svn<br></div>Would it be possible to get a hook made for this<br></span><a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__stackoverflow.com_questions_692851_can-2Di-2Dgo-2Dback-2Dand-2Dedit-2Dcomments-2Don-2Dan-2Dsvn-2Dcheckin&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=Mfk2qtn1LTDThVkh6-oGglNfMADXfJdty4_bhmuhMHA&m=rA66OwhKyNgAo35v-7-yD5YmsmgWkfGpODyqFgA-Sug&s=H-gYErVjtF74kkPHv-DrcBVKhP0r9SkTIbUmUyHW0u0&e=" target="_blank">http://stackoverflow.com/questions/692851/can-i-go-back-and-edit-comments-on-an-svn-checkin</a><br><br></div><span>It could be very useful for 3 reason.<br><br>1.<br></span></div><div><span>Currently if someone makes a commit and does not have it is not automatically closed by phabracator by adding<br></span>Differential Revision: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D11511&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=Mfk2qtn1LTDThVkh6-oGglNfMADXfJdty4_bhmuhMHA&m=rA66OwhKyNgAo35v-7-yD5YmsmgWkfGpODyqFgA-Sug&s=MO-ifc5cGb5v_ThESxcVSdiZCzBphnRq2W57Z6Wgd1M&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/DXXXXX</a><br></div><span><div>at the end of their commit<br></div><div>The committer then often has to go back and manually enter a message to tell someone that their patch has landed<br><br></div><div>With a hook for this it would be easy to update the message invoking phabracator to now close the revision as it has landed.<br><br></div><div>we could even quite easily create a script for this for any committer to use to speed up the process, example here is a script called landed<br><br></div><div>landed D11511 r243434</div></span></div></blockquote><div><br></div><div>Probably easier to have Phab just monitor the post-commit review thread on the mailing list.</div><span><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br>2.<br></div><div>If a patch is reverted at some point because it is broken.<br></div><div>It would be nice to be able to edit the commit message in rXXX to tell the reader that the commit was reverted in rYYY and reapplied with a fix in rZZZ<br></div></div></blockquote><div><br></div></span><div>Usually we chime in on the post-commit thread saying "btw, reverted in rXXXXXX".</div><span><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br>3.<br></div><div>If you have a pesky typo in your commit like I have :)<br></div></div></blockquote><div><br></div></span><div>I don't think anybody really cares very much about typos in the commit history, as long as they are clarified in the post-commit thread (just like if the code has a problem or whatever).</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><div dir="ltr"><div><br></div><div>I'm sure the hook could be made so that only an author can edit their own commit message or an admin that can.<br><br></div><div>Kind Regards<span><font color="#888888"><br></font></span></div><span><font color="#888888"><div>Martell<br></div><div><br><br></div></font></span></div>
<br></span><span>_______________________________________________<br>
LLVM Developers mailing list<br>
</span><a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" rel="noreferrer" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
<br></blockquote></div><br></div></div>
</blockquote></div><br></div></div>