[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
Tue Dec 13 21:55:33 PST 2022


haowei added a comment.

This change breaks `lld/test/MachO/stabs.s` on Fuchsia's Mac x64 Clang builder:

Failure message:

  --
  Exit Code: 1
  
  Command Output (stderr):
  --
  touch: illegal option -- d
  usage:
  touch [-A [-][[hh]mm]SS] [-acfhm] [-r file] [-t [[CC]YY]MMDDhhmm[.SS]] file ...
  
  --

Failed builder task: https://ci.chromium.org/ui/p/fuchsia/builders/toolchain.ci/clang-mac-x64/b8794839433510130609/overview

Looks like -d flag is not supported by the `touch` under Mac Big Sur, which is used by the builder above.
I tried the same command on my Ventura Mac and it still fails:

  ➜  ~ touch -d "1970-01-01 00:00:16 UTC" a
  touch: out of range or illegal time specification: YYYY-MM-DDThh:mm:SS[.frac][tz]
  ➜  ~

Due to this inconsistency under different mac os versions, I think we should avoid use `-d` flag. Could you revert this change first while trying a different approach please?


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