Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: McClintock Matthew-B29882 <B29882@freescale.com>,
	Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH 2/5] m4: Add m4-nativesdk recipes
Date: Tue, 29 Nov 2011 18:04:20 +0000	[thread overview]
Message-ID: <1322589860.7964.7.camel@ted> (raw)
In-Reply-To: <CAEsOVNcZYhLayMYXcwm8fYFpqJ=dVTfbyDjvM+Ox_qqVOmEn_w@mail.gmail.com>

On Tue, 2011-11-29 at 17:48 +0000, McClintock Matthew-B29882 wrote:
> On Tue, Nov 29, 2011 at 2:38 AM,  <wenzong.fan@windriver.com> wrote:
> > From: Wenzong Fan <wenzong.fan@windriver.com>
> >
> > [YOCTO #1603]
> >
> > m4-nativesdk is required by autoconf-nativesdk.
> 
> Is there a reason we don't just extend the m4 recipe for native and
> nativesdk? (I'm just curious)

Its a good question, BBCLASSEXTEND = "nativesdk" should really work and
those recipes look wrong (they shouldn't need to set
INHIBIT_AUTOTOOLS_DEPS = "1" or have the custom do_configure).

The m4-native recipe is a little tricker as its one of the early
bootstrap dependencies required by autotools. This is why it does need
the INHIBIT_AUTOTOOLS_DEPS and custom do_configure line although those
should be possible with modern BBCLASSEXTEND...

Cheers,

Richard




  reply	other threads:[~2011-11-29 18:11 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-29  8:38 [PATCH 0/5] add autotools nativesdk to meta-toolchain wenzong.fan
2011-11-29  8:38 ` [PATCH 1/5] gnu-config: Extend to provide nativesdk recipe wenzong.fan
2011-11-29  8:38 ` [PATCH 2/5] m4: Add m4-nativesdk recipes wenzong.fan
2011-11-29 17:48   ` McClintock Matthew-B29882
2011-11-29 18:04     ` Richard Purdie [this message]
2011-11-30  1:55       ` wenzong fan
2011-11-29  8:39 ` [PATCH 3/5] automake: Extend to provide nativesdk recipe wenzong.fan
2011-11-29  8:39 ` [PATCH 4/5] autoconf: " wenzong.fan
2011-11-29  8:49   ` Koen Kooi
2011-11-29  9:00     ` wenzong fan
2011-11-29  9:12       ` Richard Purdie
2011-11-29  9:14       ` Koen Kooi
2011-11-29  9:22         ` wenzong fan
2011-11-29  8:39 ` [PATCH 5/5] task-sdk-host-nativesdk: add autotools nativesdk to meta-toolchain wenzong.fan

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=1322589860.7964.7.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=B29882@freescale.com \
    --cc=openembedded-core@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox