Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] usb_modeswitch: disable parallel build
@ 2018-11-22 18:07 Fabrice Fontaine
  2018-11-24 12:03 ` Thomas Petazzoni
  2018-11-26 17:57 ` Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Fabrice Fontaine @ 2018-11-22 18:07 UTC (permalink / raw)
  To: buildroot

Build of package will sometime fails because of the following issue:
install-static target has two dependencies: dispatcher-static and
install-common

Because dispatcher-static is not a file but only a target, it will
always be called to build usb_modeswitch_dispatcher.
So, even if install-common depends on usb_modeswitch_dispatcher, in some
rare cases, install-static won't be able to install
usb_modeswitch_dispatcher because it is being rebuild by
dispatcher-static

To fix this issue, disable parallel build

Fixes:
 - http://autobuild.buildroot.org/results/8297be35725b816ff5afaf909605ceb41223efb6

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
 package/usb_modeswitch/usb_modeswitch.mk | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/package/usb_modeswitch/usb_modeswitch.mk b/package/usb_modeswitch/usb_modeswitch.mk
index d0c33cc187..bb0d6ab1bf 100644
--- a/package/usb_modeswitch/usb_modeswitch.mk
+++ b/package/usb_modeswitch/usb_modeswitch.mk
@@ -10,6 +10,8 @@ USB_MODESWITCH_SITE = http://www.draisberghof.de/usb_modeswitch
 USB_MODESWITCH_DEPENDENCIES = libusb
 USB_MODESWITCH_LICENSE = GPL-2.0+
 USB_MODESWITCH_LICENSE_FILES = COPYING
+#?Package does not build in parallel due to improper make rules
+USB_MODESWITCH_MAKE = $(MAKE1)
 
 USB_MODESWITCH_BUILD_TARGETS = static
 USB_MODESWITCH_INSTALL_TARGETS = install-static
-- 
2.14.1

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

* [Buildroot] [PATCH 1/1] usb_modeswitch: disable parallel build
  2018-11-22 18:07 [Buildroot] [PATCH 1/1] usb_modeswitch: disable parallel build Fabrice Fontaine
@ 2018-11-24 12:03 ` Thomas Petazzoni
  2018-11-26 17:57 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2018-11-24 12:03 UTC (permalink / raw)
  To: buildroot

Hello,

On Thu, 22 Nov 2018 19:07:46 +0100, Fabrice Fontaine wrote:
> Build of package will sometime fails because of the following issue:
> install-static target has two dependencies: dispatcher-static and
> install-common
> 
> Because dispatcher-static is not a file but only a target, it will
> always be called to build usb_modeswitch_dispatcher.
> So, even if install-common depends on usb_modeswitch_dispatcher, in some
> rare cases, install-static won't be able to install
> usb_modeswitch_dispatcher because it is being rebuild by
> dispatcher-static
> 
> To fix this issue, disable parallel build
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/8297be35725b816ff5afaf909605ceb41223efb6
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
>  package/usb_modeswitch/usb_modeswitch.mk | 2 ++
>  1 file changed, 2 insertions(+)

Applied to master, thanks. It would be nice to report the issue
upstream. Thanks!

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

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

* [Buildroot] [PATCH 1/1] usb_modeswitch: disable parallel build
  2018-11-22 18:07 [Buildroot] [PATCH 1/1] usb_modeswitch: disable parallel build Fabrice Fontaine
  2018-11-24 12:03 ` Thomas Petazzoni
@ 2018-11-26 17:57 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2018-11-26 17:57 UTC (permalink / raw)
  To: buildroot

>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:

 > Build of package will sometime fails because of the following issue:
 > install-static target has two dependencies: dispatcher-static and
 > install-common

 > Because dispatcher-static is not a file but only a target, it will
 > always be called to build usb_modeswitch_dispatcher.
 > So, even if install-common depends on usb_modeswitch_dispatcher, in some
 > rare cases, install-static won't be able to install
 > usb_modeswitch_dispatcher because it is being rebuild by
 > dispatcher-static

 > To fix this issue, disable parallel build

 > Fixes:
 >  - http://autobuild.buildroot.org/results/8297be35725b816ff5afaf909605ceb41223efb6

 > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Committed to 2018.02.x and 2018.08.x, thanks.

-- 
Bye, Peter Korsgaard

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

end of thread, other threads:[~2018-11-26 17:57 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-11-22 18:07 [Buildroot] [PATCH 1/1] usb_modeswitch: disable parallel build Fabrice Fontaine
2018-11-24 12:03 ` Thomas Petazzoni
2018-11-26 17:57 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox