Hi -

> >     With PR25365, we accidentally lost the ability to rmdir client-cache
> >     directories corresponding to buildids.  Bring this back, with some
> >     attention to a possible race between a client doing cleanup and
> >     another client doing lookups at the same time.
> 
> OK. So if I understand correctly the trailing / in the regexp made
> directories not match (because their name doesn't actually end in a /)?

Even worse, the previous regexp had the parenthesized list mandatory,
so a directory name (even if it were to end in /) would not have
matched.
 
      ".*/[a-f0-9]+/(debuginfo|executable|source.*)$"

- FChE

Reply via email to