Bug 199220

Summary: [PATCH] audio/quimup: Update to version 1.4.0
Product: Ports & Packages Reporter: tkato432
Component: Individual Port(s)Assignee: John Marino <marino>
Status: Closed FIXED    
Severity: Affects Only Me CC: danfe, marino
Priority: --- Keywords: patch
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
audio_quimup.diff none

Description tkato432 2015-04-06 18:39:34 UTC
Created attachment 155258 [details]
audio_quimup.diff
Comment 1 Alexey Dokuchaev freebsd_committer freebsd_triage 2015-04-11 15:42:44 UTC
Your PR does not explain the switch from Qt4 to Qt5 at all.  Note that Qt4 is still being primarily used for vast majority of ports.  If needed, another quimup-qt5 port should be created instead (or quimup-qt4, if Qt5-version gains sufficient reasons to become default).
Comment 2 tkato432 2015-04-14 11:00:35 UTC
It appears that upstream author intentionally discontinued Qt4
support from the latest version.

Meanwhile, in general, keeping different versions of the same
applications featuring different toolkits as different ports
should be avoided unless upstream author claims that the latest
version hasn't been stable enough or each ports are necessary for
other ports as dependency.

In this case, it seem to me that it would be unnecessary to
preserve older version in ports tree due to retirement of Qt4
support.
Comment 3 commit-hook freebsd_committer freebsd_triage 2015-10-10 12:16:21 UTC
A commit references this bug:

Author: marino
Date: Sat Oct 10 12:15:58 UTC 2015
New revision: 398991
URL: https://svnweb.freebsd.org/changeset/ports/398991

Log:
  audio/quimup: Upgrade version 1.3.2 => 1.4.0

  I tweaked provided patch to revert some unnecessary changes. Technically
  the change of ${PORTNAME} to quimup is unnecessary but I fully support
  the reversal of PORTNAME abuse.

  PR:		199220
  Submitted by:	ports fury

Changes:
  head/audio/quimup/Makefile
  head/audio/quimup/distinfo
Comment 4 John Marino freebsd_committer freebsd_triage 2015-10-10 12:16:45 UTC
thanks