[PATCH] D83374: [analyzer][tests] Fix zip unpacking

2020-07-10 Thread Valeriy Savchenko via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG00997d1cad9e: [analyzer][tests] Fix zip unpacking (authored by vsavchenko). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83374/new/ https://reviews.llvm.or

[PATCH] D83374: [analyzer][tests] Fix zip unpacking

2020-07-09 Thread Valeriy Savchenko via Phabricator via cfe-commits
vsavchenko added a comment. 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 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST AC

[PATCH] D83374: [analyzer][tests] Fix zip unpacking

2020-07-08 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. > I think that the problem is actually with the second '/' I thought the OS always ignores those? Ok anyway fair enough! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://r

[PATCH] D83374: [analyzer][tests] Fix zip unpacking

2020-07-08 Thread Valeriy Savchenko via Phabricator via cfe-commits
vsavchenko added a comment. 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 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83374/new/ h

[PATCH] D83374: [analyzer][tests] Fix zip unpacking

2020-07-08 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. For my own education, what was wrong with `join`? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83374/new/ https://reviews.llvm.org/D83374 ___ cfe-commits mailing list cfe-commits@

[PATCH] D83374: [analyzer][tests] Fix zip unpacking

2020-07-08 Thread Valeriy Savchenko via Phabricator via cfe-commits
vsavchenko created this revision. vsavchenko added reviewers: NoQ, xazax.hun. Herald added subscribers: cfe-commits, ASDenysPetrov, Charusso, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware. Herald added a project: clang. Repository: rG LLVM Gith