Page MenuHomeFreeBSD

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

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

Details

Reviewers
jrtc27
dim
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
rS FreeBSD src repository - subversion
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 40545
Build 37434: arc lint + arc unit

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.Tue, Jul 20, 7:28 PM