All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/dtc: Bump to v1.5.1
@ 2019-10-25 20:00 Titouan Christophe
  2019-10-26 13:39 ` Thomas Petazzoni
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Titouan Christophe @ 2019-10-25 20:00 UTC (permalink / raw)
  To: buildroot

Remove 0002-Makefile-Add-EXTRA_CFLAGS-variable.patch which is now
applied upstream

Signed-off-by: Titouan Christophe <titouan.christophe@railnova.eu>
---
 ...ards-for-older-kernel-u-boot-sources.patch | 13 +++++--
 ...2-Makefile-Add-EXTRA_CFLAGS-variable.patch | 37 -------------------
 package/dtc/dtc.hash                          |  2 +-
 package/dtc/dtc.mk                            |  2 +-
 4 files changed, 11 insertions(+), 43 deletions(-)
 delete mode 100644 package/dtc/0002-Makefile-Add-EXTRA_CFLAGS-variable.patch

diff --git a/package/dtc/0001-Fix-include-guards-for-older-kernel-u-boot-sources.patch b/package/dtc/0001-Fix-include-guards-for-older-kernel-u-boot-sources.patch
index bff5f4c07a..bf3d61a8ce 100644
--- a/package/dtc/0001-Fix-include-guards-for-older-kernel-u-boot-sources.patch
+++ b/package/dtc/0001-Fix-include-guards-for-older-kernel-u-boot-sources.patch
@@ -15,6 +15,8 @@ Signed-off-by: Lothar Felten <lothar.felten@gmail.com>
 [ThomasDS: also update fdt.h which has the same issue, seen on U-Boot
 2011.03]
 Signed-off-by: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
+[Titouan: Taking new SPDX line into account]
+Signed-off-by: Titouan Christophe <titouan.christophe@railnova.eu>
 ---
  libfdt/fdt.h        | 4 ++++
  libfdt/libfdt.h     | 4 ++++
@@ -25,7 +27,8 @@ diff --git a/libfdt/fdt.h b/libfdt/fdt.h
 index 74961f9..2904f48 100644
 --- a/libfdt/fdt.h
 +++ b/libfdt/fdt.h
-@@ -1,3 +1,7 @@
+@@ -1,4 +1,8 @@
+ /* SPDX-License-Identifier: (GPL-2.0-or-later OR BSD-2-Clause) */
 +#ifdef _FDT_H
 +#warning "Please consider updating your kernel and/or u-boot version"
 +#define FDT_H
@@ -37,7 +40,8 @@ diff --git a/libfdt/libfdt.h b/libfdt/libfdt.h
 index 830b77e..bef4566 100644
 --- a/libfdt/libfdt.h
 +++ b/libfdt/libfdt.h
-@@ -1,3 +1,7 @@
+@@ -1,4 +1,8 @@
+ /* SPDX-License-Identifier: (GPL-2.0-or-later OR BSD-2-Clause) */
 +#ifdef _LIBFDT_H
 +#warning "Please consider updating your kernel and/or u-boot version"
 +#define LIBFDT_H
@@ -49,7 +53,8 @@ diff --git a/libfdt/libfdt_env.h b/libfdt/libfdt_env.h
 index eb20538..6a61e6a 100644
 --- a/libfdt/libfdt_env.h
 +++ b/libfdt/libfdt_env.h
-@@ -1,3 +1,7 @@
+@@ -1,4 +1,8 @@
+ /* SPDX-License-Identifier: (GPL-2.0-or-later OR BSD-2-Clause) */
 +#ifdef _LIBFDT_ENV_H
 +#warning "Please consider updating your kernel and/or u-boot version"
 +#define LIBFDT_ENV_H
@@ -58,5 +63,5 @@ index eb20538..6a61e6a 100644
  #define LIBFDT_ENV_H
  /*
 -- 
-2.19.2
+2.21.0
 
diff --git a/package/dtc/0002-Makefile-Add-EXTRA_CFLAGS-variable.patch b/package/dtc/0002-Makefile-Add-EXTRA_CFLAGS-variable.patch
deleted file mode 100644
index b3f14a6991..0000000000
--- a/package/dtc/0002-Makefile-Add-EXTRA_CFLAGS-variable.patch
+++ /dev/null
@@ -1,37 +0,0 @@
-From f7c659d7daff5dbf4a0be959a544f15eab7fc49c Mon Sep 17 00:00:00 2001
-From: Joel Stanley <joel@jms.id.au>
-Date: Mon, 22 Jul 2019 12:21:49 +0930
-Subject: [PATCH] Makefile: Add EXTRA_CFLAGS variable
-
-Distributions packaging dtc may need to set extra flags. Currently when
-they do that it overrides the ones set by the makefile. This is
-particularly problematic when compiling without yaml, as the yaml
-detection is ignored.
-
- ld: dtc.o: in function `main':
- dtc.c:(.text.startup+0x718): undefined reference to `dt_to_yaml'
-
-This patch provides a EXTRA_CFLAGS variable that is added to the list of
-CFLAGS, and can be set on the command line when packaging.
-
-Signed-off-by: Joel Stanley <joel@jms.id.au>
----
- Makefile | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-diff --git a/Makefile b/Makefile
-index e6b32cf1cbf5..6807669569b3 100644
---- a/Makefile
-+++ b/Makefile
-@@ -18,7 +18,7 @@ CONFIG_LOCALVERSION =
- CPPFLAGS = -I libfdt -I .
- WARNINGS = -Wall -Wpointer-arith -Wcast-qual -Wnested-externs \
- 	-Wstrict-prototypes -Wmissing-prototypes -Wredundant-decls -Wshadow
--CFLAGS = -g -Os $(SHAREDLIB_CFLAGS) -Werror $(WARNINGS)
-+CFLAGS = -g -Os $(SHAREDLIB_CFLAGS) -Werror $(WARNINGS) $(EXTRA_CFLAGS)
- 
- BISON = bison
- LEX = flex
--- 
-2.20.1
-
diff --git a/package/dtc/dtc.hash b/package/dtc/dtc.hash
index 4d0aaea601..3adfcc45a9 100644
--- a/package/dtc/dtc.hash
+++ b/package/dtc/dtc.hash
@@ -1,5 +1,5 @@
 # from https://www.kernel.org/pub/software/utils/dtc/sha256sums.asc
-sha256 c672e443c9f7e39f5a7c8e602da6777f9ad55ad70de87de300a43828c8050172  dtc-1.5.0.tar.xz
+sha256 660b74039690fc37013660544d09191834efb58503c73c555c5513ba75ab031f  dtc-1.5.1.tar.xz
 
 # Hash for license files
 sha256 171ce1c292cf148a5be9ff95ce7e5c3d79a002721ab8e119eae3c0f5336d2b3d  README.license
diff --git a/package/dtc/dtc.mk b/package/dtc/dtc.mk
index 0ed914ba41..ce094cfb86 100644
--- a/package/dtc/dtc.mk
+++ b/package/dtc/dtc.mk
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-DTC_VERSION = 1.5.0
+DTC_VERSION = 1.5.1
 DTC_SOURCE = dtc-$(DTC_VERSION).tar.xz
 DTC_SITE = https://www.kernel.org/pub/software/utils/dtc
 DTC_LICENSE = GPL-2.0+ or BSD-2-Clause (library)
-- 
2.21.0

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

* [Buildroot] [PATCH 1/1] package/dtc: Bump to v1.5.1
  2019-10-25 20:00 [Buildroot] [PATCH 1/1] package/dtc: Bump to v1.5.1 Titouan Christophe
@ 2019-10-26 13:39 ` Thomas Petazzoni
  2019-10-26 23:05   ` Carlos Santos
  2019-10-26 18:41 ` Matthew Weber
  2019-11-05 21:20 ` Peter Korsgaard
  2 siblings, 1 reply; 7+ messages in thread
From: Thomas Petazzoni @ 2019-10-26 13:39 UTC (permalink / raw)
  To: buildroot

On Fri, 25 Oct 2019 22:00:34 +0200
Titouan Christophe <titouan.christophe@railnova.eu> wrote:

> Remove 0002-Makefile-Add-EXTRA_CFLAGS-variable.patch which is now
> applied upstream
> 
> Signed-off-by: Titouan Christophe <titouan.christophe@railnova.eu>
> ---
>  ...ards-for-older-kernel-u-boot-sources.patch | 13 +++++--
>  ...2-Makefile-Add-EXTRA_CFLAGS-variable.patch | 37 -------------------
>  package/dtc/dtc.hash                          |  2 +-
>  package/dtc/dtc.mk                            |  2 +-
>  4 files changed, 11 insertions(+), 43 deletions(-)
>  delete mode 100644 package/dtc/0002-Makefile-Add-EXTRA_CFLAGS-variable.patch

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

* [Buildroot] [PATCH 1/1] package/dtc: Bump to v1.5.1
  2019-10-25 20:00 [Buildroot] [PATCH 1/1] package/dtc: Bump to v1.5.1 Titouan Christophe
  2019-10-26 13:39 ` Thomas Petazzoni
