Page MenuHomeFreeBSD

tools/build: Don't redefine open() for the linux bootstrap
ClosedPublic

Authored by arichardson on Jul 20 2021, 12:25 AM.

Details

Summary

This is needed to bootstrap llvm-tblgen on Linux since LLVM calls
::open(...) which does not work if open is a statement macro.
Also stop defining O_SHLOCK/O_EXLOCK and update the only bootstrap tools
user of those flags to deal with missing definitions.

Split out from D31057

Diff Detail

Repository
R10 FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

lib/libc/db/db/db.c
68

You missed dropping these changes

tools/build/cross-build/include/linux/fcntl.h
46–49

How come this is going away?

tools/build/cross-build/include/linux/fcntl.h
46–49

That should have been removed as part of the initial commit since it's in __unused_workaround_end.h. Forgot to remove that one when I moved it into a separate header.

This revision is now accepted and ready to land.Jul 20 2021, 7:28 PM