Page MenuHomeFreeBSD

Add Linux/macOS compatibility system headers to tools/build/cross-build
ClosedPublic

Authored by arichardson on Feb 11 2018, 12:07 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 14, 10:46 PM
Unknown Object (File)
Oct 19 2024, 2:04 AM
Unknown Object (File)
Oct 14 2024, 12:45 AM
Unknown Object (File)
Oct 4 2024, 5:11 AM
Unknown Object (File)
Oct 3 2024, 8:40 PM
Unknown Object (File)
Oct 3 2024, 12:01 AM
Unknown Object (File)
Oct 2 2024, 4:26 PM
Unknown Object (File)
Oct 2 2024, 2:19 PM
Subscribers

Details

Summary

These headers are required in order to build the bootstrap tools on macOS
and Linux. A follow-up commit will add implementations of functions that
don't exist on those operating systems to -legacy when bootstrapping.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 32907
Build 30305: arc lint + arc unit

Event Timeline

rebased, should be ready for review now

arichardson retitled this revision from WIP: Add Linux/MacOS compatibility system headers to tools/build/cross-build to Add Linux/macOS compatibility system headers to tools/build/cross-build.Aug 6 2020, 6:42 PM
arichardson edited the summary of this revision. (Show Details)

A few whitespace and comment formatting nits (not every instance marked), but generally looks fine.

tools/build/cross-build/include/common/machine/atomic.h
45 ↗(On Diff #75514)

probably remove this line

tools/build/cross-build/include/linux/stdlib.h
62

I'd tend to pick one indent level for all the functions.

tools/build/cross-build/include/linux/sys/param.h
44

/* on separate line

tools/build/cross-build/include/linux/sys/stat.h
41

Too many *s

45

Probably use C-style comment.

tools/build/cross-build/include/mac/stdlib.h
45

should pick one indentation style.

This revision is now accepted and ready to land.Aug 6 2020, 7:44 PM
tools/build/cross-build/include/common/machine/atomic.h
45 ↗(On Diff #75514)

In fact I think this header is no longer needed. Will see if I can remove it.

arichardson marked 6 inline comments as done.
arichardson edited the summary of this revision. (Show Details)
  • fix C++ style commments
  • Use clang-format to get consistent style
This revision now requires review to proceed.Aug 6 2020, 8:38 PM
brooks added inline comments.
tools/build/cross-build/include/common/pwd.h
57

Normally return (1);

This revision is now accepted and ready to land.Aug 10 2020, 5:14 PM
  • Add $FreeBSD$ and missing parens around return statements
This revision now requires review to proceed.Aug 11 2020, 2:21 PM
This revision is now accepted and ready to land.Aug 11 2020, 8:42 PM