Bug 258502 - [exp-run] drop INSTALLS_ICONS in favour of triggers
Summary: [exp-run] drop INSTALLS_ICONS in favour of triggers
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Ports Framework (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Baptiste Daroussin
URL: https://reviews.freebsd.org/D31960
Keywords:
Depends on:
Blocks:
 
Reported: 2021-09-14 15:38 UTC by Baptiste Daroussin
Modified: 2021-10-10 17:52 UTC (History)
1 user (show)

See Also:
antoine: exp-run+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Baptiste Daroussin freebsd_committer freebsd_triage 2021-09-14 15:38:49 UTC

    
Comment 1 Antoine Brodin freebsd_committer freebsd_triage 2021-09-16 12:23:29 UTC
There is a new leftover in a lot of ports:

=>> Checking for extra files and directories
=>> Error: Files or directories left over:
share/icons/hicolor/icon-theme.cache

I don't know if it's an issue in a port or if Mk/Scripts/check_leftovers.sh has to be updated?
Comment 2 Baptiste Daroussin freebsd_committer freebsd_triage 2021-09-20 07:55:40 UTC
Nope, it needs more work from the trigger point of view.

I'll ping you when I have fixed the issue
Comment 3 Baptiste Daroussin freebsd_committer freebsd_triage 2021-09-21 08:30:35 UTC
done, I have update to diff in the review!
Note: it requires pkg 1.17.2!
Comment 5 Baptiste Daroussin freebsd_committer freebsd_triage 2021-09-27 09:32:34 UTC
This issue should imho be fixed by adding share/icons to the default mtree

I will do that.
Comment 6 Baptiste Daroussin freebsd_committer freebsd_triage 2021-09-27 13:05:08 UTC
done, imho it would be worth another run
Comment 7 Antoine Brodin freebsd_committer freebsd_triage 2021-09-29 18:43:03 UTC
(In reply to Baptiste Daroussin from comment #6)
f
This does not fix the leftover directory.  (I am not sure that the mtree file is still used in packages after auto-directory removal)
Comment 8 Baptiste Daroussin freebsd_committer freebsd_triage 2021-10-01 20:30:17 UTC
really fixed this time (the patch has been updated in phabricator)
Comment 9 Antoine Brodin freebsd_committer freebsd_triage 2021-10-09 07:23:20 UTC
Exp-run looks fine
Comment 10 commit-hook freebsd_committer freebsd_triage 2021-10-10 17:48:39 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=46bcb098c82f7a18c7667fc99f6a7824b2b6eeef

commit 46bcb098c82f7a18c7667fc99f6a7824b2b6eeef
Author:     Baptiste Daroussin <bapt@FreeBSD.org>
AuthorDate: 2021-09-14 15:24:59 +0000
Commit:     Baptiste Daroussin <bapt@FreeBSD.org>
CommitDate: 2021-10-10 17:46:46 +0000

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

    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

    PR:                     258502
    Exp-run:                antoine
    Reviewed by:            tcberner
    Differential Revision:  https://reviews.freebsd.org/D31960

 graphics/gtk-update-icon-cache/Makefile            |  3 ++-
 .../files/gtk-update-icon-cache.ucl.in (new)       | 30 ++++++++++++++++++++++
 2 files changed, 32 insertions(+), 1 deletion(-)