* [Buildroot] [PATCH 1/7 v5] package/gcc: add hashes
2015-04-19 12:44 [Buildroot] [PATCH 0/7 v5] toolchain: better handle hashes (branch yem/dl-hash-toolchains) Yann E. MORIN
@ 2015-04-19 12:44 ` Yann E. MORIN
2015-04-19 16:41 ` Arnout Vandecappelle
2015-04-19 12:44 ` [Buildroot] [PATCH 2/7 v5] package/binutils: " Yann E. MORIN
` (6 subsequent siblings)
7 siblings, 1 reply; 28+ messages in thread
From: Yann E. MORIN @ 2015-04-19 12:44 UTC (permalink / raw)
To: buildroot
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Arnout Vandecappelle <arnout@mind.be>
---
Changes v4 -> v5:
- use alternate upstream as source of hashes (Arnout)
---
package/gcc/gcc-final/gcc-final.hash | 1 +
package/gcc/gcc-initial/gcc-initial.hash | 1 +
package/gcc/gcc.hash | 10 ++++++++++
3 files changed, 12 insertions(+)
create mode 120000 package/gcc/gcc-final/gcc-final.hash
create mode 120000 package/gcc/gcc-initial/gcc-initial.hash
create mode 100644 package/gcc/gcc.hash
diff --git a/package/gcc/gcc-final/gcc-final.hash b/package/gcc/gcc-final/gcc-final.hash
new file mode 120000
index 0000000..7ac9361
--- /dev/null
+++ b/package/gcc/gcc-final/gcc-final.hash
@@ -0,0 +1 @@
+../gcc.hash
\ No newline at end of file
diff --git a/package/gcc/gcc-initial/gcc-initial.hash b/package/gcc/gcc-initial/gcc-initial.hash
new file mode 120000
index 0000000..7ac9361
--- /dev/null
+++ b/package/gcc/gcc-initial/gcc-initial.hash
@@ -0,0 +1 @@
+../gcc.hash
\ No newline at end of file
diff --git a/package/gcc/gcc.hash b/package/gcc/gcc.hash
new file mode 100644
index 0000000..5eec97f
--- /dev/null
+++ b/package/gcc/gcc.hash
@@ -0,0 +1,10 @@
+# From ftp://gcc.gnu.org/pub/gcc/releases/gcc-4.5.4/sha512.sum
+sha512 78696b287d46aacd6f150920da376ea32f58ad9f0dafd2d3b7fa6dbdd8dd7afe659108d1dfaa5807617fc5248719f50b39c37ade0173ea5b9fec2091f4656115 gcc-4.5.4.tar.bz2
+# From ftp://gcc.gnu.org/pub/gcc/releases/gcc-4.7.4/sha512.sum
+sha512 dfcb737073191e628231031a3571ec77ee760a59377630f4a6e4fdfa66f9ddad39fde47e3f0f227eb43cdf90e0d34cde5abdc9ac892c1e111a911062a66c9189 gcc-4.7.4.tar.bz2
+# From ftp://gcc.gnu.org/pub/gcc/releases/gcc-4.8.4/sha512.sum
+sha512 b4c84ebd754c026029d706ef52c4242df4dcadc6a22cc5669b3aa37ac9dc25bc9cd9651fda83fb24dd5d17b1710e6865a35bb6c5cd9b95d0971717453935ae8e gcc-4.8.4.tar.bz2
+# From ftp://gcc.gnu.org/pub/gcc/releases/gcc-4.9.2/sha512.sum
+sha512 e96cba06d572dbe7f382a2b00fd9297dcf9f9219d46a9ad0bd904dc36d5e7db3335e422640b79a27fed6fde91b0731732b94a0e2fbcd47344d6779f8ba1f4f9c gcc-4.9.2.tar.bz2
+# No hash for the ARC variant, comes from the github-helper:
+none xxx gcc-arc-2014.12.tar.gz
--
1.9.1
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 0/7 v5] toolchain: better handle hashes (branch yem/dl-hash-toolchains)
@ 2015-04-19 12:44 Yann E. MORIN
2015-04-19 12:44 ` [Buildroot] [PATCH 1/7 v5] package/gcc: add hashes Yann E. MORIN
` (7 more replies)
0 siblings, 8 replies; 28+ messages in thread
From: Yann E. MORIN @ 2015-04-19 12:44 UTC (permalink / raw)
To: buildroot
Hello All!
This series makes it so we can download custom external toolchains, and
still have mandatory checks of hashes.
It also adds the long-awaited-for hashes for gcc and binutils.
Changes v4 -> v5:
- drop patch #8, that errored-out when an excluded file did have
one or more associated hashes (Arnout)
- typoes (Arnout)
Changes v3 -> v4:
- add missing fake-hash for uClibc eXtensa git snapshot; add uClibc
daily snapshots to exclusion list (Arnout)
- drop using a magic value; use a list of excluded files (Arnout)
Changes v2 -> v3:
- add hashes for gcc and binutils
- fix a few more typoes
Changes v1 -> v2:
- fix a few typoes
Regards,
Yann E. MORIN.
The following changes since commit 81e22ebd5ee3b9aa36fcbc86602a8fc5bfe87c17:
package/python-m2crypto: add hash (2015-04-19 14:25:28 +0200)
are available in the git repository at:
git://git.busybox.net/~ymorin/git/buildroot yem/dl-hash-toolchains
for you to fetch changes up to 92db9d9c24d6a53fa0db2aa7b3bff1456498d690:
support/download: restore mandatory check of hashes (2015-04-19 14:35:03 +0200)
----------------------------------------------------------------
Yann E. MORIN (7):
package/gcc: add hashes
package/binutils: add hashes
package/uclibc: add missing hash for eXtensa
support/download: add possibility to not fail on missing hash
toolchain/external: ignore missing hash for custom downloaded toolchain
package/uclibc: ignore missing hash for snapshots
support/download: restore mandatory check of hashes
package/binutils/binutils.hash | 7 +++++++
package/gcc/gcc-final/gcc-final.hash | 1 +
package/gcc/gcc-initial/gcc-initial.hash | 1 +
package/gcc/gcc.hash | 10 ++++++++++
package/pkg-download.mk | 3 +++
package/uclibc/uclibc.hash | 3 +++
package/uclibc/uclibc.mk | 1 +
support/download/check-hash | 8 +++++++-
toolchain/toolchain-external/toolchain-external.mk | 2 ++
9 files changed, 35 insertions(+), 1 deletion(-)
create mode 100644 package/binutils/binutils.hash
create mode 120000 package/gcc/gcc-final/gcc-final.hash
create mode 120000 package/gcc/gcc-initial/gcc-initial.hash
create mode 100644 package/gcc/gcc.hash
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 2/7 v5] package/binutils: add hashes
2015-04-19 12:44 [Buildroot] [PATCH 0/7 v5] toolchain: better handle hashes (branch yem/dl-hash-toolchains) Yann E. MORIN
2015-04-19 12:44 ` [Buildroot] [PATCH 1/7 v5] package/gcc: add hashes Yann E. MORIN
@ 2015-04-19 12:44 ` Yann E. MORIN
2015-04-19 16:42 ` Arnout Vandecappelle
2015-04-19 12:44 ` [Buildroot] [PATCH 3/7 v5] package/uclibc: add missing hash for eXtensa Yann E. MORIN
` (5 subsequent siblings)
7 siblings, 1 reply; 28+ messages in thread
From: Yann E. MORIN @ 2015-04-19 12:44 UTC (permalink / raw)
To: buildroot
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Arnout Vandecappelle <arnout@mind.be>
---
Changes v4 -> v5:
- use alternate upstream as source for hashes (Arnout)
---
package/binutils/binutils.hash | 7 +++++++
1 file changed, 7 insertions(+)
create mode 100644 package/binutils/binutils.hash
diff --git a/package/binutils/binutils.hash b/package/binutils/binutils.hash
new file mode 100644
index 0000000..028fc10
--- /dev/null
+++ b/package/binutils/binutils.hash
@@ -0,0 +1,7 @@
+# From ftp://gcc.gnu.org/pub/binutils/releases/sha512.sum
+sha512 ffe8ef263ef99183e8cc823fe8487ff7d0f7bf9a8efd2853b5f4636aca0023850d13de4eac7d77a5f69413d8a50e6f95bb14569be53df86c0bce38034525ab74 binutils-2.22.tar.bz2
+sha512 dec753bbba008f1526b89cf1bd85feba78f362f5333ffdf93953fd131eb755976dec82a0a4ba38c43d2434da007137780cfe674de5414be5cf7ce7fbc6af6d16 binutils-2.23.2.tar.bz2
+sha512 5ec95ad47d49b12c4558a8db0ca2109d3ee1955e3776057f3330c4506f8f4d1cf5e505fbf8a16b98403a0fcdeaaf986fe0a22be6456247dbdace63ce1f776b12 binutils-2.24.tar.bz2
+sha512 49438970642f379978aba988e3d2e7b63dab6fd0b46d91f4100f083e87a84d242089daca58572245fe620457a77bbbe2d78588ff3fb79314e500d1a39bf96f30 binutils-2.25.tar.bz2
+# No hash for the ARC variant, comes from the github-helper:
+none xxx binutils-arc-2014.12.tar.gz
--
1.9.1
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 3/7 v5] package/uclibc: add missing hash for eXtensa
2015-04-19 12:44 [Buildroot] [PATCH 0/7 v5] toolchain: better handle hashes (branch yem/dl-hash-toolchains) Yann E. MORIN
2015-04-19 12:44 ` [Buildroot] [PATCH 1/7 v5] package/gcc: add hashes Yann E. MORIN
2015-04-19 12:44 ` [Buildroot] [PATCH 2/7 v5] package/binutils: " Yann E. MORIN
@ 2015-04-19 12:44 ` Yann E. MORIN
2015-04-19 16:47 ` Arnout Vandecappelle
2015-04-19 12:44 ` [Buildroot] [PATCH 4/7 v5] support/download: add possibility to not fail on missing hash Yann E. MORIN
` (4 subsequent siblings)
7 siblings, 1 reply; 28+ messages in thread
From: Yann E. MORIN @ 2015-04-19 12:44 UTC (permalink / raw)
To: buildroot
This is a git snapshot, we can't have a hash for it.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Arnout Vandecappelle <arnout@mind.be>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
package/uclibc/uclibc.hash | 3 +++
1 file changed, 3 insertions(+)
diff --git a/package/uclibc/uclibc.hash b/package/uclibc/uclibc.hash
index 69cc103..971ab26 100644
--- a/package/uclibc/uclibc.hash
+++ b/package/uclibc/uclibc.hash
@@ -6,3 +6,6 @@ sha256 0a9085a11615296cfe7783f16ff29073393eb330749947adece0eeddb31705a4 uClibc-a
# From http://www.uclibc-ng.org/trac/
sha256 e561040293ba4f21dcf3285ca1f683b5a8493eb0ffd3b3e495122cfde5a64a0e uClibc-ng-1.0.1.tar.xz
+
+# No hash for this git snapshot
+none xxx uClibc-7bf35c8b7d4a1f97174eb49f47f33946b282114c.tar.gz
--
1.9.1
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 4/7 v5] support/download: add possibility to not fail on missing hash
2015-04-19 12:44 [Buildroot] [PATCH 0/7 v5] toolchain: better handle hashes (branch yem/dl-hash-toolchains) Yann E. MORIN
` (2 preceding siblings ...)
2015-04-19 12:44 ` [Buildroot] [PATCH 3/7 v5] package/uclibc: add missing hash for eXtensa Yann E. MORIN
@ 2015-04-19 12:44 ` Yann E. MORIN
2015-04-19 17:41 ` Arnout Vandecappelle
2015-04-19 12:44 ` [Buildroot] [PATCH 5/7 v5] toolchain/external: ignore missing hash for custom downloaded toolchain Yann E. MORIN
` (3 subsequent siblings)
7 siblings, 1 reply; 28+ messages in thread
From: Yann E. MORIN @ 2015-04-19 12:44 UTC (permalink / raw)
To: buildroot
In very constrained cases, it might be needed to not fail if a hash is
missing. This is notably the case for custom external toolchains to be
downloaded, because we do have a .hash file for external toolchains,
but we obviously can not have hashes for all existing custom toolchains
(he, "custom"!).
So, add a way to avoid failing in that case.
From the Makefile, we export the list of files for which not to check
the hash. Then, from the check-hash script, if no check was done, and
the file we were trying to match in in this exclusion list, we just exit
without error.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Arnout Vandecappelle <arnout@mind.be>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
Changes v4 -> v5:
- micro-optimisation, use case-esac instead of a for-loop (Arnout)
- typoes (Arnout)
Changes v3 -> v4:
- drop the magic value, use a list of excluded files (Arnout)
Changes v1 -> v2:
- fix typoes in commit log
---
package/pkg-download.mk | 3 +++
support/download/check-hash | 6 ++++++
2 files changed, 9 insertions(+)
diff --git a/package/pkg-download.mk b/package/pkg-download.mk
index e274712..1fc23c4 100644
--- a/package/pkg-download.mk
+++ b/package/pkg-download.mk
@@ -60,6 +60,9 @@ domainseparator = $(if $(1),$(1),/)
# github(user,package,version): returns site of GitHub repository
github = https://github.com/$(1)/$(2)/archive/$(3)
+# Expressly do not check hashes for those files
+export BR_NO_CHECK_HASH_FOR
+
################################################################################
# The DOWNLOAD_* helpers are in charge of getting a working copy
# of the source repository for their corresponding SCM,
diff --git a/support/download/check-hash b/support/download/check-hash
index 678a7ef..1280322 100755
--- a/support/download/check-hash
+++ b/support/download/check-hash
@@ -99,6 +99,12 @@ while read t h f; do
done <"${h_file}"
if [ ${nb_checks} -eq 0 ]; then
+ case " ${BR_NO_CHECK_HASH_FOR} " in
+ " ${base} ")
+ # File explicitly has no hash
+ exit 0
+ ;;
+ esac
printf "ERROR: No hash found for %s\n" "${base}" >&2
exit 0
fi
--
1.9.1
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 5/7 v5] toolchain/external: ignore missing hash for custom downloaded toolchain
2015-04-19 12:44 [Buildroot] [PATCH 0/7 v5] toolchain: better handle hashes (branch yem/dl-hash-toolchains) Yann E. MORIN
` (3 preceding siblings ...)
2015-04-19 12:44 ` [Buildroot] [PATCH 4/7 v5] support/download: add possibility to not fail on missing hash Yann E. MORIN
@ 2015-04-19 12:44 ` Yann E. MORIN
2015-04-19 12:44 ` [Buildroot] [PATCH 6/7 v5] package/uclibc: ignore missing hash for snapshots Yann E. MORIN
` (2 subsequent siblings)
7 siblings, 0 replies; 28+ messages in thread
From: Yann E. MORIN @ 2015-04-19 12:44 UTC (permalink / raw)
To: buildroot
We will *always* be missing a hash file for custom external toolchains
that are downloaded.
So, just ignore that failure.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Arnout Vandecappelle <arnout@mind.be>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
Chamges v3 -> v4:
- drop magic value, append to the exclusion list (Arnout)
Changes v1 -> v2:
- fix typoes in title
---
toolchain/toolchain-external/toolchain-external.mk | 2 ++
1 file changed, 2 insertions(+)
diff --git a/toolchain/toolchain-external/toolchain-external.mk b/toolchain/toolchain-external/toolchain-external.mk
index c0429bb..09b16d1 100644
--- a/toolchain/toolchain-external/toolchain-external.mk
+++ b/toolchain/toolchain-external/toolchain-external.mk
@@ -400,6 +400,8 @@ else
# Custom toolchain
TOOLCHAIN_EXTERNAL_SITE = $(dir $(call qstrip,$(BR2_TOOLCHAIN_EXTERNAL_URL)))
TOOLCHAIN_EXTERNAL_SOURCE = $(notdir $(call qstrip,$(BR2_TOOLCHAIN_EXTERNAL_URL)))
+# We can't check hashes for custom downloaded toolchains
+BR_NO_CHECK_HASH_FOR += $(TOOLCHAIN_EXTERNAL_SOURCE)
endif
# In fact, we don't need to download the toolchain, since it is already
--
1.9.1
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 6/7 v5] package/uclibc: ignore missing hash for snapshots
2015-04-19 12:44 [Buildroot] [PATCH 0/7 v5] toolchain: better handle hashes (branch yem/dl-hash-toolchains) Yann E. MORIN
` (4 preceding siblings ...)
2015-04-19 12:44 ` [Buildroot] [PATCH 5/7 v5] toolchain/external: ignore missing hash for custom downloaded toolchain Yann E. MORIN
@ 2015-04-19 12:44 ` Yann E. MORIN
2015-04-19 17:46 ` Arnout Vandecappelle
2015-04-19 12:45 ` [Buildroot] [PATCH 7/7 v5] support/download: restore mandatory check of hashes Yann E. MORIN
2015-04-19 17:55 ` [Buildroot] [PATCH v6 1/8] package/gcc: add hashes Arnout Vandecappelle
7 siblings, 1 reply; 28+ messages in thread
From: Yann E. MORIN @ 2015-04-19 12:44 UTC (permalink / raw)
To: buildroot
We can't have a hash for snapshots, so add them to the exclusion list.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Arnout Vandecappelle <arnout@mind.be>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
package/uclibc/uclibc.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/package/uclibc/uclibc.mk b/package/uclibc/uclibc.mk
index 72a46d9..1ec8b10 100644
--- a/package/uclibc/uclibc.mk
+++ b/package/uclibc/uclibc.mk
@@ -11,6 +11,7 @@ UCLIBC_LICENSE_FILES = COPYING.LIB
ifeq ($(BR2_UCLIBC_VERSION_SNAPSHOT),y)
UCLIBC_SITE = http://www.uclibc.org/downloads/snapshots
+BR_NO_CHECK_HASH_FOR += uClibc-$(UCLIBC_VERSION).tar.xz
else ifeq ($(BR2_UCLIBC_VERSION_NG),y)
UCLIBC_SITE = http://downloads.uclibc-ng.org/releases/$(UCLIBC_VERSION)
UCLIBC_SOURCE = uClibc-ng-$(UCLIBC_VERSION).tar.xz
--
1.9.1
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 7/7 v5] support/download: restore mandatory check of hashes
2015-04-19 12:44 [Buildroot] [PATCH 0/7 v5] toolchain: better handle hashes (branch yem/dl-hash-toolchains) Yann E. MORIN
` (5 preceding siblings ...)
2015-04-19 12:44 ` [Buildroot] [PATCH 6/7 v5] package/uclibc: ignore missing hash for snapshots Yann E. MORIN
@ 2015-04-19 12:45 ` Yann E. MORIN
2015-04-19 17:54 ` Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 1/8] package/gcc: add hashes Arnout Vandecappelle
7 siblings, 1 reply; 28+ messages in thread
From: Yann E. MORIN @ 2015-04-19 12:45 UTC (permalink / raw)
To: buildroot
Now that custom external toolchains to be downloaded proprely instruct
to not fail on a missing hash, restore the mandatory hash check for
everything else.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Arnout Vandecappelle <arnout@mind.be>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
support/download/check-hash | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/support/download/check-hash b/support/download/check-hash
index 1280322..383bd5c 100755
--- a/support/download/check-hash
+++ b/support/download/check-hash
@@ -106,5 +106,5 @@ if [ ${nb_checks} -eq 0 ]; then
;;
esac
printf "ERROR: No hash found for %s\n" "${base}" >&2
- exit 0
+ exit 3
fi
--
1.9.1
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 1/7 v5] package/gcc: add hashes
2015-04-19 12:44 ` [Buildroot] [PATCH 1/7 v5] package/gcc: add hashes Yann E. MORIN
@ 2015-04-19 16:41 ` Arnout Vandecappelle
0 siblings, 0 replies; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 16:41 UTC (permalink / raw)
To: buildroot
On 19/04/15 14:44, Yann E. MORIN wrote:
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
> Cc: Arnout Vandecappelle <arnout@mind.be>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Regards,
Arnout
>
> ---
> Changes v4 -> v5:
> - use alternate upstream as source of hashes (Arnout)
> ---
> package/gcc/gcc-final/gcc-final.hash | 1 +
> package/gcc/gcc-initial/gcc-initial.hash | 1 +
> package/gcc/gcc.hash | 10 ++++++++++
> 3 files changed, 12 insertions(+)
> create mode 120000 package/gcc/gcc-final/gcc-final.hash
> create mode 120000 package/gcc/gcc-initial/gcc-initial.hash
> create mode 100644 package/gcc/gcc.hash
>
> diff --git a/package/gcc/gcc-final/gcc-final.hash b/package/gcc/gcc-final/gcc-final.hash
> new file mode 120000
> index 0000000..7ac9361
> --- /dev/null
> +++ b/package/gcc/gcc-final/gcc-final.hash
> @@ -0,0 +1 @@
> +../gcc.hash
> \ No newline at end of file
> diff --git a/package/gcc/gcc-initial/gcc-initial.hash b/package/gcc/gcc-initial/gcc-initial.hash
> new file mode 120000
> index 0000000..7ac9361
> --- /dev/null
> +++ b/package/gcc/gcc-initial/gcc-initial.hash
> @@ -0,0 +1 @@
> +../gcc.hash
> \ No newline at end of file
> diff --git a/package/gcc/gcc.hash b/package/gcc/gcc.hash
> new file mode 100644
> index 0000000..5eec97f
> --- /dev/null
> +++ b/package/gcc/gcc.hash
> @@ -0,0 +1,10 @@
> +# From ftp://gcc.gnu.org/pub/gcc/releases/gcc-4.5.4/sha512.sum
> +sha512 78696b287d46aacd6f150920da376ea32f58ad9f0dafd2d3b7fa6dbdd8dd7afe659108d1dfaa5807617fc5248719f50b39c37ade0173ea5b9fec2091f4656115 gcc-4.5.4.tar.bz2
> +# From ftp://gcc.gnu.org/pub/gcc/releases/gcc-4.7.4/sha512.sum
> +sha512 dfcb737073191e628231031a3571ec77ee760a59377630f4a6e4fdfa66f9ddad39fde47e3f0f227eb43cdf90e0d34cde5abdc9ac892c1e111a911062a66c9189 gcc-4.7.4.tar.bz2
> +# From ftp://gcc.gnu.org/pub/gcc/releases/gcc-4.8.4/sha512.sum
> +sha512 b4c84ebd754c026029d706ef52c4242df4dcadc6a22cc5669b3aa37ac9dc25bc9cd9651fda83fb24dd5d17b1710e6865a35bb6c5cd9b95d0971717453935ae8e gcc-4.8.4.tar.bz2
> +# From ftp://gcc.gnu.org/pub/gcc/releases/gcc-4.9.2/sha512.sum
> +sha512 e96cba06d572dbe7f382a2b00fd9297dcf9f9219d46a9ad0bd904dc36d5e7db3335e422640b79a27fed6fde91b0731732b94a0e2fbcd47344d6779f8ba1f4f9c gcc-4.9.2.tar.bz2
> +# No hash for the ARC variant, comes from the github-helper:
> +none xxx gcc-arc-2014.12.tar.gz
>
--
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: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 2/7 v5] package/binutils: add hashes
2015-04-19 12:44 ` [Buildroot] [PATCH 2/7 v5] package/binutils: " Yann E. MORIN
@ 2015-04-19 16:42 ` Arnout Vandecappelle
0 siblings, 0 replies; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 16:42 UTC (permalink / raw)
To: buildroot
On 19/04/15 14:44, Yann E. MORIN wrote:
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
> Cc: Arnout Vandecappelle <arnout@mind.be>
Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Regards,
Arnout
> ---
> Changes v4 -> v5:
> - use alternate upstream as source for hashes (Arnout)
> ---
> package/binutils/binutils.hash | 7 +++++++
> 1 file changed, 7 insertions(+)
> create mode 100644 package/binutils/binutils.hash
>
> diff --git a/package/binutils/binutils.hash b/package/binutils/binutils.hash
> new file mode 100644
> index 0000000..028fc10
> --- /dev/null
> +++ b/package/binutils/binutils.hash
> @@ -0,0 +1,7 @@
> +# From ftp://gcc.gnu.org/pub/binutils/releases/sha512.sum
> +sha512 ffe8ef263ef99183e8cc823fe8487ff7d0f7bf9a8efd2853b5f4636aca0023850d13de4eac7d77a5f69413d8a50e6f95bb14569be53df86c0bce38034525ab74 binutils-2.22.tar.bz2
> +sha512 dec753bbba008f1526b89cf1bd85feba78f362f5333ffdf93953fd131eb755976dec82a0a4ba38c43d2434da007137780cfe674de5414be5cf7ce7fbc6af6d16 binutils-2.23.2.tar.bz2
> +sha512 5ec95ad47d49b12c4558a8db0ca2109d3ee1955e3776057f3330c4506f8f4d1cf5e505fbf8a16b98403a0fcdeaaf986fe0a22be6456247dbdace63ce1f776b12 binutils-2.24.tar.bz2
> +sha512 49438970642f379978aba988e3d2e7b63dab6fd0b46d91f4100f083e87a84d242089daca58572245fe620457a77bbbe2d78588ff3fb79314e500d1a39bf96f30 binutils-2.25.tar.bz2
> +# No hash for the ARC variant, comes from the github-helper:
> +none xxx binutils-arc-2014.12.tar.gz
>
--
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: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 3/7 v5] package/uclibc: add missing hash for eXtensa
2015-04-19 12:44 ` [Buildroot] [PATCH 3/7 v5] package/uclibc: add missing hash for eXtensa Yann E. MORIN
@ 2015-04-19 16:47 ` Arnout Vandecappelle
2015-04-19 16:57 ` [Buildroot] [PATCH] uclibc: update hash for arc Arnout Vandecappelle
2015-04-19 17:03 ` [Buildroot] [PATCH 3/7 v5] package/uclibc: add missing hash for eXtensa Yann E. MORIN
0 siblings, 2 replies; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 16:47 UTC (permalink / raw)
To: buildroot
On 19/04/15 14:44, Yann E. MORIN wrote:
> This is a git snapshot, we can't have a hash for it.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
> Cc: Arnout Vandecappelle <arnout@mind.be>
> Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
My reviewed-by still stands, but you should probably also fix the hash for arc,
which currently still points at 2014.08 while we use 2014.12 and it's from github
none xxx uClibc-arc-2014.12.tar.gz
Regards,
Arnout
> ---
> package/uclibc/uclibc.hash | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/package/uclibc/uclibc.hash b/package/uclibc/uclibc.hash
> index 69cc103..971ab26 100644
> --- a/package/uclibc/uclibc.hash
> +++ b/package/uclibc/uclibc.hash
> @@ -6,3 +6,6 @@ sha256 0a9085a11615296cfe7783f16ff29073393eb330749947adece0eeddb31705a4 uClibc-a
>
> # From http://www.uclibc-ng.org/trac/
> sha256 e561040293ba4f21dcf3285ca1f683b5a8493eb0ffd3b3e495122cfde5a64a0e uClibc-ng-1.0.1.tar.xz
> +
> +# No hash for this git snapshot
> +none xxx uClibc-7bf35c8b7d4a1f97174eb49f47f33946b282114c.tar.gz
>
--
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: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH] uclibc: update hash for arc
2015-04-19 16:47 ` Arnout Vandecappelle
@ 2015-04-19 16:57 ` Arnout Vandecappelle
2015-04-19 17:03 ` [Buildroot] [PATCH 3/7 v5] package/uclibc: add missing hash for eXtensa Yann E. MORIN
1 sibling, 0 replies; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 16:57 UTC (permalink / raw)
To: buildroot
The old hash for 2014.08 was never updated. Since uclibc-arc-2014.12 is
downloaded from the github helper, it is converted into a 'none' hash.
Reordered the hashes so the none-hashes are together.
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
This should be inserted into Yann's series before or after 3/7.
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
package/uclibc/uclibc.hash | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/package/uclibc/uclibc.hash b/package/uclibc/uclibc.hash
index 971ab26..01e71c0 100644
--- a/package/uclibc/uclibc.hash
+++ b/package/uclibc/uclibc.hash
@@ -1,11 +1,11 @@
# Locally calculated after checking pgp signature
sha256 62333167b79afb0b25a843513288c67b59547acf653e8fbe62ee64e71ebd1587 uClibc-0.9.33.2.tar.xz
-# Locally calculated
-sha256 0a9085a11615296cfe7783f16ff29073393eb330749947adece0eeddb31705a4 uClibc-arc-2014.08.tar.gz
-
# From http://www.uclibc-ng.org/trac/
sha256 e561040293ba4f21dcf3285ca1f683b5a8493eb0ffd3b3e495122cfde5a64a0e uClibc-ng-1.0.1.tar.xz
+# No hash for the ARC variant, comes from the github-helper
+none xxx uClibc-arc-2014.12.tar.gz
+
# No hash for this git snapshot
none xxx uClibc-7bf35c8b7d4a1f97174eb49f47f33946b282114c.tar.gz
--
2.1.4
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 3/7 v5] package/uclibc: add missing hash for eXtensa
2015-04-19 16:47 ` Arnout Vandecappelle
2015-04-19 16:57 ` [Buildroot] [PATCH] uclibc: update hash for arc Arnout Vandecappelle
@ 2015-04-19 17:03 ` Yann E. MORIN
1 sibling, 0 replies; 28+ messages in thread
From: Yann E. MORIN @ 2015-04-19 17:03 UTC (permalink / raw)
To: buildroot
Arnout, All,
On 2015-04-19 18:47 +0200, Arnout Vandecappelle spake thusly:
> On 19/04/15 14:44, Yann E. MORIN wrote:
> > This is a git snapshot, we can't have a hash for it.
> >
> > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> > Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
> > Cc: Arnout Vandecappelle <arnout@mind.be>
> > Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
>
> My reviewed-by still stands, but you should probably also fix the hash for arc,
> which currently still points at 2014.08 while we use 2014.12 and it's from github
>
> none xxx uClibc-arc-2014.12.tar.gz
Damn, I did not even check that the version was correct. Thanks for
spotting! :-)
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 4/7 v5] support/download: add possibility to not fail on missing hash
2015-04-19 12:44 ` [Buildroot] [PATCH 4/7 v5] support/download: add possibility to not fail on missing hash Yann E. MORIN
@ 2015-04-19 17:41 ` Arnout Vandecappelle
0 siblings, 0 replies; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 17:41 UTC (permalink / raw)
To: buildroot
On 19/04/15 14:44, Yann E. MORIN wrote:
> In very constrained cases, it might be needed to not fail if a hash is
> missing. This is notably the case for custom external toolchains to be
> downloaded, because we do have a .hash file for external toolchains,
> but we obviously can not have hashes for all existing custom toolchains
> (he, "custom"!).
>
> So, add a way to avoid failing in that case.
>
>>From the Makefile, we export the list of files for which not to check
> the hash. Then, from the check-hash script, if no check was done, and
> the file we were trying to match in in this exclusion list, we just exit
> without error.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
> Cc: Arnout Vandecappelle <arnout@mind.be>
> Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Ouch, it doesn't work...
>
> ---
> Changes v4 -> v5:
> - micro-optimisation, use case-esac instead of a for-loop (Arnout)
> - typoes (Arnout)
>
> Changes v3 -> v4:
> - drop the magic value, use a list of excluded files (Arnout)
>
> Changes v1 -> v2:
> - fix typoes in commit log
> ---
> package/pkg-download.mk | 3 +++
> support/download/check-hash | 6 ++++++
> 2 files changed, 9 insertions(+)
>
> diff --git a/package/pkg-download.mk b/package/pkg-download.mk
> index e274712..1fc23c4 100644
> --- a/package/pkg-download.mk
> +++ b/package/pkg-download.mk
> @@ -60,6 +60,9 @@ domainseparator = $(if $(1),$(1),/)
> # github(user,package,version): returns site of GitHub repository
> github = https://github.com/$(1)/$(2)/archive/$(3)
>
> +# Expressly do not check hashes for those files
> +export BR_NO_CHECK_HASH_FOR
The comment is not very clear. Perhaps
# Files in this variable do not get their hash checked.
export BR_NO_CHECK_HASH_FOR
> +
> ################################################################################
> # The DOWNLOAD_* helpers are in charge of getting a working copy
> # of the source repository for their corresponding SCM,
> diff --git a/support/download/check-hash b/support/download/check-hash
> index 678a7ef..1280322 100755
> --- a/support/download/check-hash
> +++ b/support/download/check-hash
> @@ -99,6 +99,12 @@ while read t h f; do
> done <"${h_file}"
>
> if [ ${nb_checks} -eq 0 ]; then
> + case " ${BR_NO_CHECK_HASH_FOR} " in
> + " ${base} ")
This does not work. If there is more than one file, there can be something
before/after ${base}, so we need to add * at the beginning and the end. But
then, the quotes stop the matching of *, so we have to write:
*\ ${base}\ *)
Regards,
Arnout
> + # File explicitly has no hash
> + exit 0
> + ;;
> + esac
> printf "ERROR: No hash found for %s\n" "${base}" >&2
> exit 0
> fi
>
--
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: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 6/7 v5] package/uclibc: ignore missing hash for snapshots
2015-04-19 12:44 ` [Buildroot] [PATCH 6/7 v5] package/uclibc: ignore missing hash for snapshots Yann E. MORIN
@ 2015-04-19 17:46 ` Arnout Vandecappelle
0 siblings, 0 replies; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 17:46 UTC (permalink / raw)
To: buildroot
On 19/04/15 14:44, Yann E. MORIN wrote:
> We can't have a hash for snapshots, so add them to the exclusion list.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
> Cc: Arnout Vandecappelle <arnout@mind.be>
> Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
> package/uclibc/uclibc.mk | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/package/uclibc/uclibc.mk b/package/uclibc/uclibc.mk
> index 72a46d9..1ec8b10 100644
> --- a/package/uclibc/uclibc.mk
> +++ b/package/uclibc/uclibc.mk
> @@ -11,6 +11,7 @@ UCLIBC_LICENSE_FILES = COPYING.LIB
>
> ifeq ($(BR2_UCLIBC_VERSION_SNAPSHOT),y)
> UCLIBC_SITE = http://www.uclibc.org/downloads/snapshots
> +BR_NO_CHECK_HASH_FOR += uClibc-$(UCLIBC_VERSION).tar.xz
The file is .bz2, not .xz.
I think it's better to put $(UCLIBC_SOURCE) instead.
Regards,
Arnout
> else ifeq ($(BR2_UCLIBC_VERSION_NG),y)
> UCLIBC_SITE = http://downloads.uclibc-ng.org/releases/$(UCLIBC_VERSION)
> UCLIBC_SOURCE = uClibc-ng-$(UCLIBC_VERSION).tar.xz
>
--
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: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH 7/7 v5] support/download: restore mandatory check of hashes
2015-04-19 12:45 ` [Buildroot] [PATCH 7/7 v5] support/download: restore mandatory check of hashes Yann E. MORIN
@ 2015-04-19 17:54 ` Arnout Vandecappelle
0 siblings, 0 replies; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 17:54 UTC (permalink / raw)
To: buildroot
On 19/04/15 14:45, Yann E. MORIN wrote:
> Now that custom external toolchains to be downloaded proprely instruct
properly
Also, it's not just the external toolchains that you fixed in this series.
Regards,
Arnout
> to not fail on a missing hash, restore the mandatory hash check for
> everything else.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
> Cc: Arnout Vandecappelle <arnout@mind.be>
> Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
> support/download/check-hash | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/support/download/check-hash b/support/download/check-hash
> index 1280322..383bd5c 100755
> --- a/support/download/check-hash
> +++ b/support/download/check-hash
> @@ -106,5 +106,5 @@ if [ ${nb_checks} -eq 0 ]; then
> ;;
> esac
> printf "ERROR: No hash found for %s\n" "${base}" >&2
> - exit 0
> + exit 3
> fi
>
--
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: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH v6 1/8] package/gcc: add hashes
2015-04-19 12:44 [Buildroot] [PATCH 0/7 v5] toolchain: better handle hashes (branch yem/dl-hash-toolchains) Yann E. MORIN
` (6 preceding siblings ...)
2015-04-19 12:45 ` [Buildroot] [PATCH 7/7 v5] support/download: restore mandatory check of hashes Yann E. MORIN
@ 2015-04-19 17:55 ` Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 2/8] package/binutils: " Arnout Vandecappelle
` (6 more replies)
7 siblings, 7 replies; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 17:55 UTC (permalink / raw)
To: buildroot
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Arnout Vandecappelle <arnout@mind.be>
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
package/gcc/gcc-final/gcc-final.hash | 1 +
package/gcc/gcc-initial/gcc-initial.hash | 1 +
package/gcc/gcc.hash | 10 ++++++++++
3 files changed, 12 insertions(+)
create mode 120000 package/gcc/gcc-final/gcc-final.hash
create mode 120000 package/gcc/gcc-initial/gcc-initial.hash
create mode 100644 package/gcc/gcc.hash
diff --git a/package/gcc/gcc-final/gcc-final.hash b/package/gcc/gcc-final/gcc-final.hash
new file mode 120000
index 0000000..7ac9361
--- /dev/null
+++ b/package/gcc/gcc-final/gcc-final.hash
@@ -0,0 +1 @@
+../gcc.hash
\ No newline at end of file
diff --git a/package/gcc/gcc-initial/gcc-initial.hash b/package/gcc/gcc-initial/gcc-initial.hash
new file mode 120000
index 0000000..7ac9361
--- /dev/null
+++ b/package/gcc/gcc-initial/gcc-initial.hash
@@ -0,0 +1 @@
+../gcc.hash
\ No newline at end of file
diff --git a/package/gcc/gcc.hash b/package/gcc/gcc.hash
new file mode 100644
index 0000000..5eec97f
--- /dev/null
+++ b/package/gcc/gcc.hash
@@ -0,0 +1,10 @@
+# From ftp://gcc.gnu.org/pub/gcc/releases/gcc-4.5.4/sha512.sum
+sha512 78696b287d46aacd6f150920da376ea32f58ad9f0dafd2d3b7fa6dbdd8dd7afe659108d1dfaa5807617fc5248719f50b39c37ade0173ea5b9fec2091f4656115 gcc-4.5.4.tar.bz2
+# From ftp://gcc.gnu.org/pub/gcc/releases/gcc-4.7.4/sha512.sum
+sha512 dfcb737073191e628231031a3571ec77ee760a59377630f4a6e4fdfa66f9ddad39fde47e3f0f227eb43cdf90e0d34cde5abdc9ac892c1e111a911062a66c9189 gcc-4.7.4.tar.bz2
+# From ftp://gcc.gnu.org/pub/gcc/releases/gcc-4.8.4/sha512.sum
+sha512 b4c84ebd754c026029d706ef52c4242df4dcadc6a22cc5669b3aa37ac9dc25bc9cd9651fda83fb24dd5d17b1710e6865a35bb6c5cd9b95d0971717453935ae8e gcc-4.8.4.tar.bz2
+# From ftp://gcc.gnu.org/pub/gcc/releases/gcc-4.9.2/sha512.sum
+sha512 e96cba06d572dbe7f382a2b00fd9297dcf9f9219d46a9ad0bd904dc36d5e7db3335e422640b79a27fed6fde91b0731732b94a0e2fbcd47344d6779f8ba1f4f9c gcc-4.9.2.tar.bz2
+# No hash for the ARC variant, comes from the github-helper:
+none xxx gcc-arc-2014.12.tar.gz
--
2.1.4
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH v6 2/8] package/binutils: add hashes
2015-04-19 17:55 ` [Buildroot] [PATCH v6 1/8] package/gcc: add hashes Arnout Vandecappelle
@ 2015-04-19 17:55 ` Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 3/8] package/uclibc: add missing hash for eXtensa Arnout Vandecappelle
` (5 subsequent siblings)
6 siblings, 0 replies; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 17:55 UTC (permalink / raw)
To: buildroot
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Arnout Vandecappelle <arnout@mind.be>
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
package/binutils/binutils.hash | 7 +++++++
1 file changed, 7 insertions(+)
create mode 100644 package/binutils/binutils.hash
diff --git a/package/binutils/binutils.hash b/package/binutils/binutils.hash
new file mode 100644
index 0000000..028fc10
--- /dev/null
+++ b/package/binutils/binutils.hash
@@ -0,0 +1,7 @@
+# From ftp://gcc.gnu.org/pub/binutils/releases/sha512.sum
+sha512 ffe8ef263ef99183e8cc823fe8487ff7d0f7bf9a8efd2853b5f4636aca0023850d13de4eac7d77a5f69413d8a50e6f95bb14569be53df86c0bce38034525ab74 binutils-2.22.tar.bz2
+sha512 dec753bbba008f1526b89cf1bd85feba78f362f5333ffdf93953fd131eb755976dec82a0a4ba38c43d2434da007137780cfe674de5414be5cf7ce7fbc6af6d16 binutils-2.23.2.tar.bz2
+sha512 5ec95ad47d49b12c4558a8db0ca2109d3ee1955e3776057f3330c4506f8f4d1cf5e505fbf8a16b98403a0fcdeaaf986fe0a22be6456247dbdace63ce1f776b12 binutils-2.24.tar.bz2
+sha512 49438970642f379978aba988e3d2e7b63dab6fd0b46d91f4100f083e87a84d242089daca58572245fe620457a77bbbe2d78588ff3fb79314e500d1a39bf96f30 binutils-2.25.tar.bz2
+# No hash for the ARC variant, comes from the github-helper:
+none xxx binutils-arc-2014.12.tar.gz
--
2.1.4
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH v6 3/8] package/uclibc: add missing hash for eXtensa
2015-04-19 17:55 ` [Buildroot] [PATCH v6 1/8] package/gcc: add hashes Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 2/8] package/binutils: " Arnout Vandecappelle
@ 2015-04-19 17:55 ` Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 4/8] uclibc: update hash for arc Arnout Vandecappelle
` (4 subsequent siblings)
6 siblings, 0 replies; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 17:55 UTC (permalink / raw)
To: buildroot
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
This is a git snapshot, we can't have a hash for it.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Arnout Vandecappelle <arnout@mind.be>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
package/uclibc/uclibc.hash | 3 +++
1 file changed, 3 insertions(+)
diff --git a/package/uclibc/uclibc.hash b/package/uclibc/uclibc.hash
index 69cc103..971ab26 100644
--- a/package/uclibc/uclibc.hash
+++ b/package/uclibc/uclibc.hash
@@ -6,3 +6,6 @@ sha256 0a9085a11615296cfe7783f16ff29073393eb330749947adece0eeddb31705a4 uClibc-a
# From http://www.uclibc-ng.org/trac/
sha256 e561040293ba4f21dcf3285ca1f683b5a8493eb0ffd3b3e495122cfde5a64a0e uClibc-ng-1.0.1.tar.xz
+
+# No hash for this git snapshot
+none xxx uClibc-7bf35c8b7d4a1f97174eb49f47f33946b282114c.tar.gz
--
2.1.4
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH v6 4/8] uclibc: update hash for arc
2015-04-19 17:55 ` [Buildroot] [PATCH v6 1/8] package/gcc: add hashes Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 2/8] package/binutils: " Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 3/8] package/uclibc: add missing hash for eXtensa Arnout Vandecappelle
@ 2015-04-19 17:55 ` Arnout Vandecappelle
2015-04-19 18:28 ` Yann E. MORIN
2015-04-19 17:55 ` [Buildroot] [PATCH v6 5/8] support/download: add possibility to not fail on missing hash Arnout Vandecappelle
` (3 subsequent siblings)
6 siblings, 1 reply; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 17:55 UTC (permalink / raw)
To: buildroot
The old hash for 2014.08 was never updated. Since uclibc-arc-2014.12 is
downloaded from the github helper, it is converted into a 'none' hash.
Reordered the hashes so the none-hashes are together.
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
This should be inserted into Yann's series before or after 3/7.
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
package/uclibc/uclibc.hash | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/package/uclibc/uclibc.hash b/package/uclibc/uclibc.hash
index 971ab26..01e71c0 100644
--- a/package/uclibc/uclibc.hash
+++ b/package/uclibc/uclibc.hash
@@ -1,11 +1,11 @@
# Locally calculated after checking pgp signature
sha256 62333167b79afb0b25a843513288c67b59547acf653e8fbe62ee64e71ebd1587 uClibc-0.9.33.2.tar.xz
-# Locally calculated
-sha256 0a9085a11615296cfe7783f16ff29073393eb330749947adece0eeddb31705a4 uClibc-arc-2014.08.tar.gz
-
# From http://www.uclibc-ng.org/trac/
sha256 e561040293ba4f21dcf3285ca1f683b5a8493eb0ffd3b3e495122cfde5a64a0e uClibc-ng-1.0.1.tar.xz
+# No hash for the ARC variant, comes from the github-helper
+none xxx uClibc-arc-2014.12.tar.gz
+
# No hash for this git snapshot
none xxx uClibc-7bf35c8b7d4a1f97174eb49f47f33946b282114c.tar.gz
--
2.1.4
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH v6 5/8] support/download: add possibility to not fail on missing hash
2015-04-19 17:55 ` [Buildroot] [PATCH v6 1/8] package/gcc: add hashes Arnout Vandecappelle
` (2 preceding siblings ...)
2015-04-19 17:55 ` [Buildroot] [PATCH v6 4/8] uclibc: update hash for arc Arnout Vandecappelle
@ 2015-04-19 17:55 ` Arnout Vandecappelle
2015-04-19 18:38 ` Yann E. MORIN
2015-04-19 17:55 ` [Buildroot] [PATCH v6 6/8] toolchain/external: ignore missing hash for custom downloaded toolchain Arnout Vandecappelle
` (2 subsequent siblings)
6 siblings, 1 reply; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 17:55 UTC (permalink / raw)
To: buildroot
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
In very constrained cases, it might be needed to not fail if a hash is
missing. This is notably the case for custom external toolchains to be
downloaded, because we do have a .hash file for external toolchains,
but we obviously can not have hashes for all existing custom toolchains
(he, "custom"!).
So, add a way to avoid failing in that case.
From the Makefile, we export the list of files for which not to check
the hash. Then, from the check-hash script, if no check was done, and
the file we were trying to match in in this exclusion list, we just exit
without error.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
[Arnout: fix the pattern in the case statement]
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Tested-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
Changed v5 -> v6:
- fix the pattern in the case clause
Changes v4 -> v5:
- micro-optimisation, use case-esac instead of a for-loop (Arnout)
- typoes (Arnout)
Changes v3 -> v4:
- drop the magic value, use a list of excluded files (Arnout)
Changes v1 -> v2:
- fix typoes in commit log
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
package/pkg-download.mk | 3 +++
support/download/check-hash | 6 ++++++
2 files changed, 9 insertions(+)
diff --git a/package/pkg-download.mk b/package/pkg-download.mk
index e274712..1fc23c4 100644
--- a/package/pkg-download.mk
+++ b/package/pkg-download.mk
@@ -60,6 +60,9 @@ domainseparator = $(if $(1),$(1),/)
# github(user,package,version): returns site of GitHub repository
github = https://github.com/$(1)/$(2)/archive/$(3)
+# Expressly do not check hashes for those files
+export BR_NO_CHECK_HASH_FOR
+
################################################################################
# The DOWNLOAD_* helpers are in charge of getting a working copy
# of the source repository for their corresponding SCM,
diff --git a/support/download/check-hash b/support/download/check-hash
index 678a7ef..d378701 100755
--- a/support/download/check-hash
+++ b/support/download/check-hash
@@ -99,6 +99,12 @@ while read t h f; do
done <"${h_file}"
if [ ${nb_checks} -eq 0 ]; then
+ case " ${BR_NO_CHECK_HASH_FOR} " in
+ *\ ${base}\ *)
+ # File explicitly has no hash
+ exit 0
+ ;;
+ esac
printf "ERROR: No hash found for %s\n" "${base}" >&2
exit 0
fi
--
2.1.4
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH v6 6/8] toolchain/external: ignore missing hash for custom downloaded toolchain
2015-04-19 17:55 ` [Buildroot] [PATCH v6 1/8] package/gcc: add hashes Arnout Vandecappelle
` (3 preceding siblings ...)
2015-04-19 17:55 ` [Buildroot] [PATCH v6 5/8] support/download: add possibility to not fail on missing hash Arnout Vandecappelle
@ 2015-04-19 17:55 ` Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 7/8] package/uclibc: ignore missing hash for snapshots Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 8/8] support/download: restore mandatory check of hashes Arnout Vandecappelle
6 siblings, 0 replies; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 17:55 UTC (permalink / raw)
To: buildroot
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
We will *always* be missing a hash file for custom external toolchains
that are downloaded.
So, just ignore that failure.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Arnout Vandecappelle <arnout@mind.be>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
toolchain/toolchain-external/toolchain-external.mk | 2 ++
1 file changed, 2 insertions(+)
diff --git a/toolchain/toolchain-external/toolchain-external.mk b/toolchain/toolchain-external/toolchain-external.mk
index 2427463..5576dcb 100644
--- a/toolchain/toolchain-external/toolchain-external.mk
+++ b/toolchain/toolchain-external/toolchain-external.mk
@@ -400,6 +400,8 @@ else
# Custom toolchain
TOOLCHAIN_EXTERNAL_SITE = $(dir $(call qstrip,$(BR2_TOOLCHAIN_EXTERNAL_URL)))
TOOLCHAIN_EXTERNAL_SOURCE = $(notdir $(call qstrip,$(BR2_TOOLCHAIN_EXTERNAL_URL)))
+# We can't check hashes for custom downloaded toolchains
+BR_NO_CHECK_HASH_FOR += $(TOOLCHAIN_EXTERNAL_SOURCE)
endif
# In fact, we don't need to download the toolchain, since it is already
--
2.1.4
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH v6 7/8] package/uclibc: ignore missing hash for snapshots
2015-04-19 17:55 ` [Buildroot] [PATCH v6 1/8] package/gcc: add hashes Arnout Vandecappelle
` (4 preceding siblings ...)
2015-04-19 17:55 ` [Buildroot] [PATCH v6 6/8] toolchain/external: ignore missing hash for custom downloaded toolchain Arnout Vandecappelle
@ 2015-04-19 17:55 ` Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 8/8] support/download: restore mandatory check of hashes Arnout Vandecappelle
6 siblings, 0 replies; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 17:55 UTC (permalink / raw)
To: buildroot
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
We can't have a hash for snapshots, so add them to the exclusion list.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
[Arnout: use the correct file name]
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Arnout Vandecappelle <arnout@mind.be>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
Changes v1 -> v6: (Arnout)
- use $(UCLIBC_SOURCE) instead of a wrong filename
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
package/uclibc/uclibc.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/package/uclibc/uclibc.mk b/package/uclibc/uclibc.mk
index 72a46d9..72261f3 100644
--- a/package/uclibc/uclibc.mk
+++ b/package/uclibc/uclibc.mk
@@ -11,6 +11,7 @@ UCLIBC_LICENSE_FILES = COPYING.LIB
ifeq ($(BR2_UCLIBC_VERSION_SNAPSHOT),y)
UCLIBC_SITE = http://www.uclibc.org/downloads/snapshots
+BR_NO_CHECK_HASH_FOR += $(UCLIBC_SOURCE)
else ifeq ($(BR2_UCLIBC_VERSION_NG),y)
UCLIBC_SITE = http://downloads.uclibc-ng.org/releases/$(UCLIBC_VERSION)
UCLIBC_SOURCE = uClibc-ng-$(UCLIBC_VERSION).tar.xz
--
2.1.4
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH v6 8/8] support/download: restore mandatory check of hashes
2015-04-19 17:55 ` [Buildroot] [PATCH v6 1/8] package/gcc: add hashes Arnout Vandecappelle
` (5 preceding siblings ...)
2015-04-19 17:55 ` [Buildroot] [PATCH v6 7/8] package/uclibc: ignore missing hash for snapshots Arnout Vandecappelle
@ 2015-04-19 17:55 ` Arnout Vandecappelle
6 siblings, 0 replies; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-19 17:55 UTC (permalink / raw)
To: buildroot
From: "Yann E. MORIN" <yann.morin.1998@free.fr>
Now that custom external toolchains to be downloaded and the internal
toolchain components properly instruct to not fail on a missing hash,
restore the mandatory hash check for everything else.
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
[Arnout: fix typo in commit message]
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Arnout Vandecappelle <arnout@mind.be>
Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Tested-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
---
support/download/check-hash | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/support/download/check-hash b/support/download/check-hash
index d378701..f88b78b 100755
--- a/support/download/check-hash
+++ b/support/download/check-hash
@@ -106,5 +106,5 @@ if [ ${nb_checks} -eq 0 ]; then
;;
esac
printf "ERROR: No hash found for %s\n" "${base}" >&2
- exit 0
+ exit 3
fi
--
2.1.4
^ permalink raw reply related [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH v6 4/8] uclibc: update hash for arc
2015-04-19 17:55 ` [Buildroot] [PATCH v6 4/8] uclibc: update hash for arc Arnout Vandecappelle
@ 2015-04-19 18:28 ` Yann E. MORIN
0 siblings, 0 replies; 28+ messages in thread
From: Yann E. MORIN @ 2015-04-19 18:28 UTC (permalink / raw)
To: buildroot
Arnout, All,
On 2015-04-19 19:55 +0200, Arnout Vandecappelle (Essensium/Mind) spake thusly:
> The old hash for 2014.08 was never updated. Since uclibc-arc-2014.12 is
> downloaded from the github helper, it is converted into a 'none' hash.
>
> Reordered the hashes so the none-hashes are together.
>
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Regards,
Yann E. MORIN.
> ---
> This should be inserted into Yann's series before or after 3/7.
>
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
> package/uclibc/uclibc.hash | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/package/uclibc/uclibc.hash b/package/uclibc/uclibc.hash
> index 971ab26..01e71c0 100644
> --- a/package/uclibc/uclibc.hash
> +++ b/package/uclibc/uclibc.hash
> @@ -1,11 +1,11 @@
> # Locally calculated after checking pgp signature
> sha256 62333167b79afb0b25a843513288c67b59547acf653e8fbe62ee64e71ebd1587 uClibc-0.9.33.2.tar.xz
>
> -# Locally calculated
> -sha256 0a9085a11615296cfe7783f16ff29073393eb330749947adece0eeddb31705a4 uClibc-arc-2014.08.tar.gz
> -
> # From http://www.uclibc-ng.org/trac/
> sha256 e561040293ba4f21dcf3285ca1f683b5a8493eb0ffd3b3e495122cfde5a64a0e uClibc-ng-1.0.1.tar.xz
>
> +# No hash for the ARC variant, comes from the github-helper
> +none xxx uClibc-arc-2014.12.tar.gz
> +
> # No hash for this git snapshot
> none xxx uClibc-7bf35c8b7d4a1f97174eb49f47f33946b282114c.tar.gz
> --
> 2.1.4
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH v6 5/8] support/download: add possibility to not fail on missing hash
2015-04-19 17:55 ` [Buildroot] [PATCH v6 5/8] support/download: add possibility to not fail on missing hash Arnout Vandecappelle
@ 2015-04-19 18:38 ` Yann E. MORIN
2015-04-20 7:45 ` Arnout Vandecappelle
0 siblings, 1 reply; 28+ messages in thread
From: Yann E. MORIN @ 2015-04-19 18:38 UTC (permalink / raw)
To: buildroot
Arnout, All,
On 2015-04-19 19:55 +0200, Arnout Vandecappelle (Essensium/Mind) spake thusly:
> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
>
> In very constrained cases, it might be needed to not fail if a hash is
> missing. This is notably the case for custom external toolchains to be
> downloaded, because we do have a .hash file for external toolchains,
> but we obviously can not have hashes for all existing custom toolchains
> (he, "custom"!).
>
> So, add a way to avoid failing in that case.
>
> >From the Makefile, we export the list of files for which not to check
Stray leading '>' probably for a copy-paste from your mailer.
> the hash. Then, from the check-hash script, if no check was done, and
> the file we were trying to match in in this exclusion list, we just exit
> without error.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> [Arnout: fix the pattern in the case statement]
> Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
> Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> Tested-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
> ---
> Changed v5 -> v6:
> - fix the pattern in the case clause
Yeah, thanks. Obviously, I did not test that case (although the other
tests I ran were OK, I missed testing the custom toolchain to be
downloaded, which is a shame since that's exactly what I was trying to
fix with this series, to begin with...).
[--SNIP--]
> diff --git a/support/download/check-hash b/support/download/check-hash
> index 678a7ef..d378701 100755
> --- a/support/download/check-hash
> +++ b/support/download/check-hash
> @@ -99,6 +99,12 @@ while read t h f; do
> done <"${h_file}"
>
> if [ ${nb_checks} -eq 0 ]; then
> + case " ${BR_NO_CHECK_HASH_FOR} " in
> + *\ ${base}\ *)
I don't like much the escaping when it can be avoided. We can still use
quotes (even if not very much better), like so:
*" ${base} "*
Regards,
Yann E. MORIN.
> + # File explicitly has no hash
> + exit 0
> + ;;
> + esac
> printf "ERROR: No hash found for %s\n" "${base}" >&2
> exit 0
> fi
> --
> 2.1.4
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH v6 5/8] support/download: add possibility to not fail on missing hash
2015-04-19 18:38 ` Yann E. MORIN
@ 2015-04-20 7:45 ` Arnout Vandecappelle
2015-04-20 21:24 ` Yann E. MORIN
0 siblings, 1 reply; 28+ messages in thread
From: Arnout Vandecappelle @ 2015-04-20 7:45 UTC (permalink / raw)
To: buildroot
On 19/04/15 20:38, Yann E. MORIN wrote:
> Arnout, All,
>
> On 2015-04-19 19:55 +0200, Arnout Vandecappelle (Essensium/Mind) spake thusly:
>> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
>>
>> In very constrained cases, it might be needed to not fail if a hash is
>> missing. This is notably the case for custom external toolchains to be
>> downloaded, because we do have a .hash file for external toolchains,
>> but we obviously can not have hashes for all existing custom toolchains
>> (he, "custom"!).
>>
>> So, add a way to avoid failing in that case.
>>
>> >From the Makefile, we export the list of files for which not to check
>
> Stray leading '>' probably for a copy-paste from your mailer.
pwclient git-am actually... I would have expected patchwork (or actually, the
python mail handling infra) to handle >From correctly...
>
>> the hash. Then, from the check-hash script, if no check was done, and
>> the file we were trying to match in in this exclusion list, we just exit
>> without error.
>>
[snip]
>> diff --git a/support/download/check-hash b/support/download/check-hash
>> index 678a7ef..d378701 100755
>> --- a/support/download/check-hash
>> +++ b/support/download/check-hash
>> @@ -99,6 +99,12 @@ while read t h f; do
>> done <"${h_file}"
>>
>> if [ ${nb_checks} -eq 0 ]; then
>> + case " ${BR_NO_CHECK_HASH_FOR} " in
>> + *\ ${base}\ *)
>
> I don't like much the escaping when it can be avoided. We can still use
> quotes (even if not very much better), like so:
>
> *" ${base} "*
It's your patch, modify as you like :-)
I just reposted it since I anyway had it locally modified.
Regards,
Arnout
--
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: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
^ permalink raw reply [flat|nested] 28+ messages in thread
* [Buildroot] [PATCH v6 5/8] support/download: add possibility to not fail on missing hash
2015-04-20 7:45 ` Arnout Vandecappelle
@ 2015-04-20 21:24 ` Yann E. MORIN
0 siblings, 0 replies; 28+ messages in thread
From: Yann E. MORIN @ 2015-04-20 21:24 UTC (permalink / raw)
To: buildroot
Arnout, All,
On 2015-04-20 09:45 +0200, Arnout Vandecappelle spake thusly:
> On 19/04/15 20:38, Yann E. MORIN wrote:
[--SNIP--]
> >> >From the Makefile, we export the list of files for which not to check
> >
> > Stray leading '>' probably for a copy-paste from your mailer.
>
> pwclient git-am actually... I would have expected patchwork (or actually, the
> python mail handling infra) to handle >From correctly...
Actually, that comes from RFC 4155.
https://tools.ietf.org/html/rfc4155
TL;DR: "From " is actually used in mbox format to define the begining
of an email. Thus, any lines in a body, that begins thusly, must be
"quoted" to avoid improperly detecting the begining of an email.
See also this mail in the Patchwork ML archives:
https://lists.ozlabs.org/pipermail/patchwork/2014-November/001218.html
I don't think this was ever applied...
[--SNIP--]
> > I don't like much the escaping when it can be avoided. We can still use
> > quotes (even if not very much better), like so:
> >
> > *" ${base} "*
>
> It's your patch, modify as you like :-)
Hehe! :-)
> I just reposted it since I anyway had it locally modified.
Yeah, thanks! :-)
Regards,
Yann E. MORIN.
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
^ permalink raw reply [flat|nested] 28+ messages in thread
end of thread, other threads:[~2015-04-20 21:24 UTC | newest]
Thread overview: 28+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-19 12:44 [Buildroot] [PATCH 0/7 v5] toolchain: better handle hashes (branch yem/dl-hash-toolchains) Yann E. MORIN
2015-04-19 12:44 ` [Buildroot] [PATCH 1/7 v5] package/gcc: add hashes Yann E. MORIN
2015-04-19 16:41 ` Arnout Vandecappelle
2015-04-19 12:44 ` [Buildroot] [PATCH 2/7 v5] package/binutils: " Yann E. MORIN
2015-04-19 16:42 ` Arnout Vandecappelle
2015-04-19 12:44 ` [Buildroot] [PATCH 3/7 v5] package/uclibc: add missing hash for eXtensa Yann E. MORIN
2015-04-19 16:47 ` Arnout Vandecappelle
2015-04-19 16:57 ` [Buildroot] [PATCH] uclibc: update hash for arc Arnout Vandecappelle
2015-04-19 17:03 ` [Buildroot] [PATCH 3/7 v5] package/uclibc: add missing hash for eXtensa Yann E. MORIN
2015-04-19 12:44 ` [Buildroot] [PATCH 4/7 v5] support/download: add possibility to not fail on missing hash Yann E. MORIN
2015-04-19 17:41 ` Arnout Vandecappelle
2015-04-19 12:44 ` [Buildroot] [PATCH 5/7 v5] toolchain/external: ignore missing hash for custom downloaded toolchain Yann E. MORIN
2015-04-19 12:44 ` [Buildroot] [PATCH 6/7 v5] package/uclibc: ignore missing hash for snapshots Yann E. MORIN
2015-04-19 17:46 ` Arnout Vandecappelle
2015-04-19 12:45 ` [Buildroot] [PATCH 7/7 v5] support/download: restore mandatory check of hashes Yann E. MORIN
2015-04-19 17:54 ` Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 1/8] package/gcc: add hashes Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 2/8] package/binutils: " Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 3/8] package/uclibc: add missing hash for eXtensa Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 4/8] uclibc: update hash for arc Arnout Vandecappelle
2015-04-19 18:28 ` Yann E. MORIN
2015-04-19 17:55 ` [Buildroot] [PATCH v6 5/8] support/download: add possibility to not fail on missing hash Arnout Vandecappelle
2015-04-19 18:38 ` Yann E. MORIN
2015-04-20 7:45 ` Arnout Vandecappelle
2015-04-20 21:24 ` Yann E. MORIN
2015-04-19 17:55 ` [Buildroot] [PATCH v6 6/8] toolchain/external: ignore missing hash for custom downloaded toolchain Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 7/8] package/uclibc: ignore missing hash for snapshots Arnout Vandecappelle
2015-04-19 17:55 ` [Buildroot] [PATCH v6 8/8] support/download: restore mandatory check of hashes Arnout Vandecappelle
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox