Page MenuHomeFreeBSD
Differential D51583

Move toolchain utils to -toolchain package
ClosedPublic
Actions

Authored by ivy on Sun, Jul 27, 8:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Aug 20, 8:04 PM
Unknown Object (File)
Fri, Aug 15, 2:12 AM
Unknown Object (File)
Thu, Aug 14, 9:24 AM
Unknown Object (File)
Thu, Aug 14, 6:29 AM
Unknown Object (File)
Sat, Aug 2, 6:24 AM
Unknown Object (File)
Tue, Jul 29, 3:00 AM
Unknown Object (File)
Tue, Jul 29, 12:44 AM
Unknown Object (File)
Tue, Jul 29, 12:31 AM
View All 10 Files
Subscribers
emaste
imp

Details

Reviewers
des
dim
Group Reviewers
pkgbase
Commits
rG328110da2661: Move toolchain utils to -toolchain package
Summary

Move LLVM ar, nm, and size. Also move elftoolchain strings, which
should have already been there.

The remaining LLVM utilities (including strings) stay in -clang for now
since they're links to other executables.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 65785
Build 62668: arc lint + arc unit

Event Timeline

ivy created this revision.Sun, Jul 27, 8:44 PM
Herald added a subscriber: imp. · View Herald TranscriptSun, Jul 27, 8:44 PM
ivy requested review of this revision.Sun, Jul 27, 8:44 PM
Harbormaster completed remote builds in B65785: Diff 159255.Sun, Jul 27, 8:45 PM
ivy added a reviewer: dim.Sun, Jul 27, 8:46 PM
dim accepted this revision.Mon, Jul 28, 10:09 AM

LGTM

This revision is now accepted and ready to land.Mon, Jul 28, 10:09 AM
des accepted this revision.Mon, Jul 28, 4:25 PM
Closed by commit rG328110da2661: Move toolchain utils to -toolchain package (authored by ivy). · Explain WhyMon, Jul 28, 4:29 PM
This revision was automatically updated to reflect the committed changes.
ivy added a commit: rG328110da2661: Move toolchain utils to -toolchain package.
emaste added a subscriber: emaste.Mon, Jul 28, 4:33 PM

LGTM.

strings(1) has always been a bit special -- we included it even under WITHOUT_TOOLCHAIN, so not having it in the toolchain pkg made some sense. But I'm fine with it either way.

Revision Contents
Changeset List

PathSize
usr.bin/
clang/
2 lines
llvm-ar/
1 line
llvm-nm/
1 line
llvm-size/
1 line
2 lines
strings/
2 lines

Diff 159255

View Options

usr.bin/clang/clang.prog.mk

Loading...
View Options

usr.bin/clang/llvm-ar/Makefile

Loading...
View Options

usr.bin/clang/llvm-nm/Makefile

Loading...
View Options

usr.bin/clang/llvm-size/Makefile

Loading...
View Options

usr.bin/clang/llvm.prog.mk

Loading...
View Options

usr.bin/strings/Makefile

Loading...