@ 2019-10-26 18:41 ` Matthew Weber
  2019-10-27  9:11   ` Titouan Christophe
  2019-11-05 21:20 ` Peter Korsgaard
  2 siblings, 1 reply; 7+ messages in thread
From: Matthew Weber @ 2019-10-26 18:41 UTC (permalink / raw)
  To: buildroot

Titouan,

On Fri, Oct 25, 2019 at 3:08 PM Titouan Christophe
<titouan.christophe@railnova.eu> wrote:
>
> Remove 0002-Makefile-Add-EXTRA_CFLAGS-variable.patch which is now
> applied upstream
>
> Signed-off-by: Titouan Christophe <titouan.christophe@railnova.eu>
> ---
>  ...ards-for-older-kernel-u-boot-sources.patch | 13 +++++--
>  ...2-Makefile-Add-EXTRA_CFLAGS-variable.patch | 37 -------------------
>  package/dtc/dtc.hash                          |  2 +-
>  package/dtc/dtc.mk                            |  2 +-
>  4 files changed, 11 insertions(+), 43 deletions(-)
>  delete mode 100644 package/dtc/0002-Makefile-Add-EXTRA_CFLAGS-variable.patch
>
> diff --git a/package/dtc/0001-Fix-include-guards-for-older-kernel-u-boot-sources.patch b/package/dtc/0001-Fix-include-guards-for-older-kernel-u-boot-sources.patch
> index bff5f4c07a..bf3d61a8ce 100644
> --- a/package/dtc/0001-Fix-include-guards-for-older-kernel-u-boot-sources.patch
> +++ b/package/dtc/0001-Fix-include-guards-for-older-kernel-u-boot-sources.patch
> @@ -15,6 +15,8 @@ Signed-off-by: Lothar Felten <lothar.felten@gmail.com>
>  [ThomasDS: also update fdt.h which has the same issue, seen on U-Boot
>  2011.03]
>  Signed-off-by: Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
> +[Titouan: Taking new SPDX line into account]
> +Signed-off-by: Titouan Christophe <titouan.christophe@railnova.eu>
>  ---
>   libfdt/fdt.h        | 4 ++++
>   libfdt/libfdt.h     | 4 ++++
> @@ -25,7 +27,8 @@ diff --git a/libfdt/fdt.h b/libfdt/fdt.h
>  index 74961f9..2904f48 100644
>  --- a/libfdt/fdt.h
>  +++ b/libfdt/fdt.h
> -@@ -1,3 +1,7 @@
> +@@ -1,4 +1,8 @@
> + /* SPDX-License-Identifier: (GPL-2.0-or-later OR BSD-2-Clause) */
>  +#ifdef _FDT_H
>  +#warning "Please consider updating your kernel and/or u-boot version"
>  +#define FDT_H
> @@ -37,7 +40,8 @@ diff --git a/libfdt/libfdt.h b/libfdt/libfdt.h
>  index 830b77e..bef4566 100644
>  --- a/libfdt/libfdt.h
>  +++ b/libfdt/libfdt.h
> -@@ -1,3 +1,7 @@
> +@@ -1,4 +1,8 @@
> + /* SPDX-License-Identifier: (GPL-2.0-or-later OR BSD-2-Clause) */
>  +#ifdef _LIBFDT_H
>  +#warning "Please consider updating your kernel and/or u-boot version"
>  +#define LIBFDT_H
> @@ -49,7 +53,8 @@ diff --git a/libfdt/libfdt_env.h b/libfdt/libfdt_env.h
>  index eb20538..6a61e6a 100644
>  --- a/libfdt/libfdt_env.h
>  +++ b/libfdt/libfdt_env.h
> -@@ -1,3 +1,7 @@
> +@@ -1,4 +1,8 @@
> + /* SPDX-License-Identifier: (GPL-2.0-or-later OR BSD-2-Clause) */
>  +#ifdef _LIBFDT_ENV_H
>  +#warning "Please consider updating your kernel and/or u-boot version"
>  +#define LIBFDT_ENV_H
> @@ -58,5 +63,5 @@ index eb20538..6a61e6a 100644
>   #define LIBFDT_ENV_H
>   /*
>  --
> -2.19.2
> +2.21.0
>
> diff --git a/package/dtc/0002-Makefile-Add-EXTRA_CFLAGS-variable.patch b/package/dtc/0002-Makefile-Add-EXTRA_CFLAGS-variable.patch
> deleted file mode 100644
> index b3f14a6991..0000000000
> --- a/package/dtc/0002-Makefile-Add-EXTRA_CFLAGS-variable.patch
> +++ /dev/null
> @@ -1,37 +0,0 @@
> -From f7c659d7daff5dbf4a0be959a544f15eab7fc49c Mon Sep 17 00:00:00 2001
> -From: Joel Stanley <joel@jms.id.au>
> -Date: Mon, 22 Jul 2019 12:21:49 +0930
> -Subject: [PATCH] Makefile: Add EXTRA_CFLAGS variable
> -
> -Distributions packaging dtc may need to set extra flags. Currently when
> -they do that it overrides the ones set by the makefile. This is
> -particularly problematic when compiling without yaml, as the yaml
> -detection is ignored.
> -
> - ld: dtc.o: in function `main':
> - dtc.c:(.text.startup+0x718): undefined reference to `dt_to_yaml'
> -
> -This patch provides a EXTRA_CFLAGS variable that is added to the list of
> -CFLAGS, and can be set on the command line when packaging.
> -
> -Signed-off-by: Joel Stanley <joel@jms.id.au>
> ----
> - Makefile | 2 +-
> - 1 file changed, 1 insertion(+), 1 deletion(-)
> -
> -diff --git a/Makefile b/Makefile
> -index e6b32cf1cbf5..6807669569b3 100644
> ---- a/Makefile
> -+++ b/Makefile
> -@@ -18,7 +18,7 @@ CONFIG_LOCALVERSION =
> - CPPFLAGS = -I libfdt -I .
> - WARNINGS = -Wall -Wpointer-arith -Wcast-qual -Wnested-externs \
> -       -Wstrict-prototypes -Wmissing-prototypes -Wredundant-decls -Wshadow
> --CFLAGS = -g -Os $(SHAREDLIB_CFLAGS) -Werror $(WARNINGS)
> -+CFLAGS = -g -Os $(SHAREDLIB_CFLAGS) -Werror $(WARNINGS) $(EXTRA_CFLAGS)
> -
> - BISON = bison
> - LEX = flex
> ---
> -2.20.1
> -
> diff --git a/package/dtc/dtc.hash b/package/dtc/dtc.hash
> index 4d0aaea601..3adfcc45a9 100644
> --- a/package/dtc/dtc.hash
> +++ b/package/dtc/dtc.hash
> @@ -1,5 +1,5 @@
>  # from https://www.kernel.org/pub/software/utils/dtc/sha256sums.asc
> -sha256 c672e443c9f7e39f5a7c8e602da6777f9ad55ad70de87de300a43828c8050172  dtc-1.5.0.tar.xz
> +sha256 660b74039690fc37013660544d09191834efb58503c73c555c5513ba75ab031f  dtc-1.5.1.tar.xz
>
>  # Hash for license files
>  sha256 171ce1c292cf148a5be9ff95ce7e5c3d79a002721ab8e119eae3c0f5336d2b3d  README.license

>>> dtc 1.5.1 Collecting legal info
ERROR: README.license has wrong sha256 hash:
ERROR: expected:
171ce1c292cf148a5be9ff95ce7e5c3d79a002721ab8e119eae3c0f5336d2b3d
ERROR: got     :
78e1e1ce5728a990d59c7de61d4741c49185d5129daf783b386380a214328d68

> diff --git a/package/dtc/dtc.mk b/package/dtc/dtc.mk
> index 0ed914ba41..ce094cfb86 100644
> --- a/package/dtc/dtc.mk
> +++ b/package/dtc/dtc.mk
> @@ -4,7 +4,7 @@
>  #
>  ################################################################################
>
> -DTC_VERSION = 1.5.0
> +DTC_VERSION = 1.5.1
>  DTC_SOURCE = dtc-$(DTC_VERSION).tar.xz
>  DTC_SITE = https://www.kernel.org/pub/software/utils/dtc
>  DTC_LICENSE = GPL-2.0+ or BSD-2-Clause (library)
> --
> 2.21.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot



-- 

Matthew Weber | Associate Director Software Engineer | Commercial Avionics

COLLINS AEROSPACE

400 Collins Road NE, Cedar Rapids, Iowa 52498, USA

Tel: +1 319 295 7349 | FAX: +1 319 263 6099

matthew.weber at collins.com | collinsaerospace.com



CONFIDENTIALITY WARNING: This message may contain proprietary and/or
privileged information of Collins Aerospace and its affiliated
companies. If you are not the intended recipient, please 1) Do not
disclose, copy, distribute or use this message or its contents. 2)
Advise the sender by return email. 3) Delete all copies (including all
attachments) from your computer. Your cooperation is greatly
appreciated.


Any export restricted material should be shared using my
matthew.weber at corp.rockwellcollins.com address.

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

* [Buildroot] [PATCH 1/1] package/dtc: Bump to v1.5.1
  2019-10-26 13:39 ` Thomas Petazzoni
