Date: Tue, 30 Aug 2016 08:25:31 -0500 From: John Marino <freebsd.contact@marino.st> To: =?UTF-8?B?UGF3ZcWCIFDEmWthbGE=?= <pawel@FreeBSD.org> Cc: marino@freebsd.org, ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r420924 - head/audio/gmpc-magnatune Message-ID: <71f7c7a0-d2f3-9463-f769-0eb499d4701e@marino.st> In-Reply-To: <20160830150544.4f57349e@FreeBSD.org> References: <201608261504.u7QF46B1060800@repo.freebsd.org> <074eb0ed-40bb-e188-4754-3edde192db12@marino.st> <8fe3e649-e929-c037-6193-c23b685a1222@marino.st> <20160830150544.4f57349e@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 8/30/2016 08:05, Paweł Pękala wrote: > Hi John, > > On 2016-08-29 22:20 -0500, John Marino <freebsd.contact@marino.st> > 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 <bsd.port.mk> >>>> >>> >>> 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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?71f7c7a0-d2f3-9463-f769-0eb499d4701e>