Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Martin Jansa <martin.jansa@gmail.com>
To: Trevor Woerner <twoerner@gmail.com>
Cc: Paul Eggleton <paul.eggleton@linux.intel.com>,
	openembedded-core@lists.openembedded.org
Subject: Re: [PATCH] metadata-revs: provide more information
Date: Mon, 14 Mar 2016 11:15:14 +0100	[thread overview]
Message-ID: <20160314101514.GA2560@jama> (raw)
In-Reply-To: <56E5E180.30408@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 1444 bytes --]

On Sun, Mar 13, 2016 at 05:54:08PM -0400, Trevor Woerner wrote:
> On 03/13/16 16:53, Paul Eggleton wrote:
> > On Sun, 13 Mar 2016 16:42:41 Trevor Woerner wrote:
> >> That's the problem I'm trying to solve: how can I easily keep all the
> >> information required to reproduce this build, exactly. The whole "two
> >> meta's" thing was just a speed bump.
> >
> > Right, understood, and it does make sense. I'm convinced there is an
> > underlying problem to fix that this just papers over though.
> 
> Okay, I think I see your point, and I agree. There is a layer clone 
> problem that should be addressed.
> 
> But again, I think this is a speed bump.
> 
> My underlying point is that we're purposefully leaving out information 
> that is vital to reproducible builds.

FWIW: I think this information also useful (even as separate file in
buildhistory).

In most builds it shouldn't change very often (except the revision) so
the diff shouldn't grow significantly.

There are many reasons why people use different forks of some layers,
some of them are valid (like using own fork to verify some change before
submitting it upstream and showing buildhistory diff how you tested it),
some of them are sad, but still needed _to be shown somewhere_ (like
using fork with important fix, when upstream layer is slow or refusing
to accept the fix).

Regards,
-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]

  parent reply	other threads:[~2016-03-14 10:11 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-13  2:35 [PATCH] metadata-revs: provide more information Trevor Woerner
2016-03-13 19:26 ` Paul Eggleton
2016-03-13 20:42   ` Trevor Woerner
2016-03-13 20:53     ` Paul Eggleton
2016-03-13 21:54       ` Trevor Woerner
2016-03-13 23:03         ` Paul Eggleton
2016-03-14 10:15         ` Martin Jansa [this message]
2016-03-14 10:26           ` Jeremy Rosen
2016-03-13 20:53   ` Trevor Woerner

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=20160314101514.GA2560@jama \
    --to=martin.jansa@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=paul.eggleton@linux.intel.com \
    --cc=twoerner@gmail.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