From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from hetzner.pbcl.net (mail.pbcl.net [88.198.119.4]) by mail.openembedded.org (Postfix) with ESMTP id 9C0716CC2E for ; Sun, 19 Jan 2014 14:42:40 +0000 (UTC) Received: from [12.157.112.67] (helo=[10.71.2.210]) by hetzner.pbcl.net with esmtpsa (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.80) (envelope-from ) id 1W4taC-00074i-6K; Sun, 19 Jan 2014 15:42:40 +0100 Message-ID: <1390142552.2467.48.camel@e130.pbcl.net> From: Phil Blundell To: Martin Jansa Date: Sun, 19 Jan 2014 14:42:32 +0000 In-Reply-To: <45a8519a8075d557e8c12db5632ea20e527c1f5f.1390138091.git.Martin.Jansa@gmail.com> References: <45a8519a8075d557e8c12db5632ea20e527c1f5f.1390138091.git.Martin.Jansa@gmail.com> Organization: Phil Blundell Consulting Ltd X-Mailer: Evolution 3.8.5-2+b1 Mime-Version: 1.0 X-Spam_score: -1.0 X-Spam_score_int: -9 X-Spam_bar: - X-Spam_report: Spam detection software, running on the system "hetzner.pbcl.net", has identified this incoming email as possible spam. The original message has been attached to this so you can view it (if it isn't spam) or label similar future email. If you have any questions, see @@CONTACT_ADDRESS@@ for details. Content preview: On Sun, 2014-01-19 at 14:28 +0100, Martin Jansa wrote: > * move read_shlib_providers before registering package as provider > and don't change provider if it already exists, show warning instead This new patch doesn't seem to match the commit message any more. [...] Content analysis details: (-1.0 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -1.0 ALL_TRUSTED Passed through trusted hosts only via SMTP Cc: openembedded-core@lists.openembedded.org Subject: Re: [PATCHv2 2/5] package.bbclass: show warning when package is trying to provide already provided shlib X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 19 Jan 2014 14:42:42 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Sun, 2014-01-19 at 14:28 +0100, Martin Jansa wrote: > * move read_shlib_providers before registering package as provider > and don't change provider if it already exists, show warning instead This new patch doesn't seem to match the commit message any more. p. > + if s in shlib_provider: > + (old_pkg, old_pkgver) = shlib_provider[s] > + if old_pkg != pkg: > + bb.warn('%s-%s was registered as shlib provider for %s, changing it to %s-%s because it was built later' % (old_pkg, old_pkgver, s, pkg, pkgver)) > + bb.debug(1, 'registering %s-%s as shlib provider for %s' % (pkg, pkgver, s))