On Fri, Mar 10, 2023 at 12:50:09PM +0100, Stefan Sperling wrote:
> I have tested the above FreeBSD scenario with my patch in place.
> 
> Updating across the type change of __string from file to directory works.
> 
> When updating backwards, __string changes from a directory to a file
> and the update operation reports an obstruction:
>     ~  contrib/llvm-project/libcxx/include/__string
> However the directory is removed anyway, and if 'got update' is run again
> it will restore the now missing file __string.
> 
> I guess this is good enough, for now. At least there are no crashes or
> hard errors. I will add a test case to cover the second case, using the
> above behaviour as expected behaviour. We can make more improvements later.

Because I would like to release soon I've gone ahead and committed my patch
plus the new test case. If there are any issues please let me know.