2011/10/28 Dominik Psenner <span dir="ltr"><<a href="mailto:dpsenner@gmail.com">dpsenner@gmail.com</a>></span><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">>this also signifies that we need largefiles<br>
<br>
</div>Actually: no. You don't require the largefiles extension until you actually<br>
have .hglf in the store.<br>
<div class="im"><br>
> and is faster than walking the store -- epsecially if the repo is large.<br>
<br>
</div>Yes, it would be. But again, it would add the largefiles requirement even if<br>
it is not required yet. I.e.:<br>
<br>
$ touch .hglf<br>
$ hg st<br>
$ hg revert<br>
$ cat .hg/requirements<br>
largefiles<br>
**OPS**<br>
<div class="im"><br>
>It won't help much in the I-have-largefiles-enabled-but-don't-use-<br>
>them case<br>
<br>
</div>Instead it does. It checks the first level of files and folders in the store<br>
(which is normally a quite small number of files and folders) and adds the<br>
requirement once it finds .hglf there. We could discuss if we should not<br>
walk the files and blindly guess the location of .hglf in the store, but as<br>
said I was afraid from going that far. :-)<br>
<div class="im"><br>
> but it will help in the "I-do-have-largefiles-enabled-and-I-do-<br>
>use-them" case.<br>
<br>
</div>In this case largefiles are already in the requirements and we don't have to<br>
add the requirement anymore.<br></blockquote><div><br>I have a few comments here, but I'm content to let this go until I actually do some more profiling with largefiles.  It's possible Largefiles behaves differently than Kbfiles in this regard, so I'd better do some more research before arguing my point more (or otherwise risk putting my own foot in my mouth).<br>
<br>But in any case, I do think your patch should go in (even if I think it can be extended a bit, and probably make some other use cases faster).<br><br>Cheers,<br>Na'Tosha<br></div></div><br clear="all"><br>-- <br><div>
<div><span style="color: rgb(153, 153, 153);"><b>Na'Tosha Bard</b></span></div><div><font color="#999999">Build & Infrastructure Developer | Unity Technologies</font></div><div><font color="#999999"><br></font></div>
<div><font color="#999999"><b>E-Mail:</b> <a href="mailto:natosha@unity3d.com" target="_blank">natosha@unity3d.com</a></font></div><div><font color="#999999"><b>Skype:</b> natosha.bard</font></div></div><br>