Bug 197147

Summary: security/keepassx2: plist incomplete
Product: Ports & Packages Reporter: Dmitry Marakasov <amdmi3>
Component: Individual Port(s)Assignee: Dmitry Marakasov <amdmi3>
Status: Closed FIXED    
Severity: Affects Many People CC: eric
Priority: --- Keywords: needs-qa, patch
Version: LatestFlags: koobs: maintainer-feedback+
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Patch none

Description Dmitry Marakasov freebsd_committer freebsd_triage 2015-01-28 01:40:46 UTC
Created attachment 152289 [details]
Patch

Plist is missing bunch of files, see attached patch. While here, drop @dirrm* from plist and add empty dirs
Comment 1 Bugzilla Automation freebsd_committer freebsd_triage 2015-01-28 01:40:46 UTC
Maintainer CC'd
Comment 2 Eric Camachat 2015-01-28 02:25:47 UTC
Hmm... my build doesn't have that files, see below, could you provide your build log?

# find /usr/local/share -name '*keepassx*' -type f
/usr/local/share/icons/hicolor/16x16/apps/keepassx.png
/usr/local/share/icons/hicolor/24x24/apps/keepassx.png
/usr/local/share/icons/hicolor/32x32/apps/keepassx.png
/usr/local/share/icons/hicolor/48x48/apps/keepassx.png
/usr/local/share/icons/hicolor/64x64/apps/keepassx.png
/usr/local/share/icons/hicolor/128x128/apps/keepassx.png
/usr/local/share/icons/hicolor/256x256/apps/keepassx.png
/usr/local/share/icons/hicolor/scalable/apps/keepassx.svgz
/usr/local/share/icons/matefaenzagray/status/24/keepassx-panel.png
/usr/local/share/icons/matefaenzagray/status/22/keepassx-panel.png
/usr/local/share/icons/matefaenzagray/extras/keepassx/icons/keepassx.png
/usr/local/share/icons/matefaenzadark/status/24/keepassx-panel.png
/usr/local/share/icons/matefaenzadark/status/22/keepassx-panel.png
/usr/local/share/icons/matefaenzadark/extras/keepassx/icons/keepassx.png
/usr/local/share/icons/matefaenza/status/24/keepassx-panel.png
/usr/local/share/icons/matefaenza/status/22/keepassx-panel.png
/usr/local/share/icons/matefaenza/extras/keepassx/icons/keepassx.png
/usr/local/share/icons/matefaenza/apps/scalable/keepassx.svg
/usr/local/share/icons/matefaenza/apps/96/keepassx.png
/usr/local/share/icons/matefaenza/apps/64/keepassx.png
/usr/local/share/icons/matefaenza/apps/48/keepassx.png
/usr/local/share/icons/matefaenza/apps/32/keepassx.png
/usr/local/share/icons/matefaenza/apps/24/keepassx.png
/usr/local/share/icons/matefaenza/apps/22/keepassx.png
/usr/local/share/icons/matefaenza/apps/16/keepassx.png
/usr/local/share/keepassx/icons/application/128x128/apps/keepassx.png
/usr/local/share/keepassx/icons/application/16x16/apps/keepassx.png
/usr/local/share/keepassx/icons/application/24x24/apps/keepassx.png
/usr/local/share/keepassx/icons/application/256x256/apps/keepassx.png
/usr/local/share/keepassx/icons/application/32x32/apps/keepassx.png
/usr/local/share/keepassx/icons/application/48x48/apps/keepassx.png
/usr/local/share/keepassx/icons/application/64x64/apps/keepassx.png
/usr/local/share/keepassx/icons/application/scalable/apps/keepassx.svgz
Comment 3 Kubilay Kocak freebsd_committer freebsd_triage 2015-01-28 05:26:17 UTC
Submitter is committer
Comment 4 Dmitry Marakasov freebsd_committer freebsd_triage 2015-01-28 21:42:02 UTC
Obviously you won't have them in /usr/local because they need to be listed in pkg-plist to be included into package. You could've look into ${STAGEDIR} instead.

https://people.freebsd.org/~amdmi3/keepassx2.log
Comment 5 Eric Camachat 2015-01-28 21:53:39 UTC
Your're right, Dmitry please commit it.
Thanks.
Comment 6 commit-hook freebsd_committer freebsd_triage 2015-01-29 02:17:54 UTC
A commit references this bug:

Author: amdmi3
Date: Thu Jan 29 02:17:31 UTC 2015
New revision: 378100
URL: https://svnweb.freebsd.org/changeset/ports/378100

Log:
  - Add missing plist files and empty dirs, drop @dirrm*

  PR:		197147
  Submitted by:	amdmi3
  Approved by:	eric@camachat.org (maintainer)

Changes:
  head/security/keepassx2/Makefile
  head/security/keepassx2/pkg-plist