@ 2019-10-26 23:05   ` Carlos Santos
  0 siblings, 0 replies; 7+ messages in thread
From: Carlos Santos @ 2019-10-26 23:05 UTC (permalink / raw)
  To: buildroot

On Sat, Oct 26, 2019 at 10:39 AM Thomas Petazzoni
<thomas.petazzoni@bootlin.com> wrote:
>
> On Fri, 25 Oct 2019 22:00:34 +0200
> Titouan Christophe <titouan.christophe@railnova.eu> wrote:
>
> > Remove 0002-Makefile-Add-EXTRA_CFLAGS-variable.patch which is now
> > applied upstream
> >
> > Signed-off-by: Titouan Christophe <titouan.christophe@railnova.eu>
> > ---
> >  ...ards-for-older-kernel-u-boot-sources.patch | 13 +++++--
> >  ...2-Makefile-Add-EXTRA_CFLAGS-variable.patch | 37 -------------------
> >  package/dtc/dtc.hash                          |  2 +-
> >  package/dtc/dtc.mk                            |  2 +-
> >  4 files changed, 11 insertions(+), 43 deletions(-)
> >  delete mode 100644 package/dtc/0002-Makefile-Add-EXTRA_CFLAGS-variable.patch
>
> Applied to master, thanks.
>
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

This commit broke beaglebone:

2019-10-26T19:57:58 >>> linux 4dae378bbe721277b08699d1d88ffae12acc9b09 Building
2019-10-26T19:57:58
PATH="/work/beaglebone/host/bin:/work/beaglebone/host/sbin:/usr/share/Modules/bin:/usr/local/bin:/usr/local/sbin:/usr/bin:/usr/sbin://home/casantos/bin"
BR_BINARIES_DIR=/work/beaglebone/images KCFLAGS=-Wno-attribute-alias
/usr/bin/make -j1 HOSTCC="/usr/bin/gcc -O2
-I/work/beaglebone/host/include -L/work/beaglebone/host/lib
-Wl,-rpath,/work/beaglebone/host/lib" ARCH=arm
INSTALL_MOD_PATH=/work/beaglebone/target
CROSS_COMPILE="/work/beaglebone/host/bin/arm-buildroot-linux-uclibcgnueabihf-"
DEPMOD=/work/beaglebone/host/sbin/depmod INSTALL_MOD_STRIP=1 -C
/work/beaglebone/build/linux-4dae378bbe721277b08699d1d88ffae12acc9b09
all
2019-10-26T19:57:59 CALL    scripts/checksyscalls.sh
2019-10-26T19:57:59 HOSTCC  scripts/dtc/dtc-lexer.lex.o
2019-10-26T19:57:59 HOSTCC  scripts/dtc/dtc-parser.tab.o
2019-10-26T19:57:59 HOSTLD  scripts/dtc/dtc
2019-10-26T19:57:59 HOSTCC  scripts/dtc/libfdt/fdt_ro.o
2019-10-26T19:57:59 scripts/dtc/libfdt/fdt_ro.c:92:10: error:
redefinition of ?fdt_get_max_phandle?
2019-10-26T19:57:59 uint32_t fdt_get_max_phandle(const void *fdt)
2019-10-26T19:57:59 ^~~~~~~~~~~~~~~~~~~
2019-10-26T19:57:59 In file included from scripts/dtc/libfdt/fdt_ro.c:54:
2019-10-26T19:57:59 /work/beaglebone/host/include/libfdt.h:384:24:
note: previous definition of ?fdt_get_max_phandle? was here
2019-10-26T19:57:59 static inline uint32_t fdt_get_max_phandle(const void *fdt)
2019-10-26T19:57:59 ^~~~~~~~~~~~~~~~~~~
2019-10-26T19:57:59 make[4]: *** [scripts/Makefile.host:107:
scripts/dtc/libfdt/fdt_ro.o] Error 1
2019-10-26T19:57:59 make[3]: *** [scripts/Makefile.build:544:
scripts/dtc] Error 2
2019-10-26T19:57:59 make[2]: *** [Makefile:1066: scripts] Error 2
2019-10-26T19:57:59 make[1]: *** [package/pkg-generic.mk:241:
/work/beaglebone/build/linux-4dae378bbe721277b08699d1d88ffae12acc9b09/.stamp_built]
Error 2
2019-10-26T19:57:59 make: *** [Makefile:23: _all] Error 2

-- 
Carlos Santos <unixmania@gmail.com>

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

* [Buildroot] [PATCH 1/1] package/dtc: Bump to v1.5.1
  2019-10-26 18:41 ` Matthew Weber
