From owner-svn-ports-all@freebsd.org Tue Aug 30 13:25:36 2016 Return-Path: Delivered-To: svn-ports-all@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 289A5BC716B; Tue, 30 Aug 2016 13:25:36 +0000 (UTC) (envelope-from freebsd.contact@marino.st) Received: from shepard.synsport.net (mail.synsport.com [208.69.230.148]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id F14B580E; Tue, 30 Aug 2016 13:25:35 +0000 (UTC) (envelope-from freebsd.contact@marino.st) Received: from [127.0.0.1] (ip70-178-28-115.ks.ks.cox.net [70.178.28.115]) (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) by shepard.synsport.net (Postfix) with ESMTP id 3D32143C48; Tue, 30 Aug 2016 08:25:33 -0500 (CDT) Reply-To: marino@freebsd.org Subject: Re: svn commit: r420924 - head/audio/gmpc-magnatune References: <201608261504.u7QF46B1060800@repo.freebsd.org> <074eb0ed-40bb-e188-4754-3edde192db12@marino.st> <8fe3e649-e929-c037-6193-c23b685a1222@marino.st> <20160830150544.4f57349e@FreeBSD.org> To: =?UTF-8?B?UGF3ZcWCIFDEmWthbGE=?= Cc: marino@freebsd.org, ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org From: John Marino Message-ID: <71f7c7a0-d2f3-9463-f769-0eb499d4701e@marino.st> Date: Tue, 30 Aug 2016 08:25:31 -0500 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 MIME-Version: 1.0 In-Reply-To: <20160830150544.4f57349e@FreeBSD.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit X-Antivirus: avast! (VPS 160829-1, 08/29/2016), Outbound message X-Antivirus-Status: Clean X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 30 Aug 2016 13:25:36 -0000 On 8/30/2016 08:05, Paweł Pękala wrote: > Hi John, > > On 2016-08-29 22:20 -0500, John Marino > wrote: >> On 8/29/2016 22:17, John Marino wrote: >>> On 8/26/2016 10:04, Pawel Pekala wrote: >>>> Author: pawel >>>> Date: Fri Aug 26 15:04:06 2016 >>>> New Revision: 420924 >>>> URL: https://svnweb.freebsd.org/changeset/ports/420924 >>>> >>>> Log: >>>> - Convert to USES=sqlite >>>> - Convert to PLIST_FILES, only 2 items in plist >>>> - Register proxy dependencies found by stage-qa >>>> >>>> Deleted: >>>> head/audio/gmpc-magnatune/pkg-plist >>>> Modified: >>>> head/audio/gmpc-magnatune/Makefile >>>> >>>> Modified: head/audio/gmpc-magnatune/Makefile >>>> ============================================================================== >>>> >>>> --- head/audio/gmpc-magnatune/Makefile Fri Aug 26 14:58:33 2016 >>>> (r420923) >>>> +++ head/audio/gmpc-magnatune/Makefile Fri Aug 26 15:04:06 2016 >>>> (r420924) >>>> @@ -14,13 +14,17 @@ LICENSE= GPLv2+ >>>> LICENSE_FILE= ${WRKSRC}/COPYING >>>> >>>> LIB_DEPENDS= libmpd.so:audio/libmpd \ >>>> - libsqlite3.so:databases/sqlite3 >>>> + libfreetype.so:print/freetype2 \ >>>> + libfontconfig.so:x11-fonts/fontconfig >>>> BUILD_DEPENDS= ${LOCALBASE}/libdata/pkgconfig/gmpc.pc:audio/gmpc >>>> RUN_DEPENDS= gmpc:audio/gmpc >>>> >>>> +USES= gettext gmake libtool pkgconfig sqlite >>>> +USE_GNOME= cairo gdkpixbuf2 glib20 gtk20 intltool libglade2 >>>> GNU_CONFIGURE= yes >>>> -USE_GNOME= intltool glib20 gtk20 libglade2 >>>> -USES= gettext gmake libtool pkgconfig >>>> INSTALL_TARGET= install-strip >>>> >>>> +PLIST_FILES= lib/gmpc/plugins/magnatuneplugin.so \ >>>> + share/gmpc/plugins/magnatune/magnatune.png >>>> + >>>> .include >>>> >>> >>> Pawel, >>> Many library dependencies were added. That requires a PORTREVISION >>> bump! >> >> actually, I meant to write this about audio/gmpc but it appears >> audio/gmpc-magnatune had the same issue. > > Dependencies added in both ports showed up in poudriere logs so they > are be definition proxy. So my commits register that this packages link > to particular libraries, no new dependencies are added - they are > already pulled in by packages needed to build port. > > I do check for new dependencies by comparing poudriere vs live > build stage-qa outputs to find new ones, but it's not the case here. > I do not believe your method is valid. If you use pkg(8) to check the dependencies before and after a change and there's not difference, no bump is needed. However, you're relying on dependencies to pull in other dependencies to say it's equivalent. Please check with mat@, he should confirm. Your method will cause poudriere and other builders to skip updating the port because it wasn't bumped. When in doubt, please bump. It's better to bump unnecessarily than to not bump when it's required. John --- This email has been checked for viruses by Avast antivirus software. https://www.avast.com/antivirus