From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <jukka.rissanen@linux.intel.com>
Received: from mga01.intel.com (mga01.intel.com [192.55.52.88])
	by mail.openembedded.org (Postfix) with ESMTP id 665577169F
	for <openembedded-core@lists.openembedded.org>;
	Mon,  4 May 2015 05:52:38 +0000 (UTC)
Received: from orsmga003.jf.intel.com ([10.7.209.27])
	by fmsmga101.fm.intel.com with ESMTP; 03 May 2015 22:52:40 -0700
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.13,364,1427785200"; d="scan'208";a="565730762"
Received: from jrissane-mobl.ger.corp.intel.com ([10.237.67.151])
	by orsmga003.jf.intel.com with ESMTP; 03 May 2015 22:52:38 -0700
Message-ID: <1430718757.2652.61.camel@linux.intel.com>
From: Jukka Rissanen <jukka.rissanen@linux.intel.com>
To: "Burton, Ross" <ross.burton@intel.com>
Date: Mon, 04 May 2015 08:52:37 +0300
In-Reply-To: <CAJTo0LYtz5=Onm6ERW1fomWViwLNpWtaGnPZUYEfE53YNBsRmw@mail.gmail.com>
References: <1430480517-8245-1-git-send-email-jukka.rissanen@linux.intel.com>
	<CAJTo0LYtz5=Onm6ERW1fomWViwLNpWtaGnPZUYEfE53YNBsRmw@mail.gmail.com>
X-Mailer: Evolution 3.12.11 (3.12.11-1.fc21) 
Mime-Version: 1.0
Cc: OE-core <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH] connman: Enable automake dependency tracking
X-BeenThere: openembedded-core@lists.openembedded.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Patches and discussions about the oe-core layer
	<openembedded-core.lists.openembedded.org>
List-Unsubscribe: <http://lists.openembedded.org/mailman/options/openembedded-core>,
	<mailto:openembedded-core-request@lists.openembedded.org?subject=unsubscribe>
List-Archive: <http://lists.openembedded.org/pipermail/openembedded-core/>
List-Post: <mailto:openembedded-core@lists.openembedded.org>
List-Help: <mailto:openembedded-core-request@lists.openembedded.org?subject=help>
List-Subscribe: <http://lists.openembedded.org/mailman/listinfo/openembedded-core>,
	<mailto:openembedded-core-request@lists.openembedded.org?subject=subscribe>
X-List-Received-Date: Mon, 04 May 2015 05:52:42 -0000
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit

Hi Ross,

On pe, 2015-05-01 at 15:03 +0100, Burton, Ross wrote:
> 
> On 1 May 2015 at 12:41, Jukka Rissanen
> <jukka.rissanen@linux.intel.com> wrote:
>         The automake option --disable-dependency-tracking does not
>         work nicely
>         with ConnMan 1.29 release.
>         The reason is ConnMan commits
>         ac332c5d01b0737c18cb58c8ccc67cf6b0427e1d
>         and ea6c21b0b20f6f25b25386285d81d859dc843820 that change how
>         and when
>         various .in files are generated. After those commits, the auto
>         generated files / directories are created during compilation.
>         Earlier
>         they were generated by configure script.
> 
> 
> Yesterday I fixed this the other way: by adding mkdir calls to connman
> so that instead of requiring dependency tracking to be on so that
> directories are generated, the build makes the directories as
> required.
> 
> 
> 970e0ae was where dependency tracking was disabled on the rationale
> that it's slower, has races, and theoretically won't give us an
> advantage as we don't generally rebuild the same source tree
> repeatedly.

Ok, that makes sense.

> 
> 
> I'll submit my fix to connman in a moment.

Thanks, the connman patch looks good to me.


Cheers,
Jukka