All of lore.kernel.org
 help / color / mirror / Atom feed
From: Martin Jansa <Martin.Jansa@mk.cvut.cz>
To: openembedded-devel@lists.openembedded.org
Subject: Re: checksums.ini - should check only against filename not	uri_src
Date: Sat, 18 Oct 2008 20:30:16 +0200	[thread overview]
Message-ID: <20081018183016.GA4310@jama> (raw)
In-Reply-To: <200810172041.10116.openembedded@haerwu.biz>

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

On Fri, Oct 17, 2008 at 08:41:09PM +0200, Marcin Juszkiewicz wrote:
> If they have different checksums then they are not same.

And they shouldn't be fetched to the same DL_DIR in the first place, so
checksums.ini can define both with pn-filename, and then bitbake would
show an error when the other file would be stored in DL_DIR.

> Your patch lack part which would write simplified checksums.ini file. 
> 
> I think that mirrors informations would be still handy as there were 
> ideas to write wget-like tool for fetching sources which would use own 
> database for informations about source locations. We could make use of 
> format like this:
> 
> [DirectFB-1.1.0.tar.gz]
> url0=http://www.directfb.org/download/DirectFB/DirectFB-1.1.0.tar.gz
> url1=http://www.directfb.org/downloads/Old/DirectFB-1.1.0.tar.gz
> md5=da30fa2748c7501913c8832a83e654ab
> sha256=0ee87256a278079955d78e4f235c15a174afbb507b5434f1966cd3e5f5a3f23a
>  
> And any change to checksums.ini format would require updating 
> contrib/source-checker/ scripts as they are used to check source mirrors 
> integrity and to sort checksums.ini file.

Thanks for comments.

I would like to improve the patch and add parts for scripts too, if you think it
could be used. From discussion on irc it seemed like global checksums.ini
is not a good idea at all, at least because of high probability of merge
conflicts, and that checksums should be moved to .bb files or defined on
per-package files.

Regards,

-- 
uin:136542059                jid:JaMa@jabber.mk.cvut.cz
Jansa Martin                 sip:jamasip@voip.wengo.fr 
JaMa                         

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

  reply	other threads:[~2008-10-18 18:31 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-17 13:17 checksums.ini - should check only against filename not uri_src Martin Jansa
2008-10-17 18:41 ` Marcin Juszkiewicz
2008-10-18 18:30   ` Martin Jansa [this message]
2008-10-18 19:49     ` Leon Woestenberg
2008-10-19 15:55       ` Koen Kooi
2008-10-20 20:21         ` Leon Woestenberg

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=20081018183016.GA4310@jama \
    --to=martin.jansa@mk.cvut.cz \
    --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.