From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from 93-97-173-237.zone5.bethere.co.uk ([93.97.173.237] helo=tim.rpsys.net) by linuxtogo.org with esmtp (Exim 4.72) (envelope-from ) id 1S0DSW-0003Ox-5N for bitbake-devel@lists.openembedded.org; Wed, 22 Feb 2012 15:46:20 +0100 Received: from localhost (localhost [127.0.0.1]) by tim.rpsys.net (8.13.6/8.13.8) with ESMTP id q1MEc0E2029341; Wed, 22 Feb 2012 14:38:00 GMT Received: from tim.rpsys.net ([127.0.0.1]) by localhost (tim.rpsys.net [127.0.0.1]) (amavisd-new, port 10024) with LMTP id 29280-01; Wed, 22 Feb 2012 14:37:56 +0000 (GMT) Received: from [192.168.3.10] ([192.168.3.10]) (authenticated bits=0) by tim.rpsys.net (8.13.6/8.13.8) with ESMTP id q1MEbpYv029335 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Wed, 22 Feb 2012 14:37:52 GMT Message-ID: <1329921469.20261.158.camel@ted> From: Richard Purdie To: Martin Jansa Date: Wed, 22 Feb 2012 14:37:49 +0000 In-Reply-To: <1329915784-24760-1-git-send-email-Martin.Jansa@gmail.com> References: <1329915784-24760-1-git-send-email-Martin.Jansa@gmail.com> X-Mailer: Evolution 3.2.2- Mime-Version: 1.0 X-Virus-Scanned: amavisd-new at rpsys.net Cc: bitbake-devel@lists.openembedded.org Subject: Re: [PATCH] siggen: compare runtaskdeps dictionary even when they have the same size X-BeenThere: bitbake-devel@lists.openembedded.org X-Mailman-Version: 2.1.11 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 22 Feb 2012 14:46:20 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Wed, 2012-02-22 at 14:03 +0100, Martin Jansa wrote: > * otherwise it reports lots of changed checksums just because it compares different tasks > > notice linux-libc-headers_3.1.bb.do_package_write: > Task dependency hash changed from 42acced29debf54d598802474c5e9cbb to f438a54f995df77620d0727d6f4b4ce5 (for linux-libc-headers_3.1.bb.do_package_write and linux-gta04_git.bb.do_deploy) > Task dependency hash changed from 61f8babe1d10c6e7fb1423112bb04e1e to 1b3e21ff106ecfcb7ddf76a1e29537bb (for linux-nokia900-meego_git.bb.do_deploy and linux-gta04_git.bb.do_package_write) > Task dependency hash changed from 512f9d6686d760b318d8b11c8b589226 to 42acced29debf54d598802474c5e9cbb (for linux-nokia900-meego_git.bb.do_package_write and linux-libc-headers_3.1.bb.do_package_write) > Task dependency hash changed from 153e91dfd1d2053fda7b98cc08d4b802 to 92a293bdd8ed234932b87a66025038c5 (for systemd-serialgetty.bb.do_package_write and systemd-serialgetty.bb.do_package_write) > Task dependency hash changed from 730abebf9954794bb440c2f3239f79fe to 413eaebaff27a2fd16f5cf68c1f4ff17 (for systemd_git.bb.do_package_write and systemd_git.bb.do_package_write) > > with this patch: > Dependency on task linux-nokia900-meego_git.bb.do_package_write was added > Dependency on task linux-nokia900-meego_git.bb.do_deploy was added > Dependency on task linux-gta04_git.bb.do_deploy was removed > Dependency on task linux-gta04_git.bb.do_package_write was removed > Hash for dependent task systemd_git.bb.do_package_write changed from 730abebf9954794bb440c2f3239f79fe to 413eaebaff27a2fd16f5cf68c1f4ff17 > Hash for dependent task systemd-serialgetty.bb.do_package_write changed from 153e91dfd1d2053fda7b98cc08d4b802 to 92a293bdd8ed234932b87a66025038c5 > > Signed-off-by: Martin Jansa This basically reverts http://git.openembedded.org/bitbake/commit/?id=7da7dff83ed765c9cde1d7f91ee1b65e49520481 although I guess the elif in the original is no longer present (I can't decide if that is an issue or not). The problem is bitbake only cares about the hashes themselves, it doesn't care about the name attached to them. The name only exists to aid debugging. We were therefore seeing bitbake-diffsigs report differences between sstate files when there was no difference as far as bitbake was concerned, the hashes just had different tasknames associated with them. I strongly disliked having bitbake-diffsigs report a difference where bitbake would treat them the as the same thing. I'd therefore suggest what the code should do is: a) Discount duplicates based on hash value regardless of whether the tasknames match or not. b) Print the differences as per your patch on the data after the removal of the above. So yes, the code needs to be improved to firstly discount duplicates, then give a better summary but the patch needs tweaking. Cheers, Richard