<div dir="ltr"><div>I believe the buildfarm already runs Release.  </div><div><br></div><div>In general you don't want to set the build type from CMake that should be a setting of the environment so that the person building the packages can choose what build type they want.  It is very ugly to be required to patch the source of a repo to change the build type.  As we move toward supporting cross compilation and leveraging the out of source builds, we need to get better about not putting build parameters into the source.  <br>

</div><div><br></div><div>The correct place to force this setting for the debians for example is in the debian/rules file.  </div><div><br></div><div>Tully</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">

On Thu, Jul 18, 2013 at 9:24 AM, William Woodall <span dir="ltr"><<a href="mailto:william@osrfoundation.org" target="_blank">william@osrfoundation.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr">If you set the CMAKE_BUILD_TYPE, you should do it in a already defined guard:<div><br></div><div>if(NOT DEFINED CMAKE_BUILD_TYPE)</div><div>  set(CMAKE_BUILD_TYPE Release)</div><div>endif()</div><div><br></div>


<div>That way it can still be explicitly overridden if desired.</div></div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">On Thu, Jul 18, 2013 at 12:31 AM, Ruben Smits <span dir="ltr"><<a href="mailto:ruben.smits@intermodalics.eu" target="_blank">ruben.smits@intermodalics.eu</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Austin,<br><div class="gmail_extra"><br><br><div class="gmail_quote"><div>On Thu, Jul 18, 2013 at 4:25 AM, Austin Hendrix <span dir="ltr"><<a href="mailto:ahendrix@willowgarage.com" target="_blank">ahendrix@willowgarage.com</a>></span> wrote:<br>




<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">I have a few packages containing realtime controllers that were<br>





previously built with:<br>
set(ROS_BUILD_TYPE Release), which among other things enabled<br>
optimizations as part of the build process.<br>
<br>
Since the catkin doesn't supply a replacement for ROS_BUILD_TYPE, I've<br>
omitted it when catkinizing packages, but I'm discovering now that<br>
packages compiled on the build farm are not sufficiently optimized for<br>
my use case.<br>
<br>
Is there a way to enable optimizations or the Release build type,<br>
either on the farm as a whole, in bloom, or in my packages?<br>
<br></blockquote><div><br></div></div><div>For plain cmake packages or catkin you can use the CMAKE_BUILD_TYPE variable which takes the exact same values as rosbuilds ROS_BUILD_TYPE variable:</div><div><br></div><div>set(CMAKE_BUILD_TYPE Release)<br>




</div><div><br></div><div>see <a href="http://www.cmake.org/Wiki/CMake_Useful_Variables#Compilers_and_Tools" target="_blank">http://www.cmake.org/Wiki/CMake_Useful_Variables#Compilers_and_Tools</a> for more info</div><div>


<br></div><div>

Ruben</div><div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
-Austin<br>
_______________________________________________<br>
Ros-release mailing list<br>
<a href="mailto:Ros-release@code.ros.org" target="_blank">Ros-release@code.ros.org</a><br>
<a href="https://code.ros.org/mailman/listinfo/ros-release" target="_blank">https://code.ros.org/mailman/listinfo/ros-release</a><br>
</blockquote></div></div><span><font color="#888888"><br><br clear="all"><div><br></div>-- <br>Ruben Smits, Phd<br>Chief Technology Officer<br>Intermodalics BVBA<br><a href="tel:%2B32479511786" value="+32479511786" target="_blank">+32479511786</a><br>


<a href="http://www.intermodalics.eu" target="_blank">www.intermodalics.eu</a>
</font></span></div></div>
<br>_______________________________________________<br>
Ros-release mailing list<br>
<a href="mailto:Ros-release@code.ros.org" target="_blank">Ros-release@code.ros.org</a><br>
<a href="https://code.ros.org/mailman/listinfo/ros-release" target="_blank">https://code.ros.org/mailman/listinfo/ros-release</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br></div></div><span class="HOEnZb"><font color="#888888">William Woodall<div>ROS Development Team</div><div><a href="mailto:william@osrfoundation.org" target="_blank">william@osrfoundation.org</a></div>

<div><a href="http://williamjwoodall.com/" target="_blank">http://williamjwoodall.com/</a></div>

</font></span></div>
<br>_______________________________________________<br>
Ros-release mailing list<br>
<a href="mailto:Ros-release@code.ros.org">Ros-release@code.ros.org</a><br>
<a href="https://code.ros.org/mailman/listinfo/ros-release" target="_blank">https://code.ros.org/mailman/listinfo/ros-release</a><br>
<br></blockquote></div><br></div>