From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from pigeon.gentoo.org ([208.92.234.80] helo=lists.gentoo.org) by finch.gentoo.org with esmtp (Exim 4.60) (envelope-from ) id 1Mp6o6-0002xG-V3 for garchives@archives.gentoo.org; Sat, 19 Sep 2009 20:45:23 +0000 Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 92418E062C; Sat, 19 Sep 2009 20:45:22 +0000 (UTC) Received: from mail-ew0-f222.google.com (mail-ew0-f222.google.com [209.85.219.222]) by pigeon.gentoo.org (Postfix) with ESMTP id 3CE28E062C for ; Sat, 19 Sep 2009 20:45:22 +0000 (UTC) Received: by ewy22 with SMTP id 22so1867796ewy.14 for ; Sat, 19 Sep 2009 13:45:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=googlemail.com; s=gamma; h=domainkey-signature:received:received:date:from:to:cc:subject :message-id:in-reply-to:references:x-mailer:mime-version :content-type; bh=t7pcFV6lC1T0Orunpb6WrXfi0hWYsUJ7vAPGI9ECuKQ=; b=hrFdaxdgTfIe+6gWMV8hSC6NbtK7N/BIbpj5V28M3hWgo/xHnnzzLcKaIu1TfoZI54 5vX/zyWNEp3KSBf8H37Ccax40bPL7eg4VM13LR9Mzn08FXkam7bmvMSSvxfco9EmgJfK e0NBnkIOYk+WdL7lS6e+8o/BnoFzb9DC0htno= DomainKey-Signature: a=rsa-sha1; c=nofws; d=googlemail.com; s=gamma; h=date:from:to:cc:subject:message-id:in-reply-to:references:x-mailer :mime-version:content-type; b=OJbtJVZo/H89jWNvP74uZlWQemRYXKPezduVxJhrMF/qnhFgDXkQT9gAG/IPFkZSl+ ZKVLDd987gAJ/lTR98CPDrTkteQ0IuMoHgYDQYlVltkzrR+hoPSmo2AvF9RzSOpC7hKf In9xnwG0dIg+SF67G0TA4FmMq01YVFrrlwZ8I= Received: by 10.210.96.23 with SMTP id t23mr1132434ebb.11.1253393121617; Sat, 19 Sep 2009 13:45:21 -0700 (PDT) Received: from snowmobile (92-235-187-79.cable.ubr18.sgyl.blueyonder.co.uk [92.235.187.79]) by mx.google.com with ESMTPS id 24sm890052eyx.23.2009.09.19.13.45.21 (version=TLSv1/SSLv3 cipher=RC4-MD5); Sat, 19 Sep 2009 13:45:21 -0700 (PDT) Date: Sat, 19 Sep 2009 21:45:15 +0100 From: Ciaran McCreesh To: Patrick Lauer Cc: gentoo-pms@lists.gentoo.org Subject: Re: [gentoo-pms] tree-layout.tex small cleanup Message-ID: <20090919214515.6b314a3f@snowmobile> In-Reply-To: <200909192234.39415.patrick@gentoo.org> References: <200909192215.41235.patrick@gentoo.org> <20090919212541.269df025@snowcone> <200909192234.39415.patrick@gentoo.org> X-Mailer: Claws Mail 3.7.2 (GTK+ 2.16.5; i686-pc-linux-gnu) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Package Manager Specification discussions X-BeenThere: gentoo-pms@gentoo.org X-BeenThere: gentoo-pms@lists.gentoo.org Mime-Version: 1.0 Content-Type: multipart/signed; micalg=PGP-SHA1; boundary="Sig_/M9C=G1+Ro1mavMkYYLklb1C"; protocol="application/pgp-signature" X-Archives-Salt: d70ebbe6-7cdd-4f1d-b90b-8c0c95c40ebb X-Archives-Hash: 584257631bf6bd753f689f4eaf2c54de --Sig_/M9C=G1+Ro1mavMkYYLklb1C Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Sat, 19 Sep 2009 22:34:39 +0200 > > > Three small edits. > > > > > > - Removing a sentence that has no content, but spans three lines > >=20 > > No, the two sentences you're removing both have meaning: > >=20 > > The first says that it's ok for things to exist in > > profiles/categories that don't have a directory > > Rationale for that? People have done it in the past. It's also something that ends up happening if categories are removed but people are syncing using something that doesn't remove empty directories or directories that still contain editor-created backup files lying around. > > The second says that the package manager mustn't treat empty > > categories and categories that don't exist differently. > Not quite. What it says is that an empty and a non-existing category > are equivalent, which doesn't explain how to treat them. Your current=20 > interpretation is already a large improvement. The wording in PMS is sound, and says exactly what it needs to say. If you'd like to propose clarifications to that wording that make it easier to understand, feel free to do so, but the actual meaning mustn't be changed. > > Both are necessary. > > No, first one is confusing to read How is it in any way confusing? >, second one is a tautology. It's not. It would be quite possible to write an implementation that treats categories that don't exist as an error rather than an empty category. We have to forbid such an implementation. > > > - Simplifying the ebuild naming - since suffix is always "ebuild" > > > there is no need to use an indirection > > > > > > - Fixing the list because "suffix is ebuild" now is redundant > >=20 > > Uhm. That part of the patch doesn't apply, and the revision against > > which you're basing it isn't in the repository. Where did you get > > 'b78fde2' from? > > > Oh. I have over 900 lines diff already. Just pulling out the obvious > changes before moving on to the more subjective and debatable ones.=20 If that 900 line diff is 'drop kdebuild', I suggest you don't bother. In any case, please learn how to use 'git rebase' and only send patches that are against current master -- even for patches that do apply, if you're basing them upon unpublished changes, we can't use three way merges when applying them. --=20 Ciaran McCreesh --Sig_/M9C=G1+Ro1mavMkYYLklb1C Content-Type: application/pgp-signature; name=signature.asc Content-Disposition: attachment; filename=signature.asc -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.12 (GNU/Linux) iEYEARECAAYFAkq1Qt4ACgkQ96zL6DUtXhF4NACcDyifH7x5BOJqxexzqRoqlE7u iYEAoIrKRqPp3rgwQM4QFdJPGVwYy+MW =NV8G -----END PGP SIGNATURE----- --Sig_/M9C=G1+Ro1mavMkYYLklb1C--