@ 2019-10-27  9:11   ` Titouan Christophe
  0 siblings, 0 replies; 7+ messages in thread
From: Titouan Christophe @ 2019-10-27  9:11 UTC (permalink / raw)
  To: buildroot

Hi Matt,

On 10/26/19 8:41 PM, Matthew Weber wrote:
> Titouan,
> 
> On Fri, Oct 25, 2019 at 3:08 PM Titouan Christophe
> <titouan.christophe@railnova.eu> wrote:
>>
>> diff --git a/package/dtc/dtc.hash b/package/dtc/dtc.hash
>> index 4d0aaea601..3adfcc45a9 100644
>> --- a/package/dtc/dtc.hash
>> +++ b/package/dtc/dtc.hash
>> @@ -1,5 +1,5 @@
>>   # from https://www.kernel.org/pub/software/utils/dtc/sha256sums.asc
>> -sha256 c672e443c9f7e39f5a7c8e602da6777f9ad55ad70de87de300a43828c8050172  dtc-1.5.0.tar.xz
>> +sha256 660b74039690fc37013660544d09191834efb58503c73c555c5513ba75ab031f  dtc-1.5.1.tar.xz
>>
>>   # Hash for license files
>>   sha256 171ce1c292cf148a5be9ff95ce7e5c3d79a002721ab8e119eae3c0f5336d2b3d  README.license
> 
>>>> dtc 1.5.1 Collecting legal info
> ERROR: README.license has wrong sha256 hash:
> ERROR: expected:
> 171ce1c292cf148a5be9ff95ce7e5c3d79a002721ab8e119eae3c0f5336d2b3d
> ERROR: got     :
> 78e1e1ce5728a990d59c7de61d4741c49185d5129daf783b386380a214328d68

Indeed, I have to shamely admit I almost never check that. Thank you for 
pointing that out. In the meantime, Fabrice did provide a fix, but I 
also added a pre-commit hook to avoid this problem in the future.

> 
>> diff --git a/package/dtc/dtc.mk b/package/dtc/dtc.mk
>> index 0ed914ba41..ce094cfb86 100644
>> --- a/package/dtc/dtc.mk
>> +++ b/package/dtc/dtc.mk
>> @@ -4,7 +4,7 @@
>>   #
>>   ################################################################################
>>
>> -DTC_VERSION = 1.5.0
>> +DTC_VERSION = 1.5.1
>>   DTC_SOURCE = dtc-$(DTC_VERSION).tar.xz
>>   DTC_SITE = https://www.kernel.org/pub/software/utils/dtc
>>   DTC_LICENSE = GPL-2.0+ or BSD-2-Clause (library)
>> --
>> 2.21.0
>>
>> _______________________________________________
>> buildroot mailing list
>> buildroot at busybox.net
>> http://lists.busybox.net/mailman/listinfo/buildroot
> 

Best,

Titouan

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

* [Buildroot] [PATCH 1/1] package/dtc: Bump to v1.5.1
  2019-10-25 20:00 [Buildroot] [PATCH 1/1] package/dtc: Bump to v1.5.1 Titouan Christophe
  2019-10-26 13:39 ` Thomas Petazzoni
  2019-10-26 18:41 ` Matthew Weber
@ 2019-11-05 21:20 ` Peter Korsgaard
  2019-11-05 21:43   ` Titouan Christophe
  2 siblings, 1 reply; 7+ messages in thread
From: Peter Korsgaard @ 2019-11-05 21:20 UTC (permalink / raw)
  To: buildroot

>>>>> "Titouan" == Titouan Christophe <titouan.christophe@railnova.eu> writes:

 > Remove 0002-Makefile-Add-EXTRA_CFLAGS-variable.patch which is now
 > applied upstream

 > Signed-off-by: Titouan Christophe <titouan.christophe@railnova.eu>

Given that this breaks a number of defconfigs I have reverted this for
2019.11.

We can find a solution and add it again in next.

-- 
Bye, Peter Korsgaard

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

* [Buildroot] [PATCH 1/1] package/dtc: Bump to v1.5.1
  2019-11-05 21:20 ` Peter Korsgaard
