From owner-freebsd-ports@FreeBSD.ORG Mon Jul 23 01:37:16 2012 Return-Path: Delivered-To: freebsd-ports@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 4D263106564A; Mon, 23 Jul 2012 01:37:16 +0000 (UTC) (envelope-from kpaasial@gmail.com) Received: from mail-vb0-f54.google.com (mail-vb0-f54.google.com [209.85.212.54]) by mx1.freebsd.org (Postfix) with ESMTP id 8913D8FC0A; Mon, 23 Jul 2012 01:37:15 +0000 (UTC) Received: by vbmv11 with SMTP id v11so5279058vbm.13 for ; Sun, 22 Jul 2012 18:37:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=3EuzF2I85r3H2o6jrOrpy4aHgRioviZzKJ7GJLfCTuI=; b=0tlfbLF4dWdIXs00UImRPIosECguKeBI4fDXcgTZ2fhMOn3J95c2APGyYGxTMBHID9 GixkGaRRtd/Jj4p2C7eqYZ0DGcAInttMAmEvbySXz7PheeUStO5aCpDG4NYIoP7TWdUl k/xdfzr6N5xTUOwm6paOOgShgd00aWIHDhNTEugTMwCDRkBm9esCSnrgpCciV+oqNoaI n6Pdm927+/ksfX3TlTRoLKO3g9ErQ9E/pOUFktArLl/HLMSo/THFZsYoxjFvjlw2cjJx w7+OZkViA0le6gc2hMpxOe1Ugyc9QE4yj0tQf7aYfV+2pOCvqeWx7XZXa2Hutjec7auj z5JQ== MIME-Version: 1.0 Received: by 10.52.36.180 with SMTP id r20mr9504253vdj.15.1343007428840; Sun, 22 Jul 2012 18:37:08 -0700 (PDT) Received: by 10.58.145.10 with HTTP; Sun, 22 Jul 2012 18:37:08 -0700 (PDT) In-Reply-To: <500CA767.2020206@FreeBSD.org> References: <15617903.FBMimp13fy@mocha.verizon.net> <2193820.CL19acAFmS@mocha.verizon.net> <500B1953.10809@FreeBSD.org> <500B8811.9090609@FreeBSD.org> <500C6DA9.1070305@FreeBSD.org> <500CA767.2020206@FreeBSD.org> Date: Mon, 23 Jul 2012 04:37:08 +0300 Message-ID: From: Kimmo Paasiala To: Doug Barton Content-Type: text/plain; charset=UTF-8 Cc: Scot Hetzel , kde@freebsd.org, Jeremy Messenger , ruby@freebsd.org, Edwin Groothuis , freebsd-ports@freebsd.org, gnome@freebsd.org, "Jason E. Hale" Subject: Re: How to remove erroneous deps from pkgng X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 23 Jul 2012 01:37:16 -0000 On Mon, Jul 23, 2012 at 4:22 AM, Doug Barton wrote: > On 07/22/2012 17:50, Jeremy Messenger wrote: >> On Sun, Jul 22, 2012 at 4:16 PM, Doug Barton wrote: >>> On 07/22/2012 07:51, Jeremy Messenger wrote: >>>> You need to start to respect the people's work and plan, Doug. Also I >>>> am part of team that maintaining the bsd.gnome.mk. Yes it will be >>>> re-add if anyone plan to remove it, because I already have planned add >>>> the :build/:run feature in the bsd.gnome.mk that the without :* will >>>> be both build/run time dependency. It is impossible for me to not >>>> re-add it. It is not going to be long because I already have function >>>> :build/:run in the bsd.mate.mk (was repocopied from bsd.gnome.mk) >>>> available. >>>> >>>> The only thing that I don't plan is to chase thousands of port to have >>>> the 'pkgconfig;build'. If anyone want to take up this task is cool >>>> with me. >>> >>> So what you're saying is that in spite of the fact that the community >>> has identified a bug that it wants to fix, a bug which was added by the >>> team maintaining bsd.gnome.mk, that you are refusing to modify your >>> patch to handle the bug; but instead are demanding your right to >>> reintroduce the bug, and then further demanding the right to not fix it >>> after the patch is in? >> >> Remove the runtime dependency is a bug either because there are a few >> that need it in the runtime dependency. > > Which ports are those? > > And even if these ports exist, an explicit run dep can be added for > them. Better to add one more dep for a few ports than bogus ones for > thousands. > >> I have written a best solution >> than remove that runtime line. It will be 99% bug free when all ports >> have the right :build/:run. > > I'm interested in your feedback to Max' point that the equivalent > functionality for kde.mk already has the right logic to handle proper > defaults if the right :label isn't set. > >>> And furthermore, you're refusing to even look at >>> the other dependencies which are handled by bsd.gnome.mk to see if >>> similar bugs were introduced *by your team*, but you're happy to let >>> everyone else sort out the problems in thousands of ports for you. >> >> Quiet simple, I do not have time for that huge project. > > That's fine, stand aside and let people who are willing to solve the > problem properly do the work. Then when your patch is in proper shape it > can be committed without fear of adding new bugs, or re-adding old ones. > > The position you're taking that we can't fix the bug because you demand > your right to get *your* patch in is entirely unreasonable. > > Doug > > -- > > Change is hard. > > > > _______________________________________________ > freebsd-ports@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-ports > To unsubscribe, send any mail to "freebsd-ports-unsubscribe@freebsd.org" Looking at the bsd.kde4.mk it looks like there's not much logic to detect the "right" kind of dependency. It seems to use suffixes _build and _run to achieve the same effect as pkgconfig:build and pkgconfig:run would have. It defaults to both build and run dependency if no type is specified. In my opinion Jeremy's solution would be fine for now, least amount of immediate destruction :P Regards, Kimmo Paasiala