All of lore.kernel.org
 help / color / mirror / Atom feed
From: Saul Wold <saul.wold@intel.com>
To: Yu Ke <ke.yu@intel.com>
Cc: poky@yoctoproject.org
Subject: Re: [PATCH 0/1] check if lockfile is writable (bug 606 patch V2)
Date: Wed, 29 Dec 2010 10:01:40 -0800	[thread overview]
Message-ID: <4D1B7784.6080604@intel.com> (raw)
In-Reply-To: <cover.1293616099.git.ke.yu@intel.com>

On 12/29/2010 01:54 AM, Yu Ke wrote:
> I've sent a patch for bug 606 before, which add sanity check for $DL_DIR.
>
> After digging more, the root cause is hidden in bb.utils.lockfile(), so it
> is more general to add check there. hence comes out the V2 patch for bug606.
> please discard my V1 patch.
>
Ke,

I still wonder about this.  If the DL_DIR is read-only, since it is a 
shared environment (as example), when a fetch occurs, it may be for 
different reasons.

A couple of examples are:
1) an upstream fetch was missed (world build, but different hardware)
2) a developer changed the version information

In cases like this we really do want to do the fetch, so how can we 
handle that issue correctly, and not just fail and force the user to 
re-download all the packages (which may not be possible).

One option I can think of is to add a "WRITABLE_DL_DIR" (losy name but 
you get the idea) or something like that which if the DL_DIR is 
read-only could be written to for the lock file and ultimately the fetch 
of the missing upstream package.

Thoughts?

Sau!

> Pull URL: git://git.pokylinux.org/poky-contrib.git
>    Branch: kyu3/bug606-v2
>    Browse: http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=kyu3/bug606-v2
>
> Thanks,
>      Yu Ke<ke.yu@intel.com>
> ---
>
>
> Yu Ke (1):
>    bb.utils: check if lock file is writable, to fix bug 606
>
>   bitbake/lib/bb/utils.py |    4 ++++
>   1 files changed, 4 insertions(+), 0 deletions(-)
>
> _______________________________________________
> poky mailing list
> poky@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/poky
>



  parent reply	other threads:[~2010-12-29 18:01 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-29  9:54 [PATCH 0/1] check if lockfile is writable (bug 606 patch V2) Yu Ke
2010-12-29  9:54 ` [PATCH 1/1] bb.utils: check if lock file is writable, to fix bug 606 Yu Ke
2010-12-29 18:01 ` Saul Wold [this message]
2010-12-29 20:26   ` [PATCH 0/1] check if lockfile is writable (bug 606 patch V2) Richard Purdie
2010-12-30  2:19     ` Yu Ke
2010-12-30  9:29 ` 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=4D1B7784.6080604@intel.com \
    --to=saul.wold@intel.com \
    --cc=ke.yu@intel.com \
    --cc=poky@yoctoproject.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.