Page MenuHomeFreeBSD
Differential D31960

graphics/gtk-update-icon-cache: use triggers
ClosedPublic
Actions

Authored by bapt on Sep 14 2021, 3:29 PM.
Tags
None
Referenced Files
F156755005: D31960.id96106.diff
Sat, May 16, 3:15 AM
F156754884: D31960.id96106.diff
Sat, May 16, 3:15 AM
F156754881: D31960.id96630.diff
Sat, May 16, 3:15 AM
F156754878: D31960.id95433.diff
Sat, May 16, 3:15 AM
F156754859: D31960.id95154.diff
Sat, May 16, 3:15 AM
F156754558: D31960.diff
Sat, May 16, 3:12 AM
Unknown Object (File)
Wed, Apr 29, 9:33 PM
Unknown Object (File)
Wed, Apr 29, 2:17 PM
View All Files
Subscribers
Alexander88207_protonmail.com
mat

Details

Reviewers
tcberner
Group Reviewers
portmgr
Commits
R11:46bcb098c82f: graphics/gtk-update-icon-cache: use triggers
R11:6f5d234138dc: framework: new USES trigger
Summary

Use triggers instead of enforcing the INSTALL_ICONS which enforces
gtk-update-icon-cache dependency on every ports installing icons.

The bonus:

  • factorize gtk-update-icon-cache update at the end of the build
  • stop adding a gratuicious dependency on gtk-update-icon-cache on packages which do not need it
  • works on packages where INSTALLS_ICONS were forgotten

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bapt created this revision.Sep 14 2021, 3:29 PM
Herald added 1 blocking reviewer(s): portmgr. · View Herald TranscriptSep 14 2021, 3:29 PM
Herald added a subscriber: mat. · View Herald Transcript
bapt requested review of this revision.Sep 14 2021, 3:29 PM
Harbormaster completed remote builds in B41511: Diff 95153.Sep 14 2021, 3:30 PM
bapt updated this revision to Diff 95154.Sep 14 2021, 4:03 PM

Create USES=trigger to simplify everything

Harbormaster completed remote builds in B41512: Diff 95154.Sep 14 2021, 4:03 PM
tcberner added a comment.Sep 14 2021, 4:08 PM
In D31960#720956, @bapt wrote:

Create USES=trigger to simplify everything

very nice

tcberner added inline comments.Sep 14 2021, 4:09 PM
Mk/Uses/trigger.mk
7

maybe add
defaults to ${PORTNAME}.

tcberner added a comment.Sep 14 2021, 4:13 PM

I assume the obsolete INSTALLS_ICONS lines in all the other ports are just not part of this review for brevity?

Alexander88207_protonmail.com added a subscriber: Alexander88207_protonmail.com.Sep 20 2021, 5:23 AM
bapt updated this revision to Diff 95433.Sep 21 2021, 8:29 AM

Fix leftovers

Harbormaster completed remote builds in B41617: Diff 95433.Sep 21 2021, 8:29 AM
bapt marked an inline comment as done.Sep 23 2021, 7:59 PM
bapt updated this revision to Diff 96106.Oct 1 2021, 8:29 PM

Fix share/icons leftover

Harbormaster completed remote builds in B41903: Diff 96106.Oct 1 2021, 8:29 PM
This revision was not accepted when it landed; it landed in state Needs Review.Oct 10 2021, 5:47 PM
Closed by commit R11:6f5d234138dc: framework: new USES trigger (authored by bapt). · Explain Why
This revision was automatically updated to reflect the committed changes.
bapt added a commit: R11:6f5d234138dc: framework: new USES trigger.
bapt added a commit: R11:46bcb098c82f: graphics/gtk-update-icon-cache: use triggers.

Revision Contents
Changeset List

PathSize
Mk/
Uses/
25 lines
29 lines

Diff 96630

View Options

Mk/Uses/gnome.mk

Loading...
View Options

Mk/Uses/trigger.mk

Loading...