From: "Dominic Sacré" <dominic.sacre@gmx.de>
To: Thomas Gleixner <tglx@linutronix.de>,
Steven Rostedt <rostedt@goodmis.org>
Cc: Sebastian Andrzej Siewior <bigeasy@linutronix.de>,
linux-rt-users@vger.kernel.org, Julia Cartwright <julia@ni.com>
Subject: Re: Changing checksums of RT patches
Date: Sat, 11 Mar 2017 00:13:25 +0100 [thread overview]
Message-ID: <067fe46c-dd9f-6bb4-36d5-4ceccfee2f88@gmx.de> (raw)
In-Reply-To: <alpine.DEB.2.20.1702281856310.4732@nanos>
Hi all,
I'm sorry to bring this up again, but it looks like
4.1/older/patch-4.1.38-rt45.patch.gz changed when -rt46 was released, so
the Bitbake recipes that use this patch are now broken again.
On 2017-02-28 19:03, Thomas Gleixner wrote:
> What matters is the content and not the compressed thingy. Again:
>
> - The sha256 only tells you that the download was not corrupted.
>
> - The PGP signature is what protects the content and that does not change
> whether you move it or upload the same thing again.
I don't disagree, but there's currently no support for verifying
downloads by PGP signature in Bitbake; sha256 is the best we've got.
Besides, we're talking about a system for automated builds here.
I would argue that verifying the authenticity of files (e.g. using PGP
signatures) is the responsibility of the person writing the Bitbake recipe.
For those who then use that recipe to build the software, all that
matters is that the download is not corrupted, and that the file being
downloaded is actually the same one that was used by the author of the
recipe.
Of course, it really is the content that matters. But working with
sha256 checksums of the compressed files simplifies both the build
system, and, perhaps more importantly, the process of writing recipes.
IMHO it's bad practice for files to change once they've been released
(with a version number and everything), even if the contents remain the
same. The sha256-based approach seems to be working for thousands of
projects in OpenEmbedded; I don't see any particular reason for patches
on kernel.org to be any different.
By the way, the 4.1.38-rt46 release is currently still missing from the
"older" directory.
Cheers,
Dominic
next prev parent reply other threads:[~2017-03-10 23:14 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-09 9:09 Changing checksums of RT patches Dominic Sacré
2017-02-10 18:54 ` Sebastian Andrzej Siewior
2017-02-13 22:56 ` Dominic Sacré
2017-02-13 23:15 ` Dominic Sacré
2017-02-14 14:18 ` Steven Rostedt
2017-02-14 18:42 ` Dominic Sacré
2017-02-14 18:57 ` Steven Rostedt
2017-02-15 13:38 ` Dominic Sacré
2017-02-15 13:52 ` Sebastian Andrzej Siewior
2017-02-15 14:50 ` Dominic Sacré
2017-02-15 17:15 ` Sebastian Andrzej Siewior
2017-02-28 18:03 ` Thomas Gleixner
2017-02-28 18:30 ` Steven Rostedt
2017-03-10 23:13 ` Dominic Sacré [this message]
2017-03-10 23:19 ` Steven Rostedt
2017-03-11 2:27 ` Julia Cartwright
2017-03-11 3:21 ` Steven Rostedt
2017-03-11 11:14 ` Dominic Sacré
2017-03-11 13:42 ` Steven Rostedt
2017-03-11 9:45 ` Thomas Gleixner
2017-03-11 13:43 ` Steven Rostedt
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=067fe46c-dd9f-6bb4-36d5-4ceccfee2f88@gmx.de \
--to=dominic.sacre@gmx.de \
--cc=bigeasy@linutronix.de \
--cc=julia@ni.com \
--cc=linux-rt-users@vger.kernel.org \
--cc=rostedt@goodmis.org \
--cc=tglx@linutronix.de \
/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;
as well as URLs for NNTP newsgroup(s).