All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Eggleton <paul.eggleton@linux.intel.com>
To: Koen Kooi <koen@dominion.thruhere.net>
Cc: openembedded-devel@lists.openembedded.org
Subject: Re: [meta-webserver][PATCH 6/6] xdebug: add new recipe
Date: Wed, 03 Oct 2012 14:08:52 +0100	[thread overview]
Message-ID: <2127422.3DHfZaTr9h@helios> (raw)
In-Reply-To: <k4h8l4$2d0$1@ger.gmane.org>

On Wednesday 03 October 2012 13:47:52 Koen Kooi wrote:
> >> Why not do autotools_do_configure?
> > 
> > Because AFAICT PHP modules are somewhat unusual when it comes to
> > configure, and totally break if you try to autoreconf. IIRC this was the
> > only way I could get the thing to build.
> 
> Can you add a comment saying that? A lot of recipes in OE classic replace
> the configure step, but our autotools support has improved a lot, so a
> comment saying what is going wrong will help people updating the recipe.

That's a good point, I've added a comment within do_configure on the branch 
noting why it's been done that way.

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre



  reply	other threads:[~2012-10-03 13:21 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-01 16:14 [meta-webserver][PATCH 0/6] Add meta-webserver Paul Eggleton
2012-10-01 16:14 ` [meta-webserver][PATCH 1/6] Add meta-webserver layer Paul Eggleton
2012-10-01 16:14 ` [meta-webserver][PATCH 2/6] apache2: add from OE-Classic Paul Eggleton
2012-10-03 10:17   ` Koen Kooi
2012-10-03 11:28     ` Paul Eggleton
2012-10-03 11:41       ` Koen Kooi
2012-10-03 12:23         ` Paul Eggleton
2012-10-01 16:14 ` [meta-webserver][PATCH 3/6] modphp: " Paul Eggleton
2012-10-03 10:20   ` Koen Kooi
2012-10-03 11:28     ` Paul Eggleton
2012-10-03 11:44       ` Koen Kooi
2012-10-01 16:14 ` [meta-webserver][PATCH 4/6] apache2: update to version 2.4.2 and fix Paul Eggleton
2012-10-01 16:14 ` [meta-webserver][PATCH 5/6] modphp: update to 5.3.14 " Paul Eggleton
2012-10-01 16:14 ` [meta-webserver][PATCH 6/6] xdebug: add new recipe Paul Eggleton
2012-10-03 10:21   ` Koen Kooi
2012-10-03 11:28     ` Paul Eggleton
2012-10-03 11:47       ` Koen Kooi
2012-10-03 13:08         ` Paul Eggleton [this message]
2012-10-02 14:40 ` [meta-webserver][PATCH 0/6] Add meta-webserver Koen Kooi
2012-10-02 14:53   ` Paul Eggleton
2012-10-03 10:24 ` Koen Kooi
2012-10-03 10:43   ` Paul Eggleton
2012-10-03 11:27     ` Koen Kooi
2012-10-03 11:29       ` Paul Eggleton
2012-10-03 11:43         ` Koen Kooi
2012-10-03 12:23           ` Paul Eggleton
2012-10-03 13:12             ` Koen Kooi
2012-10-03 13:43               ` Paul Eggleton
2012-10-03 14:08                 ` McClintock Matthew-B29882
2012-10-03 14:18                   ` Paul Eggleton
2012-10-03 14:19                   ` Khem Raj

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=2127422.3DHfZaTr9h@helios \
    --to=paul.eggleton@linux.intel.com \
    --cc=koen@dominion.thruhere.net \
    --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.