Bug 213211

Summary: devel/visualparadigm: Update to 13.2 build 20161002
Product: Ports & Packages Reporter: Fernando Apesteguía <fernape>
Component: Individual Port(s)Assignee: freebsd-ports-bugs (Nobody) <ports-bugs>
Status: Closed FIXED    
Severity: Affects Some People CC: woodsb02
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
URL: https://www.visual-paradigm.com/
See Also: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=212951
Attachments:
Description Flags
patch to the ports tree fernape: maintainer-approval+

Description Fernando Apesteguía freebsd_committer freebsd_triage 2016-10-04 21:30:11 UTC
Created attachment 175428 [details]
patch to the ports tree

The attached patch updates devel/visualparadigm to 13.2 build 20161002.

The upstream project doesn't provide information for minor updates.

This PR supersedes this one:

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=212951

This patch adds all the improvements made and suggested by woodsb02@

- Q/A
 * portlint -AC OK
 * port test OK
 * poudriere builds for {9.3,10.1,10.2,10.3}{i386,amd64} OK (logs available on request)
Comment 1 Fernando Apesteguía freebsd_committer freebsd_triage 2016-10-14 21:34:33 UTC
Any news on this?
Comment 2 commit-hook freebsd_committer freebsd_triage 2016-10-15 08:08:43 UTC
A commit references this bug:

Author: woodsb02
Date: Sat Oct 15 08:08:04 UTC 2016
New revision: 424007
URL: https://svnweb.freebsd.org/changeset/ports/424007

Log:
  devel/visualparadigm: Update to 13.2.20161021

  - Adds LICENSE details, which are quite restrictive with redistribution
  - Set the WRKSRC to the directory the distfile will be extracted into
  - Install into ${DATADIR} instead of LOCALBASE/visualparadigm
  - Use ${INSTALL_PROGRAM} for binaries, rather than pkg-plist @mode 0755

  PR:		213211
  Submitted by:	fernando.apesteguia@gmail.com (maintainer)
  Approved by:	adamw (mentor, implicit)

Changes:
  head/devel/visualparadigm/Makefile
  head/devel/visualparadigm/distinfo
  head/devel/visualparadigm/pkg-plist
Comment 3 Ben Woods freebsd_committer freebsd_triage 2016-10-15 08:09:31 UTC
Committed. Nice work Fernando, thanks for implementing INSTALL_PROGRAM to replace @mode 0755. Apologies for the delay on getting this committed.
Comment 4 Ben Woods freebsd_committer freebsd_triage 2016-10-15 08:11:29 UTC
Oh yeah, for bonus points I bumped it to version 13.2.20161021 in the commit.
Comment 5 Fernando Apesteguía freebsd_committer freebsd_triage 2016-10-15 15:00:58 UTC
(In reply to Ben Woods from comment #3)

I actually used INSTALL_SCRIPT to avoid stripping script files ;)
Comment 6 Fernando Apesteguía freebsd_committer freebsd_triage 2016-10-15 15:01:54 UTC
(In reply to Ben Woods from comment #4)

Perfect! That's why I asked, to avoid deprecating this PR because of the new version.

Thank you very much, Ben!