Page MenuHomeFreeBSD
Differential D16767

Add a tools/build/make.py script that bootstraps bmake and then runs the build
ClosedPublic
Actions

Authored by arichardson on Aug 17 2018, 1:59 PM.
Tags
None
Referenced Files
F83403197: D16767.id75557.diff
Fri, May 10, 1:00 AM
F83403196: D16767.id75739.diff
Fri, May 10, 1:00 AM
Unknown Object (File)
Thu, May 9, 11:10 AM
Unknown Object (File)
Tue, May 7, 10:45 PM
Unknown Object (File)
Mon, May 6, 1:00 AM
Unknown Object (File)
Fri, May 3, 1:25 AM
Unknown Object (File)
Fri, May 3, 1:25 AM
Unknown Object (File)
Fri, May 3, 1:25 AM
View All 87 Files
Subscribers
None

Details

Reviewers
emaste
jhb
imp
bdrewery
bwidawsk
Commits
rS365949: Add a tools/build/make.py script that bootstraps bmake and then runs the build
Summary

This makes it possible to compile on non-FreeBSD systems since make will
usually be GNU make there. Even if they include bmake, it will often
either be a broken version or too old to build FreeBSD.

Test Plan

With this script I've been able to compile on Linux+MacOS (once I've applied
all the other patches from https://github.com/arichardson/cheribsd/tree/crossbuild-aug2018)

Diff Detail

Lint
Lint Errors
SeverityLocationCodeMessage
Errortools/build/make.py:38E501PEP8 E501
Errortools/build/make.py:153E501PEP8 E501
Errortools/build/make.py:160E501PEP8 E501
Unit
No Test Coverage
Build Status
Buildable 18892
Build 18546: arc lint + arc unit

Event Timeline

arichardson created this revision.Aug 17 2018, 1:59 PM
Harbormaster completed remote builds in B18892: Diff 46835.Aug 17 2018, 1:59 PM
jhb added a comment.Aug 20 2018, 10:34 AM

Probably best to save this commit for last as once you commit it, people will try it. The change generally works fine for me.

tools/build/make.py
153

We should fix bsd.compiler.mk if it isn't too gross. I recently changed it to handle GCC from base/gcc installed as /usr/bin/cc in rS333414.

bwidawsk added a comment.Oct 31 2018, 9:54 PM

I've tested the latest in github today on Arch Linux and was able to build world and kernel - however at this moment, I haven't been able to test what has been built anywhere.

arichardson updated this revision to Diff 75557.Aug 7 2020, 1:21 PM

update to latest version

Harbormaster completed remote builds in B32838: Diff 75557.Aug 7 2020, 1:21 PM
arichardson added a parent revision: D25992: Add necessary Makefile.inc1 infrastructure for building on non-FreeBSD.Aug 7 2020, 3:28 PM
arichardson removed a parent revision: D25992: Add necessary Makefile.inc1 infrastructure for building on non-FreeBSD.
arichardson added parent revisions: D25992: Add necessary Makefile.inc1 infrastructure for building on non-FreeBSD, D25990: Fix crunchgen usage of mkstemp(), D25989: Allow running lorder.sh on Linux, D25979: Fix cddl tools bootstrapping on macOS and Linux, D25978: Add missing FreeBSD functions to -legacy when building on macOS/Linux, D25928: Fix makefs bootstrap on macOS after D25563, D14318: Make bsd.linker.mk work with the MacOS linker, D14316: Add Linux/macOS compatibility system headers to tools/build/cross-build, D14245: Add CLANG/LLD/LLD to BROKEN_OPTIONS when building on non-FreeBSD.
arichardson updated this revision to Diff 75739.Aug 12 2020, 5:44 PM

Pass -fuse-ld=$XLD when building with clang

Harbormaster completed remote builds in B32947: Diff 75739.Aug 12 2020, 5:44 PM
emaste added a comment.Aug 13 2020, 12:51 PM

I gave this a cursory glance and it seems broadly reasonable. I think it's fine to commit to tools/.

arichardson added a comment.Aug 13 2020, 2:10 PM

I'll commit this once all the dependent patches have been committed.

arichardson edited parent revisions, added: D26055: Pass -fuse-ld=/path/to/ld if ${LD} != "ld", D26044: Stop using lorder and ranlib when building libraries; removed: D25989: Allow running lorder.sh on Linux.Aug 13 2020, 2:11 PM
arichardson updated this revision to Diff 75787.Aug 13 2020, 2:19 PM

Stop setting -fuse-ld=, this should be handled by D26055

Harbormaster completed remote builds in B32973: Diff 75787.Aug 13 2020, 2:19 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 21 2020, 3:49 PM
Closed by commit rS365949: Add a tools/build/make.py script that bootstraps bmake and then runs the build (authored by arichardson). · Explain Why
This revision was automatically updated to reflect the committed changes.
arichardson added a commit: rS365949: Add a tools/build/make.py script that bootstraps bmake and then runs the build.

Revision Contents
Changeset List

PathSize
tools/
build/
200 lines

Diff 46835

View Options

tools/build/make.py

Loading...