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

Pengxuan Zheng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 14 10:30:15 PST 2022


pzheng added a comment.

In D139980#3995615 <https://reviews.llvm.org/D139980#3995615>, @haowei wrote:

> 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?

I see. Thanks for helping, @haowei! Looks like I'll have to think about other alternatives. :)


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