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
F146206133: D25427.diff
Sat, Feb 28, 6:18 PM
Unknown Object (File)
Fri, Feb 27, 8:36 PM
Unknown Object (File)
Fri, Feb 27, 1:01 PM
Unknown Object (File)
Thu, Feb 26, 1:59 PM
Unknown Object (File)
Fri, Feb 20, 2:22 AM
Unknown Object (File)
Thu, Feb 19, 3:14 PM
Unknown Object (File)
Wed, Feb 18, 7:20 PM
Unknown Object (File)
Tue, Feb 17, 11:06 AM
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...