Thomas,<br>     Well, what do you know, post patch does work for gentargets! You learn something new every day. I&#39;ll change the packages that are affected and repost.<br>     Ok, I just checked on the cvs Debian patches, and now remember why I didn&#39;t check the sed routine very hard. The patch is over 12,000 lines long, however I just checked the patch and found at least on instance where the patch needs to be fixed. Line 9527, &#39;++++ ../build-tree.new/cvs-1.12.13/src/root.c    2006-05-05 23:34:12.000000000 +0800&#39; is one reason why the sed routine is needed. I haven&#39;t checked the other patch sets, but it kind of seems that the sed routine might be needed after all. Thoughts?<br>

<br>Martin<br><br><div class="gmail_quote">On Tue, Oct 5, 2010 at 11:38 AM, Martin Banky <span dir="ltr">&lt;<a href="mailto:Martin.Banky@gmail.com">Martin.Banky@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Thomas,<br>     I got the Debian patch routine from the cvs package, when I was converting it. I just figured that there was some type of formatting issue in the debian patches that needed to be addressed. If I read it correctly, it looks like it strips any prefix from the name of the package to patch. I must confess, I&#39;m not very good with sed. Is this something that doesn&#39;t need to be done? If so, just let me know, and I&#39;ll change the packages that are affected.<br>


     As to the *_TOOLS_BUILD_CMDS issue, until now, I thought that the hooks were only available for the autotargets packages. I just re-read that section in the Usage and documentation page, and it looks like you can use the hooks in a gentargets package. I&#39;ll try it out, and change the packages that have that issue.<br>

<font color="#888888">
<br>Martin</font><div><div></div><div class="h5"><br><br><div class="gmail_quote">On Tue, Oct 5, 2010 at 2:17 AM, Thomas Petazzoni <span dir="ltr">&lt;<a href="mailto:thomas.petazzoni@free-electrons.com" target="_blank">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,<br>
<br>
Thanks Martin for doing all this conversion work, this is much<br>
appreciated!<br>
<div><br>
On Tue,  5 Oct 2010 01:22:34 -0700<br>
Martin Banky &lt;<a href="mailto:martin.banky@gmail.com" target="_blank">martin.banky@gmail.com</a>&gt; wrote:<br>
<br>
&gt; +define INPUT_TOOLS_DEBIAN_PATCHES<br>
&gt; +     if [ -d $(@D)/debian/patches ]; then \<br>
&gt; +             (cd $(@D)/debian/patches &amp;&amp; for i in *; \<br>
&gt; +              do $(SED) &#39;s,^\+\+\+ .*joystick-$(INPUT_TOOLS_VERSION)/,+++ joystick-$(INPUT_TOOLS_VERSION)/,&#39; $$i; \<br>
<br>
</div>I have seen this in several of your patches, but I don&#39;t understand why<br>
it is here. There are several other packages that do apply the debian<br>
patches, but such a sed step is not needed.<br>
<div><br>
&gt; +define INPUT_TOOLS_BUILD_CMDS<br>
&gt; +     $(INPUT_TOOLS_DEBIAN_PATCHES)<br>
<br>
</div>Even though it doesn&#39;t technically make any difference, I&#39;d prefer to<br>
have this as a post-patch hook.<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" target="_blank">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>
</div></div></blockquote></div><br>