<p dir="ltr"><br>
On Feb 13, 2013 4:28 PM, "Arnout Vandecappelle" <<a href="mailto:arnout@mind.be">arnout@mind.be</a>> wrote:<br>
><br>
> On 13/02/13 14:54, Joe Halpin wrote:<br>
>><br>
>> This is a patch to add a pre-build script, which runs before the build<br>
>> starts. This lets us setup symlinks and such before the build starts,<br>
>> which makes things easier and cleaner for us. Maybe it will for other<br>
>> people too. This was made from the 20.12.11 release.<br>
><br>
><br>
>  Could you clarify a bit more what the use case is? Then we can consider if there are maybe better ways to achieve it.<br>
><br>
><br>
>  Also, a properly formatted patch has a commit message which is formatted as follows: a single summary line of max. 80 characters, an empty line, one or more paragraphs explaining why the patch is necessary, your Signed-off-by line, a line containing three dashes, the patch changelog (if you resubmit it). Look at other patches on the list for examples. And you should send the patch with git send-email or similar, so that whitespace is retained and we can comment in-line.<br>

><br>
><br>
>  Regarding the patch itself: why do you pass $(TARGET_DIR) as an argument? Since the 'dirs' step comes after the 'prepare' step, $(TARGET_DIR) doesn't exist yet...<br>
><br>
><br>
>  Regards,<br>
>  Arnout<br>
><br>
><br>
> -- <br>
> Arnout Vandecappelle                          arnout at mind be<br>
> Senior Embedded Software Architect            +32-16-286500<br>
> Essensium/Mind                                <a href="http://www.mind.be">http://www.mind.be</a><br>
> G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven<br>
> LinkedIn profile: <a href="http://www.linkedin.com/in/arnoutvandecappelle">http://www.linkedin.com/in/arnoutvandecappelle</a><br>
> GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F<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">http://lists.busybox.net/mailman/listinfo/buildroot</a></p>
<p dir="ltr">I have a use case for this.  In Microblaze builds, it would be nice to generate optimal cflags from the dts/mhs for use in all software.  This could be accomplished by a pre script.</p>
<p dir="ltr">Spenser</p>