Page MenuHomeFreeBSD
Differential D12978

devel/libfmt: Update: Added LICENSE_FILE; Added BUILD_SHARED_LIBS=ON
ClosedPublic
Actions

Authored by yuri on Nov 6 2017, 10:03 PM.
Tags
None
Referenced Files
F162926070: D12978.id.diff
Sat, Jul 18, 9:11 AM
Unknown Object (File)
Fri, Jul 17, 2:03 PM
Unknown Object (File)
Sat, Jul 11, 4:03 PM
Unknown Object (File)
Sat, Jun 20, 9:05 PM
Unknown Object (File)
Jun 4 2026, 5:50 PM
Unknown Object (File)
Jun 4 2026, 5:23 PM
Unknown Object (File)
Jun 4 2026, 6:08 AM
Unknown Object (File)
Jun 3 2026, 10:01 PM
View All Files
Subscribers
mat

Details

Reviewers
tcberner
adamw
Commits
rP453704: devel/libfmt: Update: Added LICENSE_FILE; Added BUILD_SHARED_LIBS=ON
Summary

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=223451

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 12521
Build 12796: arc lint + arc unit

Event Timeline

yuri created this revision.Nov 6 2017, 10:03 PM
Herald added a subscriber: mat. · View Herald TranscriptNov 6 2017, 10:03 PM
Harbormaster completed remote builds in B12521: Diff 34865.Nov 7 2017, 3:51 PM
mat added inline comments.Nov 7 2017, 4:51 PM
devel/libfmt/Makefile
4

Try not to change this when the version itself does not change, to avoid too much churn in the repository.

yuri updated this revision to Diff 34887.Nov 7 2017, 5:11 PM

No DISTVERSION when version didn't change.

Harbormaster completed remote builds in B12534: Diff 34887.Nov 7 2017, 5:11 PM
tcberner added inline comments.Nov 7 2017, 8:45 PM
devel/libfmt/Makefile
21

^ maybe think about using TEST_CMAKE_BOOL= FMT_TEST

yuri updated this revision to Diff 34899.Nov 7 2017, 9:10 PM
yuri marked 2 inline comments as done.

Requested changes.

Harbormaster completed remote builds in B12545: Diff 34899.Nov 7 2017, 9:10 PM
yuri added inline comments.Nov 7 2017, 9:10 PM
devel/libfmt/Makefile
21

This was a preexisting line :)

tcberner accepted this revision.Nov 7 2017, 9:35 PM
tcberner added inline comments.
devel/libfmt/Makefile
18

^ USES should appear befor any USE_FOO

21

^^ Now it isn't :D

This revision is now accepted and ready to land.Nov 7 2017, 9:35 PM
yuri marked 3 inline comments as done.Nov 7 2017, 9:44 PM
yuri closed this revision.Nov 7 2017, 9:47 PM

Revision Contents
Changeset List

PathSize
devel/
libfmt/
8 lines
4 lines

Diff 34865

View Options

devel/libfmt/Makefile

Loading...
View Options

devel/libfmt/pkg-plist

Loading...