<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Apr 9, 2014 at 10:21 PM, Thomas Moulard <span dir="ltr"><<a href="mailto:thomas.moulard@gmail.com" target="_blank">thomas.moulard@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Thu, Apr 10, 2014 at 1:48 PM, William Woodall<br>
<<a href="mailto:william@osrfoundation.org">william@osrfoundation.org</a>> wrote:<br>
> I would say no, I think it should set the flag in the rosdebian case and do<br>
> nothing in the other case, because if we have to set the value for the plain<br>
> debian case then why do we have GNUInstallDirs.cmake being used at all?<br>
<br>
</div>Thanks for the feeback. See here then (again, untested for now):<br>
<a href="https://github.com/thomas-moulard/bloom/commit/00e977421dc86bfffc09ec51a2ae19d49a80212d" target="_blank">https://github.com/thomas-moulard/bloom/commit/00e977421dc86bfffc09ec51a2ae19d49a80212d</a></blockquote><div>

<br></div><div>Other than implementation details which I commented on, I think that would be fine.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>


<br>
Would this one be ok?<br>
<br>
Regarding GNUInstallDirs.cmake, the goal of this script is to make<br>
CMake install step<br>
more configurable. The part where it tries to guess libdir (and fail)<br>
is for me a big mistake in the<br>
design of this macro. For instance, there is no way I want multiarch<br>
support for the software<br>
I install in my home, even if I am running Debian :)<br>
<div class=""><br>
> Also, this assumes that when you are running bloom you are on a debian<br>
> machine.<br>
<br>
</div>Ok I didn't know cross-releasing was allowed/supported by Bloom.<br>
<br>
[1] <a href="https://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-Standards-Version" target="_blank">https://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-Standards-Version</a><br>
<span class="HOEnZb"><font color="#888888">--<br>
Thomas<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>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>
</div></div>