Bug 242351

Summary: x11-themes/papirus-icon-theme: update to 20191201
Product: Ports & Packages Reporter: Hiroki Tagato <tagattie>
Component: Individual Port(s)Assignee: Tobias Kortkamp <tobik>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
URL: https://github.com/PapirusDevelopmentTeam/papirus-icon-theme/releases/tag/20191201
Attachments:
Description Flags
Gzipped patch
tagattie: maintainer-approval+
Gzipped patch (rev. 1) tagattie: maintainer-approval+

Comment 1 Tobias Kortkamp freebsd_committer freebsd_triage 2019-12-02 10:00:17 UTC 
The plist size is ridiculous.  Consider adding something like

post-install:
	@cd ${PREFIX} && ${FIND} share/icons -type f >> ${TMPPLIST}

to remove the need for one.
Comment 2 Tobias Kortkamp freebsd_committer freebsd_triage 2019-12-02 10:03:15 UTC 
(In reply to Tobias Kortkamp from comment #1)

I meant

post-install:
	@cd ${STAGEDIR}${PREFIX} && ${FIND} share/icons -type f >> ${TMPPLIST}
Comment 3 Hiroki Tagato freebsd_committer freebsd_triage 2019-12-06 11:49:50 UTC 
Created attachment 209755 [details]
Gzipped patch (rev. 1)
Comment 4 Hiroki Tagato freebsd_committer freebsd_triage 2019-12-06 11:50:57 UTC 
(In reply to Tobias Kortkamp from comment #2)

Thanks Tobias for the comments.

I have updated the patch as per your suggestion.
Comment 5 Tobias Kortkamp freebsd_committer freebsd_triage 2019-12-06 19:49:51 UTC 
Committed. Thanks!
Comment 6 commit-hook freebsd_committer freebsd_triage 2019-12-06 19:50:28 UTC 
A commit references this bug:

Author: tobik
Date: Fri Dec  6 19:49:39 UTC 2019
New revision: 519160
URL: https://svnweb.freebsd.org/changeset/ports/519160

Log:
  x11-themes/papirus-icon-theme: Update to 20191201

  Delete pkg-plist and dynamically generate the plist instead to ease
  future updates.  pkg-plist was 4.1 MiB large and had over 70000
  entries.  This also bloated the patch accordingly to the point it
  had to be gzipped to not go over Bugzilla's size limit for attachments.
  It was still over 100 KiB compressed.

  Changes:	https://github.com/PapirusDevelopmentTeam/papirus-icon-theme/releases
  PR:		242351
  Submitted by:	Hiroki Tagato <tagattie@yandex.com> (maintainer)

Changes:
  head/x11-themes/papirus-icon-theme/Makefile
  head/x11-themes/papirus-icon-theme/distinfo
  head/x11-themes/papirus-icon-theme/pkg-plist