Page MenuHomeFreeBSD
Differential D25427

Add WITH_CLANG_FORMAT option
ClosedPublic
Actions

Authored by cem on Jun 24 2020, 4:32 PM.
Tags
None
Referenced Files
F161375917: D25427.id73584.diff
Fri, Jul 3, 6:19 AM
F161337573: D25427.id.diff
Thu, Jul 2, 10:58 PM
Unknown Object (File)
Tue, Jun 30, 10:47 PM
Unknown Object (File)
Mon, Jun 29, 5:10 PM
Unknown Object (File)
Fri, Jun 26, 3:14 PM
Unknown Object (File)
Wed, Jun 24, 4:33 PM
Unknown Object (File)
Tue, Jun 23, 2:55 PM
Unknown Object (File)
Sun, Jun 21, 6:44 PM
View All Files
Subscribers
bdrewery
dim
imp

Details

Reviewers
emaste
Commits
rS362587: Add WITH_CLANG_FORMAT option
Summary

clang-format is enabled conditional on either WITH_CLANG_EXTRAS or
WITH_CLANG_FORMAT. Some sources in libclang are build conditional on
either rule, and obviously the clang-format binary itself depends on the
rule.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cem created this revision.Jun 24 2020, 4:32 PM
Herald added a subscriber: bdrewery. · View Herald TranscriptJun 24 2020, 4:32 PM
cem requested review of this revision.Jun 24 2020, 4:32 PM
Harbormaster completed remote builds in B31924: Diff 73580.Jun 24 2020, 4:32 PM
emaste accepted this revision.Jun 24 2020, 4:35 PM
emaste added a subscriber: dim.

LGTM

lib/clang/libclang/Makefile
25 ↗(On Diff #73580)

EXTRAS || FORMAT for alpha order?

This revision is now accepted and ready to land.Jun 24 2020, 4:36 PM
emaste added a comment.Jun 24 2020, 4:36 PM

Also add a WITH_CLANG_FORMAT text description file

cem updated this revision to Diff 73581.Jun 24 2020, 4:39 PM
cem marked an inline comment as done.

Reorder EXTRAS, FORMAT

Add WITH_CLANG_FORMAT file

This revision now requires review to proceed.Jun 24 2020, 4:39 PM
Harbormaster completed remote builds in B31925: Diff 73581.Jun 24 2020, 4:39 PM
emaste accepted this revision.Jun 24 2020, 4:48 PM
This revision is now accepted and ready to land.Jun 24 2020, 4:48 PM
Closed by commit rS362587: Add WITH_CLANG_FORMAT option (authored by cem). · Explain WhyJun 24 2020, 5:03 PM
This revision was automatically updated to reflect the committed changes.
cem added a commit: rS362587: Add WITH_CLANG_FORMAT option.
Herald added a subscriber: imp. · View Herald TranscriptJun 24 2020, 5:03 PM
dim mentioned this in rS363494: Merge llvm, clang, compiler-rt, libc++, libunwind, lld, lldb and openmp.Jul 24 2020, 8:48 PM
dim mentioned this in rS363496: Merge llvm, clang, compiler-rt, libc++, libunwind, lld, lldb and openmp.Jul 24 2020, 8:54 PM

Revision Contents
Changeset List

PathSize
head/
5 lines
lib/
clang/
libclang/
4 lines
share/
mk/
2 lines
targets/
pseudo/
bootstrap-tools/
2 lines
clang/
5 lines
tools/
build/
mk/
5 lines
options/
2 lines
usr.bin/
clang/
5 lines

Diff 73584

View Options

head/Makefile.inc1

Loading...
View Options

head/lib/clang/libclang/Makefile

Loading...
View Options

head/share/mk/src.opts.mk

Loading...
View Options

head/targets/pseudo/bootstrap-tools/Makefile

Loading...
View Options

head/targets/pseudo/clang/Makefile.depend

Loading...
View Options

head/tools/build/mk/OptionalObsoleteFiles.inc

Loading...
View Options

head/tools/build/options/WITH_CLANG_FORMAT

Loading...
View Options

head/usr.bin/clang/Makefile

Loading...