<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 6, 2013 at 8:15 PM, Sean Farley <span dir="ltr"><<a href="mailto:sean@mcs.anl.gov" target="_blank">sean@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 id=":1bp">No, I got that. I, and no other dev, can reproduce. What I was implying<br>
is an error between the chair and the keyboard. Perhaps you didn't<br>
realize you did a rebase or some other such human mistake.<br>
<div><div class="im"><br>
> $ hg log -vr de73c9a7d341d846b5e16a8d61a48  |grep fsolvebaij.h<br></div></div></div></blockquote><div><br></div><div style>If you say you cannot reproduce, please show the output of the above command.</div><div style>
<br></div><div style>It's obvious that it names files with zero changes in the web interface (see attached pic). Bear in mind that Barry made this commit. Barry uses vanilla Hg so it was produced by Hg core.</div><div style>
 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":1bp"><div><div class="im">
> files:       src/contrib/style/checks/PetscFunctionBegin.sh<br>
> src/contrib/style/checks/PetscFunctionBegin2.sh<br>
> src/contrib/style/checks/PetscFunctionReturn.sh<br>
> src/dm/impls/composite/pack.c src/dm/impls/plex/f90-custom/zplexf90.c<br>
> src/dm/impls/plex/ftn-custom/zplex.c src/dm/impls/plex/plexcreate.c<br>
> src/ksp/pc/impls/bjacobi/bjacobi.c src/ksp/pc/impls/fieldsplit/fieldsplit.c<br>
> src/ksp/pc/impls/gamg/gamg.c src/ksp/pc/impls/openmp/hpc.c<br>
> src/mat/impls/aij/mpi/mpiaij.c src/mat/impls/aij/mpi/mpiaij.h<br>
> src/mat/impls/aij/seq/aij.h<br>
> src/mat/impls/aij/seq/crl/ftn-kernels/fmultcrl.h<br>
> src/mat/impls/aij/seq/ftn-kernels/fmult.h<br>
> src/mat/impls/aij/seq/ftn-kernels/fmultadd.h<br>
> src/mat/impls/aij/seq/ftn-kernels/frelax.h<br>
> src/mat/impls/aij/seq/ftn-kernels/fsolve.h src/mat/impls/baij/seq/baij.h<br>
> src/mat/impls/baij/seq/ftn-kernels/fsolvebaij.h<br>
> src/mat/impls/dense/seq/dense.h src/mat/impls/fft/fft.h<br>
> src/mat/impls/maij/maij.h src/mat/impls/mffd/mffd.c<br>
> src/mat/impls/sbaij/seq/sbaij.h src/mat/utils/zerodiag.c<br>
> src/snes/mf/snesmfj.c src/vec/vec/impls/dvecimpl.h<br>
> src/vec/vec/impls/seq/seqcusp/cuspvecimpl.h<br>
> $ hg export de73c9a7d341d846b5e16a8d61a48  |grep fsolvebaij.h<br>
> $<br>
><br>
> So fsolvebaij.h (and 9 other files) are not affected by the commit, but are<br>
> named anyway.<br>
><br>
> Recall that this is a commit that Barry made on his own, before I had<br>
> anything to do with that branch. There was no rebase and no gitifyhg<br>
> involved. If you strip that commit and 'hg import' what you just exported,<br>
> you will get a different hash because those files that are named in the<br>
> commit, but not affected by the commit, are no longer present.<br>
><br>
> Please show me in the hg specification for commits where it is allowed for<br>
> a commit to name files that are unrelated to the commit and do not appear<br>
> in 'hg export', but whose file names are part of the SHA1.<br>
<br>
</div></div>Since I cannot reproduce this, I am skeptical of this being pure<br>
mercurial. I posit that you, perhaps unknowingly, infected the tree with<br>
gitifyhg (or rebase?).<br>
<br>
We could be in some corner case of case-collision with a<br>
case-insensitive file system but we'd need a pure mercurial test case.</div></blockquote></div><br>This *is* pure mercurial. All relevant commits were made by Barry. (He is not to blame, but I want to make it exceedingly clear that I was not involved in the creation of this "bad" commit.)</div>
</div>