Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Peter Kjellerstedt <peter.kjellerstedt@axis.com>,
	openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 0/2] Allow oe-pkgdata-util package-info to display more info
Date: Tue, 16 May 2017 14:14:49 +0100	[thread overview]
Message-ID: <1494940489.27342.30.camel@linuxfoundation.org> (raw)
In-Reply-To: <cover.1494932131.git.pkj@axis.com>

On Tue, 2017-05-16 at 12:56 +0200, Peter Kjellerstedt wrote:
> After a build in our autobuilder, we use `oe-pkgdata-util package-
> info
> -f package.manifest` to store a file with information about each
> installed package. This is typically used to compare builds later
> on. Since not every difference to a package is explained by its
> version, we have found it beneficial to also include the SRC_URI in
> the generated file.
> 
> This patch set adds SRC_URI to the pkgdata that is stored for each
> package, and adds a new option to oe-pkgdata-util package-info,
> -e <var>, that can be used to display extra variables from the
> pkgdata.

I'm going to say no to this.

The reason is that pkgdata is not really about collecting up all build
information. If you want to know how two different builds differ, you'd
use the sigdata files. If I take this patch, more will follow where you
find some new difference you want to track and there are other
mechanisms I'd suggest (buildhistory and siginfo for starters). I don't
want to turn the pkgdata files into something they're not.

Cheers,

Richard


  parent reply	other threads:[~2017-05-16 13:14 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-16 10:56 [PATCH 0/2] Allow oe-pkgdata-util package-info to display more info Peter Kjellerstedt
2017-05-16 10:56 ` [PATCH 1/2] package.bbclass: Add SRC_URI to pkgdata Peter Kjellerstedt
2017-05-16 10:56 ` [PATCH 2/2] oe-pkgdata-util: package-info: Allow extra variables to be displayed Peter Kjellerstedt
2017-05-16 13:14 ` Richard Purdie [this message]
2017-05-16 16:29   ` [PATCH 0/2] Allow oe-pkgdata-util package-info to display more info Peter Kjellerstedt
2017-05-23 13:38     ` Peter Kjellerstedt
2017-05-24 22:28     ` Richard Purdie

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1494940489.27342.30.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=peter.kjellerstedt@axis.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox