<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Apr 7, 2019 at 5:50 PM Jordi Gutiérrez Hermoso <<a href="mailto:jordigh@octave.org">jordigh@octave.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sun, 2019-04-07 at 09:48 +0900, Yuya Nishihara wrote:<br>
> So they are quite similar to -r 'file("set:copied()")', and we do include<br>
> renames in copies in general.<br>
<br>
Hm, but we don't have a renamed() fileset, do we? How do you find renames?<br></blockquote><div><br></div><div>Yes, I don't think there is one.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> I think these predicates are useful, but it's confusing to handle copies<br>
> and renames separately only in revset.<br>
<br>
I agree, consistency is important, but I don't know how to implement a<br>
fileset or revset for finding renames.<br></blockquote><div><br></div><div>Hmm, this patch has a revset for finding renames... :)</div><div><br></div><div>Anyway, I agree with Yuya that it will be confusing if the copied() fileset includes copies and the copies() revset doesn't. Perhaps we can add arguments to both of them? Something like copied(renames=True,purecopies=True)? I don't think you need to implement such support for the copied() fileset, but I'd like to have a plan for how to eventually make them consistent.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
_______________________________________________<br>
Mercurial-devel mailing list<br>
<a href="mailto:Mercurial-devel@mercurial-scm.org" target="_blank">Mercurial-devel@mercurial-scm.org</a><br>
<a href="https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel" rel="noreferrer" target="_blank">https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel</a><br>
</blockquote></div></div>