From owner-svn-ports-all@freebsd.org Sun Mar 12 01:16:35 2017 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 DB8E3CF9641; Sun, 12 Mar 2017 01:16:35 +0000 (UTC) (envelope-from rezny@freebsd.org) Received: from mail.modirum.com (mail.modirum.com [31.185.27.10]) (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 9FDD41DAB; Sun, 12 Mar 2017 01:16:35 +0000 (UTC) (envelope-from rezny@freebsd.org) Received: by mail.modirum.com with esmtpsa (TLSv1:ECDHE-RSA-AES256-SHA:256) (Exim 4.87 (FreeBSD)) (envelope-from ) id 1cmrVO-000Cdi-Ka; Sun, 12 Mar 2017 00:37:02 +0000 From: Matthew Rezny To: Tijl Coosemans Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org, swills@freebsd.org, gnome@freebsd.org Subject: Re: svn commit: r435961 - in head/www/webkit-gtk2: . files Date: Sun, 12 Mar 2017 01:36:58 +0100 Message-ID: <1651531.nGg9pBWiG5@workstation.reztek> Organization: FreeBSD User-Agent: KMail/4.14.10 (FreeBSD/11.0-STABLE; KDE/4.14.10; amd64; ; ) In-Reply-To: <20170312001958.6e8fdd4b@kalimero.tijl.coosemans.org> References: <201703112115.v2BLF3qk062113@repo.freebsd.org> <20170312001958.6e8fdd4b@kalimero.tijl.coosemans.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-SA-Authenticated: Yes X-SA-Exim-Connect-IP: 94.142.238.108 X-SA-Exim-Mail-From: rezny@freebsd.org X-SA-Exim-Scanned: No (on mail.modirum.com); SAEximRunCond expanded to false X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.23 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: Sun, 12 Mar 2017 01:16:36 -0000 On Sunday 12 March 2017 00:19:58 Tijl Coosemans wrote: > On Sat, 11 Mar 2017 21:15:03 +0000 (UTC) Matthew Rezny > > wrote: > > Author: rezny > > Date: Sat Mar 11 21:15:03 2017 > > New Revision: 435961 > > URL: https://svnweb.freebsd.org/changeset/ports/435961 > > > > Log: > > - Fix building on PPC/PPC64 [1] > > - Fix building on ARMv6 [2] > > - Add missing indirect dependencies > > You should never add the dependencies of another port to your port > because if that other port ever changes dependencies your port will still > pull them in. If you are getting warnings about missing dependencies and > you know they are indirect then you know there's a problem with one of > the other dependencies of your port. The problem needs to fixed there. > In this case it's probably because of gnome related pkg-config files. > These dependencies need to be added to Mk/Uses/gnome.mk. > > > - Possibly fix build on sparc64 (unconfirmed) > > > > PR: 212903 [1] > > Submitted by: jhibbits [1], strejda [2] > > Approved by: swills (mentor) I completely agree with that from a technical position. When stage-qa started complaining about indirect dependencies, I initially ignored them as it looked like an obvious error in the script. Surely, actual dependencies can be calculated recursively taking options into account. However, through both the actions taken on the PRs submitted at the time and direct statements when I questioned the situation, I was informed that the indirect dependencies should be added. I think it is completely unproductive and incorrect, but I had more important things to do than press the issue. I would be happy to cease adding indirect dependencies, which not only depend on the port's options but the options of the ports it depends upon, and the options of the ports those depend upon and so on. Has there been a change of policy and if so when can we expect to see a fixed stage-qa? It'll take some time to undo all the damage.