Re: [PATCH] autotools.bbclass: Add functionality to force a clean of ${B} when reconfiguring (and ${S} != ${B})

[Scott Garman <scott.a.garman@intel.com>]

On 09/12/2012 10:47 AM, McClintock Matthew-B29882 wrote:
> On Wed, Sep 12, 2012 at 9:16 AM, Richard Purdie
> <richard.purdie@linuxfoundation.org> wrote:
>> On Tue, 2012-09-11 at 19:01 +0000, McClintock Matthew-B29882 wrote:
>>> On Tue, Sep 11, 2012 at 9:22 AM, Richard Purdie
>>> <richard.purdie@linuxfoundation.org> wrote:
>>>> Unfortunately whilst rerunning configure and make against a project will mostly
>>>> work there are situations where it does not correctly do the right thing.
>>>>
>>>> In particular, eglibc and gcc will fail out with errors where settings
>>>> do not match a previously built configuration. It could be argued they are
>>>> broken but the situation is what it is. There is the possibility of more subtle
>>>> errors too.
>>>>
>>>> This patch adds removal of the build directory (${B}) when configure is
>>>> rerunning, the sstate checksum for do_configure has changed and ${S} != ${B}.
>>>> We could simply use a stamp but saving out the previous configuration checksum
>>>> adds some data at no real overhead.
>>>>
>>>> If we find there are things where we want to disable this behaviour with
>>>> CONFIGURESTAMPFILE = "" in the recipe, or users could disable it globally.
>>>>
>>>> [YOCTO #2774]
>>>> [YOCTO #2848]
>>>>
>>>> This is particularly helpful for eglibc and gcc which use split builds by default and
>>>> are a particular source of reconfigure type problems.
>>>>
>>>> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>>>
>>> Is it feasible to back port this to denzil? I've encountered what I
>>> think are similar issues reconfiguring gcc for example.
>>
>> One of the bugs above is open against denzil and the issue certainly
>> exists there. The patch should apply equally well there.
>>
>> I'd suggest we let this settle in master for a week or two and then add
>> it to the backport queue if no problems arise.
>>
>> Cc'ing Scott so he's aware of this.
>
> I've added this to my denzil branch and will start doing build testing.
>
> http://git.yoctoproject.org/cgit/cgit.cgi/poky-contrib/log/?h=mattsm/denzil

I'll be returning from travel tomorrow and hope to catch up with my 
denzil backlog, and queue up a new build on the autobuilder.

Scott

-- 
Scott Garman
Embedded Linux Engineer - Yocto Project
Intel Open Source Technology Center