This is an archive of the discontinued LLVM Phabricator instance.

[analyzer][tests] Fix zip unpacking
ClosedPublic

Authored by vsavchenko on Jul 8 2020, 1:53 AM.

Diff Detail

Event Timeline

vsavchenko created this revision.Jul 8 2020, 1:53 AM
NoQ added a comment.Jul 8 2020, 9:34 AM

For my own education, what was wrong with join?

I think that the problem is actually with the second '/', but I decided to retreat back to the form consistent to the rest glob.glob invocations in this file

NoQ accepted this revision.Jul 8 2020, 10:10 AM

I think that the problem is actually with the second '/'

I thought the OS always ignores those? Ok anyway fair enough!

This revision is now accepted and ready to land.Jul 8 2020, 10:10 AM
In D83374#2139505, @NoQ wrote:

I thought the OS always ignores those? Ok anyway fair enough!

I thought so too, but it looks like glob works in mysterious ways

This revision was automatically updated to reflect the committed changes.