[Lldb-commits] [PATCH] D76672: [lldb/Reproducers] Always collect the whole dSYM in the reproducer

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

In D76672#1939377 <https://reviews.llvm.org/D76672#1939377>, @JDevlieghere wrote:

> In D76672#1938629 <https://reviews.llvm.org/D76672#1938629>, @labath wrote:
>
> > It's not clear to me why this is needed.
> >
> > I mean, if lldb touches any of the files inside the dsym bundle, then they should be automatically recorded already, right? And if it doesn't then it does not need them...
>
>
> The dSYM can contain other resources than just the Mach-O companion file, such as script for the OS plugins or opt remarks, which might not be used by the reproducers or even LLDB at all. Once you have the reproducer on your system, tools will find and use it because spotlight indexed it. Having only a partial dSYM is really undesirable as it can break LLDB and other tools in really unexpected ways.


Interesting. Could that be fixed by adding the funny `.noindex` suffix to the reproducer name (or some subdirectory of it)?


Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D76672