diff options
author | Raphael Isemann <teemperor@gmail.com> | 2019-03-12 07:40:54 +0000 |
---|---|---|
committer | Raphael Isemann <teemperor@gmail.com> | 2019-03-12 07:40:54 +0000 |
commit | 21347b29013b6b6a97d1d800add7c5b54a657fb7 (patch) | |
tree | 9d4f168afe157cbce950ea9fc98bcb0635c2c5d2 /lldb/packages/Python/lldbsuite/test/expression_command/inline-namespace/main.cpp | |
parent | 5b1e1c0537d0e896ae1c0c5c2ff80ed68b397a80 (diff) | |
download | bcm5719-llvm-21347b29013b6b6a97d1d800add7c5b54a657fb7.tar.gz bcm5719-llvm-21347b29013b6b6a97d1d800add7c5b54a657fb7.zip |
Fix git-llvm crashing when trying to remove directory while cleaning
Summary:
I'm trying to push D59198 but it seems that `git-llvm push` can't handle the fact
that I add a new directory in the patch:
```
> git llvm push -n
Pushing 1 commit:
e7c0a9bd136 Correctly look up declarations in inline namespaces
Traceback (most recent call last):
File "llvm/utils/git-svn//git-llvm", line 431, in <module>
args.func(args)
File "llvm/utils/git-svn//git-llvm", line 385, in cmd_push
clean_svn(svn_root)
File "llvm/utils/git-svn//git-llvm", line 201, in clean_svn
os.remove(os.path.join(svn_repo, filename))
IsADirectoryError: [Errno 21] Is a directory: '.git/llvm-upstream-svn/lldb/trunk/packages/Python/lldbsuite/test/expression_command/inline-namespace'
```
This patch just uses shutil to delete the directory instead of trying to use `os.remove`
which only works for files.
Reviewers: mehdi_amini, jlebar
Reviewed By: jlebar
Subscribers: llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D59236
llvm-svn: 355896
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/expression_command/inline-namespace/main.cpp')
0 files changed, 0 insertions, 0 deletions