Page MenuHomeFreeBSD
Differential D17155

graphics/partio: add DOXYGEN option
ClosedPublic
Actions

Authored by fernape on Sep 13 2018, 4:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jan 1, 8:15 AM
Unknown Object (File)
Mon, Dec 29, 8:53 AM
Unknown Object (File)
Sun, Dec 28, 12:09 PM
Unknown Object (File)
Fri, Dec 26, 1:33 PM
Unknown Object (File)
Dec 17 2025, 12:11 PM
Unknown Object (File)
Nov 20 2025, 8:40 PM
Unknown Object (File)
Nov 20 2025, 2:01 PM
Unknown Object (File)
Nov 20 2025, 4:59 AM
View All Files
Subscribers
mat

Details

Reviewers
tcberner
tz
Commits
rP480080: graphics/partio: Remove PYTHON option.
Summary

Most of the changes come from https://reviews.freebsd.org/D16662

  • Add DOXYGEN option.
  • Remove post-patch-PYTHON-off due to PYTHON_BROKEN_OFF
  • Move some variables to their proper locations
Test Plan
  • portlint -AC OK
  • poudriere builds for {10.4,11.1}{amd64,i386}, 12i386 OK

Diff Detail

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

Event Timeline

fernape created this revision.Sep 13 2018, 4:21 PM
Herald added a subscriber: mat. · View Herald TranscriptSep 13 2018, 4:21 PM
Harbormaster completed remote builds in B19571: Diff 48009.Sep 13 2018, 4:21 PM
mat added inline comments.Sep 14 2018, 10:10 AM
graphics/partio/Makefile
16

Why move this?

38

Why move this?

fernape updated this revision to Diff 48051.Sep 14 2018, 1:52 PM

Keep USE_* sorted alphabetically

Harbormaster completed remote builds in B19596: Diff 48051.Sep 14 2018, 1:53 PM
fernape updated this revision to Diff 48052.Sep 14 2018, 1:54 PM

Keep related variables close

Harbormaster completed remote builds in B19597: Diff 48052.Sep 14 2018, 1:54 PM
fernape marked 2 inline comments as done.Sep 14 2018, 1:55 PM
mat added inline comments.Sep 14 2018, 4:42 PM
graphics/partio/Makefile
17

Why blank line?

32

Remove this maybe.

34–37

Maybe remove the option if it is required now.

fernape updated this revision to Diff 48133.Sep 17 2018, 5:04 PM
  • Remove blank line
  • Remove PYTHON option
Harbormaster completed remote builds in B19629: Diff 48133.Sep 17 2018, 5:04 PM
fernape marked 2 inline comments as done.Sep 17 2018, 5:05 PM
tcberner accepted this revision.Sep 18 2018, 7:02 PM

Looks good to me.

This revision is now accepted and ready to land.Sep 18 2018, 7:02 PM
Closed by commit rP480080: graphics/partio: Remove PYTHON option. (authored by fernape). · Explain WhySep 18 2018, 8:22 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
graphics/
partio/
27 lines

Diff 48133

View Options

graphics/partio/Makefile

Loading...