All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] 2017.02.x: update hash file for libnl 3.2.27 patch
@ 2017-06-30 16:50 julien.boibessot at free.fr
  2017-06-30 17:07 ` Arnout Vandecappelle
  2017-07-02 21:55 ` Thomas Petazzoni
  0 siblings, 2 replies; 4+ messages in thread
From: julien.boibessot at free.fr @ 2017-06-30 16:50 UTC (permalink / raw)
  To: buildroot

From: Julien BOIBESSOT <julien.boibessot@armadeus.com>

Corresponding patch is generated on the fly by github. It seems like
generation method has changed, resulting to slightly different patch,
so update BR hash file accordingly.

Signed-off-by: Julien BOIBESSOT <julien.boibessot@armadeus.com>
---
 package/libnl/libnl.hash | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/package/libnl/libnl.hash b/package/libnl/libnl.hash
index eafba4a..9b6b512 100644
--- a/package/libnl/libnl.hash
+++ b/package/libnl/libnl.hash
@@ -1,3 +1,4 @@
 # From https://github.com/thom311/libnl/releases/download/libnl3_2_27/libnl-3.2.27.tar.gz.sha256sum
 sha256	4bbbf92b3c78a90f423cf96260bf419a28b75db8cced47051217a56795f58ec6	libnl-3.2.27.tar.gz
-sha256	b7bb929194eefc56c786a7e1ae5176b54713f9013ccec63760f232742ae80361	3e18948f17148e6a3c4255bdeaaf01ef6081ceeb.patch
+# Locally computed
+sha256	a308873157080f1e2e6693b2a151e15d8dfe77884b073ba3dc9bd42772dd1aef	3e18948f17148e6a3c4255bdeaaf01ef6081ceeb.patch
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] 2017.02.x: update hash file for libnl 3.2.27 patch
  2017-06-30 16:50 [Buildroot] [PATCH] 2017.02.x: update hash file for libnl 3.2.27 patch julien.boibessot at free.fr
@ 2017-06-30 17:07 ` Arnout Vandecappelle
  2017-07-02 21:55 ` Thomas Petazzoni
  1 sibling, 0 replies; 4+ messages in thread
From: Arnout Vandecappelle @ 2017-06-30 17:07 UTC (permalink / raw)
  To: buildroot



On 30-06-17 18:50, julien.boibessot at free.fr wrote:
> From: Julien BOIBESSOT <julien.boibessot@armadeus.com>
> 
> Corresponding patch is generated on the fly by github. It seems like
> generation method has changed, resulting to slightly different patch,
> so update BR hash file accordingly.

 So this is problematic. While the tarballs generated by git(hub) are
reproducible, the patch files are not. Makes sense, since it has abbreviated
sha1s for the tree objects embedded in them and these will need to become longer
over time.

 I see no better solution than to disable hashes for bit-downloaded patches
entirely...

 Regards,
 Arnout

> 
> Signed-off-by: Julien BOIBESSOT <julien.boibessot@armadeus.com>
> ---
>  package/libnl/libnl.hash | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/package/libnl/libnl.hash b/package/libnl/libnl.hash
> index eafba4a..9b6b512 100644
> --- a/package/libnl/libnl.hash
> +++ b/package/libnl/libnl.hash
> @@ -1,3 +1,4 @@
>  # From https://github.com/thom311/libnl/releases/download/libnl3_2_27/libnl-3.2.27.tar.gz.sha256sum
>  sha256	4bbbf92b3c78a90f423cf96260bf419a28b75db8cced47051217a56795f58ec6	libnl-3.2.27.tar.gz
> -sha256	b7bb929194eefc56c786a7e1ae5176b54713f9013ccec63760f232742ae80361	3e18948f17148e6a3c4255bdeaaf01ef6081ceeb.patch
> +# Locally computed
> +sha256	a308873157080f1e2e6693b2a151e15d8dfe77884b073ba3dc9bd42772dd1aef	3e18948f17148e6a3c4255bdeaaf01ef6081ceeb.patch
> 

-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] 2017.02.x: update hash file for libnl 3.2.27 patch
  2017-06-30 16:50 [Buildroot] [PATCH] 2017.02.x: update hash file for libnl 3.2.27 patch julien.boibessot at free.fr
  2017-06-30 17:07 ` Arnout Vandecappelle
@ 2017-07-02 21:55 ` Thomas Petazzoni
  2017-07-02 22:32   ` Peter Korsgaard
  1 sibling, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2017-07-02 21:55 UTC (permalink / raw)
  To: buildroot

Hello,

On Fri, 30 Jun 2017 18:50:36 +0200, julien.boibessot at free.fr wrote:
> From: Julien BOIBESSOT <julien.boibessot@armadeus.com>
> 
> Corresponding patch is generated on the fly by github. It seems like
> generation method has changed, resulting to slightly different patch,
> so update BR hash file accordingly.
> 
> Signed-off-by: Julien BOIBESSOT <julien.boibessot@armadeus.com>
> ---
>  package/libnl/libnl.hash | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

We had some lengthy discussion about it. The problem with your patch is
that it will break things for users of non-updated Buildroot versions.

Currently (without your patch), what happens is that Buildroot tries to
download the patch from Github, fails because the hash is wrong, and
then falls back to sources.buildroot.net which has the patch matching
the hash.

Once we apply your patch, users of the updated version of Buildroot
(with your patch) will be able to get the correct patch from Github.
However, sources.buildroot.net will also automatically fetch the new
patch.

Therefore, users of non-updated Buildroot will try to:

 1. Fetch from Github, which fails because the hash doesn't match

 2. Fetch from sources.buildroot.net, which fails because the hash
    doesn't match either

I.e, with your patch, we break the build for existing Buildroot users.

So, like we are about to do in the master branch for all packages, what
we are going to do here is that we will bring the libnl patch inside
package/libnl/ instead of downloading it from Github.

Peter Korsgaard will work on this soon. In the mean time, I'll mark
your patch as Rejected in patchwork.

Thanks for having reported the issue in the first place!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [PATCH] 2017.02.x: update hash file for libnl 3.2.27 patch
  2017-07-02 21:55 ` Thomas Petazzoni
@ 2017-07-02 22:32   ` Peter Korsgaard
  0 siblings, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2017-07-02 22:32 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

Hi,

 > So, like we are about to do in the master branch for all packages, what
 > we are going to do here is that we will bring the libnl patch inside
 > package/libnl/ instead of downloading it from Github.

 > Peter Korsgaard will work on this soon. In the mean time, I'll mark
 > your patch as Rejected in patchwork.

FYI, I've pushed a patch doing this:

https://git.buildroot.net/buildroot/commit/?h=2017.02.x&id=6826e2ee8a2bc09104185c777a4ff57bc6a78872

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-07-02 22:32 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-30 16:50 [Buildroot] [PATCH] 2017.02.x: update hash file for libnl 3.2.27 patch julien.boibessot at free.fr
2017-06-30 17:07 ` Arnout Vandecappelle
2017-07-02 21:55 ` Thomas Petazzoni
2017-07-02 22:32   ` Peter Korsgaard

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.