<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Sep 4, 2014 at 10:33 AM, Satish Balay <span dir="ltr"><<a href="mailto:balay@mcs.anl.gov" target="_blank">balay@mcs.anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Thu, 4 Sep 2014, Jed Brown wrote:<br>
<br>
> Satish Balay <<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>> writes:<br>
> > And when folks hardly look at git messages - I don't thinks they will notice<br>
> > this message burried in middle of a build. [And most of the time merge/push<br>
> > is done without a build].<br>
><br>
> That would be naughty.<br>
<br>
</div>sorry - I should have said something like:<br>
<br>
The workflow is:<br>
<br>
(1). verify what you are merging<br>
git checkout master<br>
git log ..branch<br>
git diff ..branch<br>
(2). merge the branch<br>
git merge<br>
(3). Build and make sure it builds<br>
./configure && make all test<br>
(4). Now push to remote<br>
git push<br>
<br>
So the issue comes up when:<br>
<br>
(1) verify is not done and (2) merge messages are not read.<br>
<br>
And the fix relies on doing (3) build & reading messages.<br>
<br>
To me this is not really a fix [i.e fix for skipping a step to do<br>
another step. fix for not reading a message to to read another<br>
message]<br>
<br>
Obviously everyone wants (1) and (3) to be automated - but we<br>
currently don't have that..</blockquote><div><br></div><div>I, of course, disagree. (1) is now automated in that you get a warning</div><div>if you try to merge next. That will help me. Please avert your eyes.</div><div>
<br></div><div>   Matt</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888"><br>
Satish<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>
-- Norbert Wiener
</div></div>