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