Gotcha. Mea culpa. Sorry for making incorrect assumptions.
We reached synchronization just in time for me to fix it.
-- Jon
On 2/28/20 12:04 PM, Pavel Rappo wrote:
Jon, you might be confusing this directory with a similarly named
*file* TestExternalOverridenMethod.java, which you indeed renamed.
The directory I'm talking about is called "testExternalOverridenMethod",
and it is still there:
http://hg.openjdk.java.net/jdk/jdk/file/52367bbd4bd4/test/langtools/jdk/javadoc/doclet/testExternalOverridenMethod
Kindly correct me if I'm mistaken. Today is Friday,
and I may see all sorts of things which are not there.
-Pavel
On 28 Feb 2020, at 19:50, Jonathan Gibbons <[email protected]> wrote:
Mercurial doesn't clean up empty directories, nor can I do anything to the main
repo
to make the empty directory in your repo go away.
`rmdir` is your friend.
-- Jon
On 2/28/20 11:47 AM, Pavel Rappo wrote:
That's fine. However, there's still a *directory*, called
"testExternalOverridenMethod"
$ find . -iname "*overriden*"
./test/langtools/jdk/javadoc/doclet/testExternalOverridenMethod
On 28 Feb 2020, at 19:00, Jonathan Gibbons <[email protected]> wrote:
On 02/28/2020 07:25 AM, Pavel Rappo wrote:
If it is still possible, could you please fix the spelling in the name of the
"testExternalOverridenMethod" folder? I think we missed that yesterday while
reviewing (now pushed)http://hg.openjdk.java.net/jdk/jdk/rev/52367bbd4bd4.
Pavel,
That patch has the rename in it. There are no such badly-named files any more:
$ find open/test/langtools/ -name \*riden\*.java
$
-- Jon
