From owner-freebsd-ports-bugs@freebsd.org Tue Sep 1 08:37:35 2015 Return-Path: Delivered-To: freebsd-ports-bugs@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 476B59C640F for ; Tue, 1 Sep 2015 08:37:35 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from kenobi.freebsd.org (kenobi.freebsd.org [IPv6:2001:1900:2254:206a::16:76]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 185671088 for ; Tue, 1 Sep 2015 08:37:35 +0000 (UTC) (envelope-from bugzilla-noreply@freebsd.org) Received: from bugs.freebsd.org ([127.0.1.118]) by kenobi.freebsd.org (8.15.2/8.15.2) with ESMTP id t818bYlO068649 for ; Tue, 1 Sep 2015 08:37:34 GMT (envelope-from bugzilla-noreply@freebsd.org) From: bugzilla-noreply@freebsd.org To: freebsd-ports-bugs@FreeBSD.org Subject: [Bug 202737] [patch] audio/hydrogen: fix dependencies, add more options Date: Tue, 01 Sep 2015 08:37:35 +0000 X-Bugzilla-Reason: AssignedTo X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: Ports & Packages X-Bugzilla-Component: Individual Port(s) X-Bugzilla-Version: Latest X-Bugzilla-Keywords: patch X-Bugzilla-Severity: Affects Only Me X-Bugzilla-Who: FreeBSD@ShaneWare.Biz X-Bugzilla-Status: New X-Bugzilla-Priority: --- X-Bugzilla-Assigned-To: freebsd-ports-bugs@FreeBSD.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: maintainer-feedback? X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Bugzilla-URL: https://bugs.freebsd.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 01 Sep 2015 08:37:35 -0000 https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=202737 --- Comment #7 from FreeBSD@ShaneWare.Biz --- Some of the changes are useful some aren't. Adding LICENSE_FILE is not desired, hydrogen uses the gplv2 and multiple copies of the same license are not wanted. Cleanup of USE_QT4 - yes I should have looked at that a long time ago. Adding CFLAGS is not needed. Adding options DEBUG and TEST - not sure DEBUG and TEST would be needed outside of initial port testing but no harm. Adding librubberband support - this is an experimental feature that can be used in a -devel port but doesn't belong in a stable release. If a user installs audio/rubberband then hydrogen will use the cli tool to provide time stretching. While the rubberband port can be added as a dependency, hydrogen isn't really dependent on it, it will be used if it is there and ignored if it isn't. Disabling OSS does nothing, as oss is part of the base system hydrogen finds it and uses it even if you say you don't want it. The same goes for libarchive and libtar, hydrogen uses a tar format to combine multiple samples into one file, libarchive provides the same functionality with the addition of compression. If libarchive is found hydrogen will use it and ignore libtar, even if you say you don't want libarchive, as it is in the base system it will be found and libtar will never be use, so adding it as an option is pointless. The addition of USES=libarchive is useful so that the newer port version can be used. So the update I submit cleans up USE_QT4 and CMAKE_ARGS, adds libarchive to USES and adds DEBUG and TEST options. -- You are receiving this mail because: You are the assignee for the bug.