@ 2019-11-05 21:43   ` Titouan Christophe
  0 siblings, 0 replies; 7+ messages in thread
From: Titouan Christophe @ 2019-11-05 21:43 UTC (permalink / raw)
  To: buildroot

Hi Peter and all,

On 11/5/19 10:20 PM, Peter Korsgaard wrote:
>>>>>> "Titouan" == Titouan Christophe <titouan.christophe@railnova.eu> writes:
> 
>   > Remove 0002-Makefile-Add-EXTRA_CFLAGS-variable.patch which is now
>   > applied upstream
> 
>   > Signed-off-by: Titouan Christophe <titouan.christophe@railnova.eu>
> 
> Given that this breaks a number of defconfigs I have reverted this for
> 2019.11.

Good decision

> 
> We can find a solution and add it again in next.
> 
Carlos has suggested a workaround there 
http://patchwork.ozlabs.org/patch/1188569/, do you think it's worth 
going in that direction ?

Regards,

Titouan

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

end of thread, other threads:[~2019-11-05 21:43 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-25 20:00 [Buildroot] [PATCH 1/1] package/dtc: Bump to v1.5.1 Titouan Christophe
2019-10-26 13:39 ` Thomas Petazzoni
2019-10-26 23:05   ` Carlos Santos
2019-10-26 18:41 ` Matthew Weber
2019-10-27  9:11   ` Titouan Christophe
2019-11-05 21:20 ` Peter Korsgaard
2019-11-05 21:43   ` Titouan Christophe

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.