<br><div class="gmail_quote">2011/10/26 Arnout Vandecappelle <span dir="ltr"><<a href="mailto:arnout@mind.be">arnout@mind.be</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

On Monday, October 24, 2011 15:35:32 Peter Korsgaard wrote:<br>
[snip]<br>
<div class="im">>  Samuel> +# Build type<br>
>  Samuel> +OPENCV_CONF_OPT += -DCMAKE_BUILD_TYPE=$(if<br>
> $(BR2_ENABLE_DEBUG),Debug,Release)<br>
><br>
> While this works it looks quite different than other packages - We<br>
> normally do:<br>
><br>
> OPENCV_CONF_OPT += \<br>
>                 blah=.. \<br>
>                 blih=.. \<br>
>                 bloh=..<br>
><br>
> ifeq ($(BR2_PACKAGE_BLAH))<br>
> OPENCV_CONF_OPT += BLAH=on<br>
> OPENCV_DEPENDENCIES += blah<br>
> else<br>
> OPENCV_CONF_OPT += BLAH=OFF<br>
> endif<br>
><br>
> ..<br>
><br>
> I've rewritten it to use that form.<br>
<br></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"> Actually, for that particular one you haven't :-)<br>
<br>
 Also I think it's important to use<br>
<br>
ifeq ($(BR2_PACKAGE_BLAH),y)<br></blockquote><div><br></div><div>Actually, in the 1st version of the patch it was this form, then I changed it following the comments of Thomas P. ...</div><div><br></div><div>Using one or the other form does not change anything, it is just a matter of consistency.</div>

<div><br></div><div>Defining some rules like these can certainly be gathered into a "developer guide-line" or something like that.</div><div>If everybody follows the same form/standard (whatever you call it) it will make easier processing submitted patches.</div>

<div>It could also allow to develop some checker scripts.</div><div><br></div><div>Regards,</div><div><br></div><div>Sam</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">


<br>
 so you can easily disable it on the command line with<br>
<br>
make BR2_PACKAGE_BLAH=n<br>
<br>
(Of course, 'make BR2_PACKAGE_BLAH=' would still work, but it's less<br>
intuitive.)<br>
<br>
 Regards,<br>
 Arnout<br>
<font color="#888888"><br>
<br>
--<br>
Arnout Vandecappelle                               arnout at mind be<br>
Senior Embedded Software Architect                 <a href="tel:%2B32-16-286540" value="+3216286540">+32-16-286540</a><br>
Essensium/Mind                                     <a href="http://www.mind.be" target="_blank">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" target="_blank">http://www.linkedin.com/in/arnoutvandecappelle</a><br>
GPG fingerprint:  31BB CF53 8660 6F88 345D  54CC A836 5879 20D7 CF43<br>
</font></blockquote></div><br>