All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Smith <msmith@cbnco.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: [PATCH] bitbake.conf, freeze.inc: Add version lockdown implementation and use it by	default.
Date: Fri, 15 May 2009 11:55:23 -0400	[thread overview]
Message-ID: <4A0D906B.7060908@cbnco.com> (raw)
In-Reply-To: <1242247998-30548-1-git-send-email-clarson@mvista.com>

Chris Larson wrote:
> For each recipe which completes a task successfully, this emits the current
> version into ${TMPDIR}/versions.conf as a PREFERRED_VERSION line.
> ${TMPDIR}/versions.conf and conf/versions.conf are automatically included,
> in that order, in subsequent builds, to provide more deterinistic builds by
> default, and to let the user make the lockdown persist via a simple cp
> command.

Hi Chris,

If a distro doesn't lock down a particular version of a package, I 
figure after a "git pull" I'll be building the latest version of 
everything. At least, if I do a "bitbake -c clean somepkg" followed by a 
"bitbake somepkg", I'll get the latest version of somepkg.

If I understand the patch correctly, that won't happen anymore unless I 
remember to remove tmp/versions.conf. It adds an extra step to the 
process when one developer is using another's package.

I can see it leading to confusion between developers. Any time there's a 
multi-step process, humans will find a way to forget a step. :)

What's the problem the versions.conf is meant to solve? I would think if 
I'm doing a git pull and then rebuilding a package, it's because I want 
to get the latest version, but I think I'm missing something here.

Mike



      parent reply	other threads:[~2009-05-15 16:02 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-13 20:53 [PATCH] bitbake.conf, freeze.inc: Add version lockdown implementation and use it by default Chris Larson
2009-05-13 21:25 ` Denys Dmytriyenko
2009-05-18 15:48   ` Denys Dmytriyenko
2009-05-18 18:00     ` Christopher Larson
2009-05-18 18:58       ` Denys Dmytriyenko
2009-05-15  3:56 ` Mike Westerhof (mwester)
2009-05-15  7:33   ` Andrea Adami
2009-05-15 15:09     ` Chris Larson
2009-05-15 21:07       ` Andrea Adami
2009-05-18 15:44         ` Denys Dmytriyenko
2009-05-15  8:24   ` Graeme Gregory
2009-05-15 15:08   ` Chris Larson
2009-05-15 16:02     ` Tom Rini
2009-05-15 15:55 ` Michael Smith [this message]

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=4A0D906B.7060908@cbnco.com \
    --to=msmith@cbnco.com \
    --cc=openembedded-devel@lists.openembedded.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.