From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) by finch.gentoo.org (Postfix) with ESMTP id 5EBEB1381F3 for ; Mon, 21 Oct 2013 16:32:00 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 10C12E0B91; Mon, 21 Oct 2013 16:31:52 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) (using TLSv1 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 0D6ADE0A64 for ; Mon, 21 Oct 2013 16:31:50 +0000 (UTC) Received: from marga.jer-c2.orkz.net (D4B2706A.static.ziggozakelijk.nl [212.178.112.106]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) (Authenticated sender: jer) by smtp.gentoo.org (Postfix) with ESMTPSA id 7446133EED5 for ; Mon, 21 Oct 2013 16:31:49 +0000 (UTC) Date: Mon, 21 Oct 2013 18:31:43 +0200 From: Jeroen Roovers To: gentoo-dev@lists.gentoo.org Subject: Re: [gentoo-dev] Re: [Bug 488318] media-video/mpv[luajit] - Keyword request on alpha, arm, ppc, ppc64, sparc Message-ID: <20131021183143.431a7f40@marga.jer-c2.orkz.net> In-Reply-To: <20131021173203.2e1fc19c@TOMWIJ-GENTOO> References: <20131019190144.7858709c@marga.jer-c2.orkz.net> <20131019194329.41f538b0@TOMWIJ-GENTOO> <5263ADEC.9000000@gentoo.org> <5263B32F.1080302@gentoo.org> <5263C14E.8020709@gentoo.org> <20131020143056.0f775b7b@TOMWIJ-GENTOO> <20131021155034.629be51a@marga.jer-c2.orkz.net> <20131021173203.2e1fc19c@TOMWIJ-GENTOO> X-Mailer: Claws Mail 3.9.2-dirty (GTK+ 2.24.17; i686-pc-linux-gnu) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-dev@lists.gentoo.org Reply-to: gentoo-dev@lists.gentoo.org Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Archives-Salt: db50972b-b3cb-4e7c-bb95-32375d159afe X-Archives-Hash: f75b57be122e8413afb63255486e57c5 On Mon, 21 Oct 2013 17:32:03 +0200 Tom Wijsman wrote: > On Mon, 21 Oct 2013 15:50:34 +0200 > Jeroen Roovers wrote: > > > On Sun, 20 Oct 2013 14:30:56 +0200 > > Tom Wijsman wrote: > > > > There is no "instead". > > Why is there no "instead"? "I planned to unkeyword the USE flags instead" - this is wrong. The policy (see below for the link - you seem to have trouble finding it every time I point it out) is to drop keywords (that is entries in the KEYWORDS variable, see below for an extra explanation since you seem to be confused as to what it means). > > The default policy (did you read the devmanual yet?) > > Which policy are you referring to? (Did you refer me to anything?) http://devmanual.gentoo.org/keywording/index.html "Sometimes you may need to remove a keyword because of new unresolved dependencies. If you do this, you *must* file a bug notifying the relevant arch teams." > > is to DROP KEYWORDS (and let arch teams re-add them) > > Which is what I exactly did for the USE flags on most of the arches; > and a bug was planned to be filed, such that they can decide on it. You are apparently confusing "dropping keywords" (entries in the KEYWORDS variable in an ebuild) with "masking USE flags". Why would you do that? I know you are smarter than this. > > -unless- that is really cumbersome (when you need to drop more and > > more keywords as a result). > > Please do not make additional exceptions, we have enough of them. What do you mean? This common sense policy has been in place for years. If dropping one keyword breaks many (rev-)deps and is therefore not an option, it's quite the norm to either file a keyword request bug well in advance of anything in the tree breaking, or temporarily mask either ebuilds or USE flags, generally or specifically for an arch profile, and inform the arch teams. > Right; bug reference added, I see no other problem here. That should also tell you that it helps to inform arch teams by filing a bug report -before- you drop keywords or mess up the profiles. > Two entries will continue to be two entries; so, I do not see where > the difference in work comes from. Can you now please explain the > exception? There is no exception. You've made that bit up. The proper procedure is to ask arch teams to re-keyword the ebuild you had to drop their keyword for. If that is impossible, several other scenarios can be played out, such as use.masking to cover up a deficiency in porting to said arch, dropping keywords for that arch altogether, or whatever might work. The point is that you shouldn't be making that decision - you just maintain the package, not the arch profile, and you ran into a problem which you didn't propose they fix - you just covered it all up. The difference in work would amount to editing two files in the profiles/ subdir per architecture you want to abuse profiles/ for instead of asking their actual opinion, then later both undoing all that work and simply changing the files that matter, the ebuilds and auxiliary files (some 6 files in the example). For an arch developer wanting to test if the new dep should really be masked, or actually works just fine on his arch and should have been keyworded long ago, the improper procedure involves both unmasking the new dep in package.keywords as well as unmasking the USE flag on the test system, and then reversing the profile change and adding the keyword to the new dep. With just the dropped keyword, everything is normally much simpler. I don't see how you count up "entries" here - from experience re-adding keywords is a lot easier than removing profile masks. jer