labath added subscribers: jyknight, mehdi_amini.
labath added a comment.

In D65123#1596935 <https://reviews.llvm.org/D65123#1596935>, @teemperor wrote:

> Thanks for restoring this, but I'm a bit confused how this has happened. My 
> original diff <https://reviews.llvm.org/D64992?id=210828> didn't remove these 
> folders, but it seems when I committed the diff via arcanist and `git llvm` 
> it also removed these other folders. I assume `git llvm` is when translating 
> the change to SVN not just removing a folder but also all folders in the same 
> directory?


Interesting.. I wouldn't be totally surprised by that as git-llvm does some 
magic to svn check out only the files/directories that have been modified. 
Combining that with the different treatment of directories in svn and git 
(explicit modelling vs. implicit "it exists if it contains a file"), I can see 
how things might go south in some corner cases like this. But that's just 
speculation -- I'm generally trying to avoid git-llvm as it is too magical for 
my taste. Maybe @jyknight or @mehdi_amini can shed more light on this?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65123/new/

https://reviews.llvm.org/D65123



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to