depend-cleanup.sh: Remove everything before the last build epoch.
AcceptedPublic
Actions

Authored by imp on Tue, Nov 11, 6:51 PM.

Details

Reviewers

jrtc27
emaste

Summary

We were deleting the .a files for llvm when it was build in bootstrap,
but this caused us to rebuild all the bootstrap things since the
bootstrap compiler changed, making the build time 25 minutes instead of
77 seconds on one of my systems. Items before the last rebuild from
scratch epoch are no longer relevant because you have to rebuild
entirely, so the incremental conditions that lead up to them being
needed are no longer true.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 68551
Build 65434: arc lint + arc unit

Event Timeline

imp created this revision.Tue, Nov 11, 6:51 PM

Herald added a subscriber: jrtc27. · View Herald TranscriptTue, Nov 11, 6:51 PM

imp requested review of this revision.Tue, Nov 11, 6:51 PM

Harbormaster completed remote builds in B68551: Diff 166257.Tue, Nov 11, 6:51 PM

jrtc27 accepted this revision.Tue, Nov 11, 6:59 PM

jrtc27 added inline comments.

tools/build/depend-cleanup.sh
472	Technically this can go too because the build epoch didn't land until the 14th (probably the stamp should have been updated to correspond to the commit date); see `git diff 0090096e52738383d6bf48ed5d27dffc850c8a01 <remote>/main tools/build/depend-cleanup.sh`.

This revision is now accepted and ready to land.Tue, Nov 11, 6:59 PM

emaste accepted this revision.Tue, Nov 11, 7:35 PM

emaste added a subscriber: emaste.

emaste added inline comments.

tools/build/depend-cleanup.sh
268–274	This perhaps should be a new example in the comment at the top of this file

Revision Contents
Changeset List

Path

Size

tools/

build/

depend-cleanup.sh

209 lines

Diff 166257

View Options

tools/build/depend-cleanup.sh

depend-cleanup.sh: Remove everything before the last build epoch.AcceptedPublicActions