2011/10/24 Carter, Eli <span dir="ltr"><<a href="mailto:Eli.Carter@tektronix.com">Eli.Carter@tektronix.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;">
Greg,<br>
<br>
I disagree with these changes.  We need to test that the memory warning message is not displayed in the first case.  Your change of filesize eliminates that check.<br>
<br>
Also, this changes the intent of the "Commit another file that should get automatically added as a largefile" test.  The point of that test was to test that a file is automatically added as a largefile _without_ any options on the commandline for largefiles.  That does mean a 10MB file is required to test that.<br>
</blockquote><div><br>FWIW, I agree with Eli here.  I'm not sure I understand the reason to make the files smaller because now we're not testing what was supposed to be tested.<br><br>Cheers,<br>Na'Tosha<br> <br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
> -----Original Message-----<br>
> From: <a href="mailto:mercurial-devel-bounces@selenic.com">mercurial-devel-bounces@selenic.com</a> [mailto:<a href="mailto:mercurial-devel-">mercurial-devel-</a><br>
> <a href="mailto:bounces@selenic.com">bounces@selenic.com</a>] On Behalf Of Greg Ward<br>
> Sent: Thursday, October 20, 2011 9:25 PM<br>
> To: <a href="mailto:mercurial-devel@selenic.com">mercurial-devel@selenic.com</a><br>
> Subject: [PATCH 3 of 5] largefiles: use smaller files in test script<br>
<div class="im">><br>
> # HG changeset patch<br>
> # User Greg Ward <<a href="mailto:greg@gerg.ca">greg@gerg.ca</a>><br>
> # Date 1319163864 14400<br>
> # Branch stable<br>
> # Node ID 70cc4a2b4f72b14905977b65e239b5ee12efa861<br>
> # Parent  2342ce10ae6ce8dc66c9f43aa10cba75c74c700a<br>
> largefiles: use smaller files in test script<br>
><br>
> diff --git a/tests/test-largefiles.t b/tests/test-largefiles.t<br>
> --- a/tests/test-largefiles.t<br>
> +++ b/tests/test-largefiles.t<br>
> @@ -193,7 +193,7 @@<br>
>  Config settings (pattern **.dat, minsize 2 MB) are respected.<br>
><br>
>    $ echo testdata > test.dat<br>
> -  $ dd bs=3145728 count=1 if=/dev/zero of=reallylarge > /dev/null 2><br>
> /dev/null<br>
> +  $ dd bs=1k count=2k if=/dev/zero of=reallylarge > /dev/null 2><br>
</div>> + /dev/null<br>
<div><div></div><div class="h5">>    $ hg add<br>
>    adding reallylarge as a largefile<br>
>    adding test.dat as a largefile<br>
> @@ -677,16 +677,14 @@<br>
>  "lfconvert" works<br>
>    $ hg init bigfile-repo<br>
>    $ cd bigfile-repo<br>
> -  $ dd if=/dev/zero bs=1k count=23k > a-large-file 2> /dev/null<br>
> +  $ dd if=/dev/zero bs=1k count=256 > a-large-file 2> /dev/null<br>
>    $ hg addremove<br>
>    adding a-large-file<br>
> -  a-large-file: up to 72 MB of RAM may be required to manage this file<br>
> -  (use 'hg revert a-large-file' to cancel the pending addition)<br>
> -  $ hg commit -m "Commit file without making it be a largefile"<br>
> +  $ hg commit -m "add a-large-file (as a normal file)"<br>
>    $ find .hg/largefiles<br>
>    .hg/largefiles<br>
>    $ cd ..<br>
> -  $ hg lfconvert --size 10 bigfile-repo largefiles-repo<br>
> +  $ hg lfconvert --size 0.2 bigfile-repo largefiles-repo<br>
>    initializing destination largefiles-repo<br>
><br>
>  "lfconvert" adds 'largefiles' to .hg/requires.<br>
> @@ -704,13 +702,13 @@<br>
>    getting changed largefiles<br>
>    1 largefiles updated, 0 removed<br>
>    $ cat .hglf/a-large-file<br>
> -  8b0629c630f530cde051aeb42ce561756738fbe7<br>
> -  $ dd if=/dev/zero bs=1k count=11k > another-large-file 2> /dev/null<br>
> -  $ hg add another-large-file<br>
> -  $ hg commit -m "Commit another file that should get automatically added<br>
> as a largefile"<br>
> +  2e000fa7e85759c7f4c254d4d9c33ef481e459a7<br>
</div></div>> +  $ dd if=/dev/zero bs=1k count=1k > another-large-file 2> /dev/null  $<br>
> + hg add --lfsize=1 another-large-file  $ hg commit -m "add<br>
<div><div></div><div class="h5">> + another-large-file (should be a largefile)"<br>
>    $ cat .hglf/a-large-file .hglf/another-large-file<br>
> -  8b0629c630f530cde051aeb42ce561756738fbe7<br>
> -  187a0f76e02aac9c24f71c820be1f34ef1c76e76<br>
> +  2e000fa7e85759c7f4c254d4d9c33ef481e459a7<br>
> +  3b71f43ff30f4b15b5cd85dd9e95ebc7e84eb5a3<br>
>    $ cd ..<br>
>    $ rm -rf bigfile-repo largefiles-repo<br>
><br>
> _______________________________________________<br>
> Mercurial-devel mailing list<br>
> <a href="mailto:Mercurial-devel@selenic.com">Mercurial-devel@selenic.com</a><br>
> <a href="http://selenic.com/mailman/listinfo/mercurial-devel" target="_blank">http://selenic.com/mailman/listinfo/mercurial-devel</a><br>
<br>
<br>
_______________________________________________<br>
Mercurial-devel mailing list<br>
<a href="mailto:Mercurial-devel@selenic.com">Mercurial-devel@selenic.com</a><br>
<a href="http://selenic.com/mailman/listinfo/mercurial-devel" target="_blank">http://selenic.com/mailman/listinfo/mercurial-devel</a><br>
</div></div></blockquote></div><br><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>