Bug 222918

Summary: databases/squirrel-sql: Update to 3.8.0
Product: Ports & Packages Reporter: Jonathan Chen <jonc>
Component: Individual Port(s)Assignee: Guido Falsi <madpilot>
Status: Closed FIXED    
Severity: Affects Only Me CC: madpilot
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
3.8.0 update none

Description Jonathan Chen 2017-10-11 02:03:14 UTC
Created attachment 187067 [details]
3.8.0 update

Simple update to 3.8.0
Comment 1 Guido Falsi freebsd_committer freebsd_triage 2017-10-11 17:55:19 UTC
Hi,

Thanks for the submission.

the plist includes this addition:

%%DATADIR%%/icons/acorn.png.bak.png

while an acorn.png file already exists.

is this really needed? or is it a mistake in the distribution?

Maybe t can be removed during post-extract?
Comment 2 Jonathan Chen 2017-10-11 18:12:45 UTC
Yeah, I saw that as well. I think it's a mistake in the distribution, but cleaning up the staging tree seemed like too much effort for such a simple port.

I felt like that it would be simpler to just leave it and let the next update take care of it.
Comment 3 commit-hook freebsd_committer freebsd_triage 2017-10-11 20:04:48 UTC
A commit references this bug:

Author: madpilot
Date: Wed Oct 11 20:04:04 UTC 2017
New revision: 451793
URL: https://svnweb.freebsd.org/changeset/ports/451793

Log:
  Update squirrel-sql to 3.8.0

  PR:		222918
  Submitted by:	Jonathan Chen <jonc@chen.org.nz> (maintainer)

Changes:
  head/databases/squirrel-sql/Makefile
  head/databases/squirrel-sql/distinfo
  head/databases/squirrel-sql/pkg-plist
Comment 4 Guido Falsi freebsd_committer freebsd_triage 2017-10-11 20:05:10 UTC
Committed.

I added the bak file to the rm command in post-extract and adapted the plist.

Thanks!