<p dir="ltr">I have a patch for that. I'm waiting for this to be queued before I send it.</p>
<div class="gmail_quote">On Mar 3, 2016 3:53 AM, "FUJIWARA Katsunori" <<a href="mailto:foozy@lares.dti.ne.jp">foozy@lares.dti.ne.jp</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">At Wed, 02 Mar 2016 15:22:20 -0800,<br>
<a href="mailto:danek.duvall@oracle.com">danek.duvall@oracle.com</a> wrote:<br>
><br>
> # HG changeset patch<br>
> # User Danek Duvall <<a href="mailto:danek.duvall@oracle.com">danek.duvall@oracle.com</a>><br>
> # Date 1456959037 28800<br>
> #      Wed Mar 02 14:50:37 2016 -0800<br>
> # Node ID 47d43135bf430d7172c8fb8fe35fb17b6f8f029c<br>
> # Parent  c7f89ad87baef87f00c507545dfd4cc824bc3131<br>
> tests: Solaris cp doesn't support the -T option<br>
><br>
> The treemanifest tests use the -T option to cp in order to ensure that the<br>
> two directories named on the commandline are treated as peers, rather than<br>
> the usual behavior when the final argument is a directory.  GNU cp has this<br>
> option, but other implementations may not.  Thankfully, there's no pressing<br>
> reason to use it.  We can simply copy the contents of the first directory<br>
> into the target directory, since we know that the target directory already<br>
> exists.<br>
<br>
I confirmed that #1, #2 and #4 fix test portability problem on<br>
Solaris.<br>
<br>
To avoid re-raising "cp -T" problem in the future, it would be<br>
desirable to add new detection rule for usage of "cp -T" to<br>
contrib/check-code.py.<br>
<br>
> diff --git a/tests/test-treemanifest.t b/tests/test-treemanifest.t<br>
> --- a/tests/test-treemanifest.t<br>
> +++ b/tests/test-treemanifest.t<br>
> @@ -541,7 +541,7 @@ Verify reports missing dirlog<br>
>    8 integrity errors encountered!<br>
>    (first damaged changeset appears to be 0)<br>
>    [1]<br>
> -  $ cp -rT .hg/store-newcopy .hg/store<br>
> +  $ cp -r .hg/store-newcopy/* .hg/store<br>
><br>
>  Verify reports missing dirlog entry<br>
>    $ mv -f .hg/store-copy/meta/b/00manifest.* .hg/store/meta/b/<br>
> @@ -566,7 +566,7 @@ Verify reports missing dirlog entry<br>
>    8 integrity errors encountered!<br>
>    (first damaged changeset appears to be 1)<br>
>    [1]<br>
> -  $ cp -rT .hg/store-newcopy .hg/store<br>
> +  $ cp -r .hg/store-newcopy/* .hg/store<br>
><br>
>  Test cloning a treemanifest repo over http.<br>
>    $ hg serve -p $HGPORT -d --pid-file=hg.pid --errorlog=errors.log<br>
> _______________________________________________<br>
> Mercurial-devel mailing list<br>
> <a href="mailto:Mercurial-devel@mercurial-scm.org">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>
<br>
----------------------------------------------------------------------<br>
[FUJIWARA Katsunori]                             <a href="mailto:foozy@lares.dti.ne.jp">foozy@lares.dti.ne.jp</a><br>
_______________________________________________<br>
Mercurial-devel mailing list<br>
<a href="mailto:Mercurial-devel@mercurial-scm.org">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>