All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nikolay Dimitrov <picmaster@mail.bg>
To: Andrei Gherzan <andrei@gherzan.ro>,
	 Paul Eggleton <paul.eggleton@linux.intel.com>
Cc: Yocto Project <yocto@yoctoproject.org>,
	Anders Darander <anders@chargestorm.se>
Subject: Re: [meta-raspberrypi][PATCH] firmware.inc: Fetch a zip instead of cloning a git repo
Date: Fri, 10 Jul 2015 11:47:50 +0300	[thread overview]
Message-ID: <559F86B6.9030100@mail.bg> (raw)
In-Reply-To: <CAK18fxGnkbAtveQND5zCs-KntpPjoZPMzn1_hmHVNKZMWrc1zQ@mail.gmail.com>

Hi Andrei,

On 07/09/2015 11:13 PM, Andrei Gherzan wrote:
> Finally I hop on to this discussion too.
>
> On Mon, Jul 6, 2015 at 12:58 PM, Paul Eggleton
> <paul.eggleton@linux.intel.com <mailto:paul.eggleton@linux.intel.com>>
> wrote:
>
>     On Monday 06 July 2015 12:48:50 Nikolay Dimitrov wrote:
>     > One issue with the regularly changing tarball checksums is that people
>     > start to get used to thes changes (e.g. everything looks like false
>     > positive). Currently the tarball checksums and SCM revisions are
>     > probably the most important tool for builds traceability. If we get
>     > used to think about these checksums as "unreliable", it will be much
>     > easier to miss an important component change, which would otherwise
>     > ring a bell.
>
>     Fully agreed.
>
>     There are a couple of things I think we can do here:
>
>     1) Implement shallow cloning in bitbake's git fetcher as suggested. This
>     shouldn't be too tricky. I've filed a bug to track this:
>
>     https://bugzilla.yoctoproject.org/show_bug.cgi?id=7958
>
>     (Richard is the default assignee, but anyone could potentially work
>     on this).
>
>
> This should be the fix that would really fix the issue. And would be a
> useful feature for many other BSPs / layers out there.
>
>     2) In the mean time we could consider upload git mirror tarballs to
>     a source
>     mirror that gets enabled through meta-raspberrypi (would need to be via
>     PREMIRRORS to actually solve the issue). This has the advantage that it
>     wouldn't require any changes to the kernel recipe itself, but new
>     tarballs
>     would of course need to be uploaded every time SRCREV is changed in the
>     recipe.
>
>
> And until 1) is done, we can have a premirror. Paul, can you upload a
> tarball? Can I help you with anything for having this up? After we have
> this, can we force premirrors when using a specific layer? Was thinking
> of forcing it by adding PREMIRRORS to layer.conf.

I don't think this is a good move. The current solution is already
working properly, although with slower-than-ideal download speed.

Prepackaged tarballs will require constant manpower for supporting,
and it's probably better to be invested into looking for a better
solution.

>
> Using github snapshots is not a good idea. Most of the issues you guys
> pointed out above I experienced as well. In my opinion we should combine
> Paul's solutions in order to address this problem.
>
> One more thing. Given the fact the the repository we are talking about
> is not under our control, we shouldn't rely on releases or other things
> from the remote repository.
>
> Andrei
>
>

Regards,
Nikolay


      reply	other threads:[~2015-07-10  8:47 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-26  4:16 [meta-raspberrypi][PATCH] firmware.inc: Fetch a zip instead of cloning a git repo Jon Szymaniak
2015-06-26  8:31 ` Burton, Ross
2015-06-26  9:07   ` Gary Thomas
2015-06-26  9:09     ` Burton, Ross
2015-06-26 14:16   ` Jon Szymaniak
2015-06-26 14:19     ` Burton, Ross
2015-06-26 14:42       ` Jon Szymaniak
2015-06-26 14:46         ` Burton, Ross
2015-07-05 19:19     ` Petter Mabäcker
2015-07-06  5:19   ` Clemens Lang
2015-07-06  8:40     ` Anders Darander
2015-07-06  9:48       ` Nikolay Dimitrov
2015-07-06 10:58         ` Paul Eggleton
2015-07-09 20:13           ` Andrei Gherzan
2015-07-10  8:47             ` Nikolay Dimitrov [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=559F86B6.9030100@mail.bg \
    --to=picmaster@mail.bg \
    --cc=anders@chargestorm.se \
    --cc=andrei@gherzan.ro \
    --cc=paul.eggleton@linux.intel.com \
    --cc=yocto@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.