<div dir="ltr">On Fri, Jul 12, 2013 at 5:22 PM, Jack O'Quin <span dir="ltr"><<a href="mailto:jack.oquin@gmail.com" target="_blank">jack.oquin@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<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"><div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">

<div><div class="h5">On Fri, Jul 12, 2013 at 4:45 PM, Piyush <span dir="ltr"><<a href="mailto:piyushk@gmail.com" target="_blank">piyushk@gmail.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"><div dir="ltr">Hi folks,<div><br></div><div>I just realized that one of my packages built despite missing upstream dependencies. v0.1.0 did not depend on these missing deps, and they were introduced in v0.1.1. The upstream deps include navigation, which AFAIK has not seen a successful build.</div>




<div><br></div><div><a href="http://jenkins.willowgarage.com:8080/view/HbinP32/job/ros-hydro-segbot-navigation_binarydeb_precise_i386/" target="_blank">http://jenkins.willowghttp://jenkins.willowgarage.com:8080/view/HbinP32/job/ros-hydr and o-segbot-navigation_binarydeb_precise_i386/arage.com:8080/view/HbinP32/job/ros-hydro-segbot-navigation_binarydeb_precise_i386/</a></div>




<div><br></div><div>Is this something to worry about?</div></div></blockquote><div><br></div></div></div><div>Probably. </div><div><br></div><div>Since "navigation" was a rosbuild stack, now a metapackage, only other metapackages are allowed to depend on it. Since the devel version of segbot_navigation is a regular catkin package, that probably ought to generate an error.</div>


<div><br></div><div>I that dependency needed at that level?</div></div></div></div></blockquote><div><br></div><div>I was not aware of this, and missed it during catkinization. The package only has a real launch dependency on move_base, eband_local_planner and navfn. I will change this with the next release.</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"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">

<div><br></div><div>I suppose eband_local_planner must declare all the navigation packages that it actually uses.</div></div></div></div></blockquote><div><br></div><div>It looks like I did fix the dependencies for eband_local_planner correctly, and segbot_navigation has a dependency on eband that may have gotten bypassed.</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"><div dir="ltr"><div class="gmail_extra"><span class=""><font color="#888888">-- <br>

 joq
</font></span></div></div>
</blockquote></div><br></div></div>