Thomas,<br>     I&#39;ll work on making the changes a little later. I&#39;m in the middle of a test build for some changes that I&#39;m making in other areas.<br><br>Martin<br><br><div class="gmail_quote">On Tue, Sep 14, 2010 at 12:54 PM, Thomas Petazzoni <span dir="ltr">&lt;<a href="mailto:thomas.petazzoni@free-electrons.com">thomas.petazzoni@free-electrons.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hello Martin,<br>
<br>
Thanks for your patchset !<br>
<br>
On Mon, 13 Sep 2010 20:40:47 -0700<br>
Martin Banky &lt;<a href="mailto:martin.banky@gmail.com">martin.banky@gmail.com</a>&gt; wrote:<br>
<br>
&gt; +BISON_DIR:=$(BUILD_DIR)/bison-$(BISON_VERSION)<br>
<br>
This is not needed and shouldn&#39;t be there.<br>
<br>
&gt;  define BISON_DISABLE_EXAMPLES<br>
&gt;       echo &#39;all install:&#39; &gt; $(@D)/examples/Makefile<br>
&gt;  endef<br>
&gt;<br>
&gt; +define BISON_NO_WCHAR_SUPPORT<br>
&gt; +     toolchain/patch-kernel.sh $(BISON_DIR) package/bison/bison-no-wchar/ \*.patch<br>
&gt; +endef<br>
&gt; +<br>
&gt; +ifneq ($(BR2_USE_WCHAR),y)<br>
&gt; +BISON_POST_CONFIGURE_HOOKS += BISON_NO_WCHAR_SUPPORT<br>
&gt; +endif<br>
<br>
Personally, I&#39;d prefer not to have patches applied conditionally. It&#39;s<br>
the patch itself that should do the right thing depending on whether<br>
wchar support is available or not, so that the patch can eventually be<br>
sent to the upstream project.<br>
<br>
Thanks!<br>
<br>
Thomas<br>
--<br>
<font color="#888888">Thomas Petazzoni, Free Electrons<br>
Kernel, drivers, real-time and embedded Linux<br>
development, consulting, training and support.<br>
<a href="http://free-electrons.com" target="_blank">http://free-electrons.com</a><br>
_______________________________________________<br>
buildroot mailing list<br>
<a href="mailto:buildroot@busybox.net">buildroot@busybox.net</a><br>
<a href="http://lists.busybox.net/mailman/listinfo/buildroot" target="_blank">http://lists.busybox.net/mailman/listinfo/buildroot</a><br>
</font></blockquote></div><br>