[PATCH] D139980: [lld-macho][test][nfc] Update stabs.s to use touch -d instead of -t

Haowei Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 14 10:26:09 PST 2022


haowei added a comment.

In D139980#3995526 <https://reviews.llvm.org/D139980#3995526>, @pzheng wrote:

> Thanks for acting upon this quickly, @thakis!
>
> Sorry for the inconveniences caused by this change, @haowei. Do you mind trying the other form of the command (`touch -d "1970-01-01T00:00:16Z"`) on your builder and see if that works? I am hoping that we can have a simple workaround for the container issue I encountered without breaking any other stuff of course.

I don't have ssh access on builder bots, unfortunately. If you look at the error output, `-d` flag looks like is not supported at all on Big Sur. In that case, using `-d` will break LLVM build on all Big Sur or older Mac OS and it would be undesirable.

If this is the only test your container has trouble has. How about using `LIT_FILTER_OUT` env arg to bypass this test in your container?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139980



More information about the llvm-commits mailing list