From: Martin Jansa <martin.jansa@gmail.com>
To: openembedded-devel@lists.openembedded.org
Cc: Christopher Larson <chris_larson@mentor.com>
Subject: Re: [meta-oe][PATCH 3/5] dnsmasq: fix parallel make issue
Date: Sun, 10 Aug 2014 11:24:31 +0200 [thread overview]
Message-ID: <20140810092431.GP14848@jama> (raw)
In-Reply-To: <df6c707e6db09c6b640084b8a4a69919b147ad07.1407632366.git.chris_larson@mentor.com>
[-- Attachment #1: Type: text/plain, Size: 2605 bytes --]
On Sat, Aug 09, 2014 at 06:01:13PM -0700, Christopher Larson wrote:
> From: Christopher Larson <chris_larson@mentor.com>
Is this v2 of this change which was sent earlier?
commit 9550453f7d1c0773b0e5ba7af478222f55021240
Author: Shrikant Bobade <Shrikant_Bobade@mentor.com>
Date: Mon Aug 4 14:42:57 2014 +0530
dnsmasq: fix parallel make failure
> Signed-off-by: Christopher Larson <chris_larson@mentor.com>
> ---
> .../recipes-support/dnsmasq/dnsmasq_2.68.bb | 2 ++
> .../recipes-support/dnsmasq/files/parallel-make.patch | 19 +++++++++++++++++++
> 2 files changed, 21 insertions(+)
> create mode 100644 meta-networking/recipes-support/dnsmasq/files/parallel-make.patch
>
> diff --git a/meta-networking/recipes-support/dnsmasq/dnsmasq_2.68.bb b/meta-networking/recipes-support/dnsmasq/dnsmasq_2.68.bb
> index b1bef7a..6caa92a 100644
> --- a/meta-networking/recipes-support/dnsmasq/dnsmasq_2.68.bb
> +++ b/meta-networking/recipes-support/dnsmasq/dnsmasq_2.68.bb
> @@ -1,4 +1,6 @@
> require dnsmasq.inc
>
> +SRC_URI += "file://parallel-make.patch"
> +
> SRC_URI[dnsmasq-2.68.md5sum] = "6f8351ca0901f248efdb81532778d2ef"
> SRC_URI[dnsmasq-2.68.sha256sum] = "402019d61f53f1ccc04cc6218719fd3b158ea3fca9a83ab55471f981a2097c2f"
> diff --git a/meta-networking/recipes-support/dnsmasq/files/parallel-make.patch b/meta-networking/recipes-support/dnsmasq/files/parallel-make.patch
> new file mode 100644
> index 0000000..6d5b722
> --- /dev/null
> +++ b/meta-networking/recipes-support/dnsmasq/files/parallel-make.patch
> @@ -0,0 +1,19 @@
> +The dnsmasq target depends on .configured and $(objs). .configured does an rm
> +-f *.o. Yet the only thing telling make to build the .configured target before
> +the $(objs) target was the order of the dependencies of the dnsmasq target. We
> +can't rely on that order when doing a paralllel make build, so add an explicit
> +rule to enforce that order.
> +
> +Signed-off-by: Christopher Larson <chris_larson@mentor.com>
> +Upstream-status: Pending
> +
> +--- dnsmasq-2.68.orig/Makefile
> ++++ dnsmasq-2.68/Makefile
> +@@ -139,6 +139,8 @@ bloatcheck : $(BUILDDIR)/dnsmasq_baselin
> + @rm -f *.o
> + @touch $@
> +
> ++$(objs): .configured
> ++
> + $(objs:.o=.c) $(hdrs):
> + ln -s $(top)/$(SRC)/$@ .
> --
> 1.8.3.4
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 188 bytes --]
next prev parent reply other threads:[~2014-08-10 9:23 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-10 1:01 [meta-oe][PATCH 0/5] Various dnsmasq fixes/improvements Christopher Larson
2014-08-10 1:01 ` [meta-oe][PATCH 1/5] dnsmasq: add missing inherit pkgconfig Christopher Larson
2014-08-10 1:01 ` [meta-oe][PATCH 2/5] dnsmasq: obey CFLAGS & LDFLAGS Christopher Larson
2014-08-10 1:01 ` [meta-oe][PATCH 3/5] dnsmasq: fix parallel make issue Christopher Larson
2014-08-10 9:24 ` Martin Jansa [this message]
2014-08-11 0:36 ` Christopher Larson
2014-08-11 8:46 ` Martin Jansa
2014-08-11 8:51 ` Martin Jansa
2014-08-11 15:54 ` Christopher Larson
2014-08-10 1:01 ` [meta-oe][PATCH 4/5] dnsmasq: look for lua.pc, not lua5.1.pc Christopher Larson
2014-08-10 1:01 ` [meta-oe][PATCH 5/5] dnsmasq: add packageconfigs for idn, conntrack, lua Christopher Larson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140810092431.GP14848@jama \
--to=martin.jansa@gmail.com \
--cc=chris_larson@mentor.com \
--cc=openembedded-devel@lists.openembedded.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.