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)
Feb 20 2024, 8:41 AM
Unknown Object (File)
Feb 20 2024, 8:41 AM
Unknown Object (File)
Feb 20 2024, 8:41 AM
Unknown Object (File)
Feb 20 2024, 8:41 AM
Unknown Object (File)
Feb 20 2024, 8:41 AM
Unknown Object (File)
Feb 20 2024, 8:41 AM
Unknown Object (File)
Feb 20 2024, 8:27 AM
Unknown Object (File)
Jan 26 2024, 8:46 PM
View All 51 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 19597
Build 19178: 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
14–15

Why move this?

40

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
15

Why blank line?

31

Remove this maybe.

33–36

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/
23 lines

Diff 48052

View Options

graphics/partio/Makefile

Loading...