[PATCH] D44342: Introduce a setting to disable Spotlight while running the test suite

Jim Ingham via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 12 15:34:19 PDT 2018


jingham added a comment.

This is great.  I think I'm supposed to complain that there isn't a positive test that this flag works.  This is in the testsuite, but that will just give a negative signal - that we aren't getting random failures that are maybe due to leaving built test products around on your indexed volumes somewhere.

If we weren't building in a .noindex hierarchy this would be as simple as building the dSYM variant, then moving the dSYM to a subdirectory and making sure we still don't read in the dSYM.  But that won't work in a .noindex build.  Not sure we should be copying files to people's desktops.  /tmp/ won't work because Spotlight doesn't index there.  We talked briefly about having an indexed directory for test products as well as a no index one, which you could then use for that purpose.

But I'm not sure that this switch is worth that effort.  This is fine by me as is.


Repository:
  rL LLVM

https://reviews.llvm.org/D44342





More information about the llvm-commits mailing list