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

Reply via email to