* [Buildroot] [PATCH] Makefile: work around a bug in newly released make 4.3 @ 2020-03-04 12:53 Yann E. MORIN 2020-03-04 13:04 ` Thomas Petazzoni 2020-03-04 13:41 ` Mircea GLIGA 0 siblings, 2 replies; 4+ messages in thread From: Yann E. MORIN @ 2020-03-04 12:53 UTC (permalink / raw) To: buildroot Several users of rolling-release distributions have been reporting on IRC that Buildroot is broken now that they have switched to the newly released make 4.3. It turns out that the constructs we use to generated and ionclude the internal br2-external related fragments is no longer working with make-4.3. Indeed, an upstream bug report [0] seems to imply that it so far was working by chance. There has been no further feedback, whether this is really considered a fix or a regression... In the meantime, we add a workaround that fixes the issue, and should not break on older make versions either (verified on all relevant versions: 3.81, 3.82, 4.0, 4.1, and 4.2). Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr> --- Makefile | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Makefile b/Makefile index 31624f77a2..c22c1d8980 100644 --- a/Makefile +++ b/Makefile @@ -188,6 +188,9 @@ ifneq ($(BR2_EXTERNAL_ERROR),) $(error $(BR2_EXTERNAL_ERROR)) endif +# Workaround bug #57676 in make-4.3 +$(BASE_DIR)/.br2-external.mk:; + # To make sure that the environment variable overrides the .config option, # set this before including .config. ifneq ($(BR2_DL_DIR),) -- 2.20.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] Makefile: work around a bug in newly released make 4.3 2020-03-04 12:53 [Buildroot] [PATCH] Makefile: work around a bug in newly released make 4.3 Yann E. MORIN @ 2020-03-04 13:04 ` Thomas Petazzoni 2020-03-04 13:25 ` Yann E. MORIN 2020-03-04 13:41 ` Mircea GLIGA 1 sibling, 1 reply; 4+ messages in thread From: Thomas Petazzoni @ 2020-03-04 13:04 UTC (permalink / raw) To: buildroot Hello, On Wed, 4 Mar 2020 13:53:30 +0100 "Yann E. MORIN" <yann.morin.1998@free.fr> wrote: > Several users of rolling-release distributions have been reporting on > IRC that Buildroot is broken now that they have switched to the newly > released make 4.3. > > It turns out that the constructs we use to generated and ionclude the > internal br2-external related fragments is no longer working with > make-4.3. > > Indeed, an upstream bug report [0] seems to imply that it so far was > working by chance. There has been no further feedback, whether this is > really considered a fix or a regression... What does [0] points to ? > > In the meantime, we add a workaround that fixes the issue, and should > not break on older make versions either (verified on all relevant > versions: 3.81, 3.82, 4.0, 4.1, and 4.2). Could you give more details about how the workaround fixes the issue ? > > Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr> > --- > Makefile | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Makefile b/Makefile > index 31624f77a2..c22c1d8980 100644 > --- a/Makefile > +++ b/Makefile > @@ -188,6 +188,9 @@ ifneq ($(BR2_EXTERNAL_ERROR),) > $(error $(BR2_EXTERNAL_ERROR)) > endif > > +# Workaround bug #57676 in make-4.3 Perhaps an http:// link instead of just #57676 ? > +$(BASE_DIR)/.br2-external.mk:; > + > # To make sure that the environment variable overrides the .config option, > # set this before including .config. > ifneq ($(BR2_DL_DIR),) Thanks! Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] Makefile: work around a bug in newly released make 4.3 2020-03-04 13:04 ` Thomas Petazzoni @ 2020-03-04 13:25 ` Yann E. MORIN 0 siblings, 0 replies; 4+ messages in thread From: Yann E. MORIN @ 2020-03-04 13:25 UTC (permalink / raw) To: buildroot Thomas, All, On 2020-03-04 14:04 +0100, Thomas Petazzoni spake thusly: > On Wed, 4 Mar 2020 13:53:30 +0100 > "Yann E. MORIN" <yann.morin.1998@free.fr> wrote: > > > Several users of rolling-release distributions have been reporting on > > IRC that Buildroot is broken now that they have switched to the newly > > released make 4.3. > > > > It turns out that the constructs we use to generated and ionclude the > > internal br2-external related fragments is no longer working with > > make-4.3. > > > > Indeed, an upstream bug report [0] seems to imply that it so far was > > working by chance. There has been no further feedback, whether this is > > really considered a fix or a regression... > What does [0] points to ? Well, to the upstream bug report, obviously! ;-] Fixed, thanks. > > In the meantime, we add a workaround that fixes the issue, and should > > not break on older make versions either (verified on all relevant > > versions: 3.81, 3.82, 4.0, 4.1, and 4.2). > Could you give more details about how the workaround fixes the issue ? Oh yes, I surely can: it was suggested in that bug report! ;-) Honestly, I am not sure I totally understood the reason, or the workaround... > > Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr> > > --- > > Makefile | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/Makefile b/Makefile > > index 31624f77a2..c22c1d8980 100644 > > --- a/Makefile > > +++ b/Makefile > > @@ -188,6 +188,9 @@ ifneq ($(BR2_EXTERNAL_ERROR),) > > $(error $(BR2_EXTERNAL_ERROR)) > > endif > > > > +# Workaround bug #57676 in make-4.3 > Perhaps an http:// link instead of just #57676 ? Even if it is in the commit log now? Regards, Yann E. MORIN. > > +$(BASE_DIR)/.br2-external.mk:; > > + > > # To make sure that the environment variable overrides the .config option, > > # set this before including .config. > > ifneq ($(BR2_DL_DIR),) > > Thanks! > > Thomas > -- > Thomas Petazzoni, CTO, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------' ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Buildroot] [PATCH] Makefile: work around a bug in newly released make 4.3 2020-03-04 12:53 [Buildroot] [PATCH] Makefile: work around a bug in newly released make 4.3 Yann E. MORIN 2020-03-04 13:04 ` Thomas Petazzoni @ 2020-03-04 13:41 ` Mircea GLIGA 1 sibling, 0 replies; 4+ messages in thread From: Mircea GLIGA @ 2020-03-04 13:41 UTC (permalink / raw) To: buildroot Hi Yann, I've tested the patch against master and using a br2-external tree and it works. I've also tested it against buildroot-2019.02.9, but there it needs to be adjusted: `$(BASE_DIR)/.br-external.mk:;` Mircea ________________________________________ From: buildroot <buildroot-bounces@busybox.net> on behalf of Yann E. MORIN <yann.morin.1998@free.fr> Sent: Wednesday, March 4, 2020 14:53 To: buildroot at buildroot.org Cc: Yann E. MORIN Subject: [Buildroot] [PATCH] Makefile: work around a bug in newly released make 4.3 Several users of rolling-release distributions have been reporting on IRC that Buildroot is broken now that they have switched to the newly released make 4.3. It turns out that the constructs we use to generated and ionclude the internal br2-external related fragments is no longer working with make-4.3. Indeed, an upstream bug report [0] seems to imply that it so far was working by chance. There has been no further feedback, whether this is really considered a fix or a regression... In the meantime, we add a workaround that fixes the issue, and should not break on older make versions either (verified on all relevant versions: 3.81, 3.82, 4.0, 4.1, and 4.2). Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr> Tested-by: Mircea Gliga <mgliga@bitdefender.com> --- Makefile | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Makefile b/Makefile index 31624f77a2..c22c1d8980 100644 --- a/Makefile +++ b/Makefile @@ -188,6 +188,9 @@ ifneq ($(BR2_EXTERNAL_ERROR),) $(error $(BR2_EXTERNAL_ERROR)) endif +# Workaround bug #57676 in make-4.3 +$(BASE_DIR)/.br2-external.mk:; + # To make sure that the environment variable overrides the .config option, # set this before including .config. ifneq ($(BR2_DL_DIR),) -- 2.20.1 _______________________________________________ buildroot mailing list buildroot at busybox.net http://lists.busybox.net/mailman/listinfo/buildroot ________________________ This email was scanned by Bitdefender ^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-03-04 13:41 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-03-04 12:53 [Buildroot] [PATCH] Makefile: work around a bug in newly released make 4.3 Yann E. MORIN 2020-03-04 13:04 ` Thomas Petazzoni 2020-03-04 13:25 ` Yann E. MORIN 2020-03-04 13:41 ` Mircea GLIGA
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox