<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml"><head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
</head><body><p>I think this should not be a warning, but a notice. Much like autoproj status. <br></p><p>-- updating package xyz. Package is 4 commits ahead of pin / package has local changes.<br></p><p>cheers,<br></p><p>Jakob<br></p><blockquote type="cite"><p>Janosch Machowinski <Janosch.Machowinski@dfki.de> hat am 4. Juni 2015 um 11:34 geschrieben:<br><br><br>The Problem is :<br>Local repo is checked out at master / any commit that is ahead of the tag.<br>In this case Autoproj update will not move HEAD back to the tag, but<br>keep the current commit.<br>The Bad scenario now is :<br>1. Package is on master<br>2. Package is broken, and pinned back by another developer<br><br>If you now run an autoproj update you won't notice, that you should<br>be on the pin, but are still on master.<br><br>Also it is annoying in combination with the release.<br>Greetings<br> Janosch<br><br><br>Am 04.06.2015 um 11:17 schrieb Steffen Planthaber:<br>> Hi,<br>><br>> I'm not sure if i understood the problem.<br>><br>> 1. A dev adds a override to develop a package<br>> 2. Another dev pins the package to a commit<br>><br>> 3. On update the pinned commit is checked out without warning?<br>> or<br>> 3. The first dev doesn't recognize the the override?<br>><br>> In the first scenario, there should be a warning.<br>><br>> In the second, the (first) dev will encounter the same problem and<br>> either solve it in his (advanced code) and fix it there, or not<br>> encounter the problem at all. Both is fine for me.<br>><br>> I think in the second case, no warning is required.<br>><br>> Steffen<br>><br>><br>><br>><br>> Am 04.06.2015 um 10:53 schrieb Matthias Goldhoorn:<br>>> Hi Dev's,<br>>> we have a non-consens regarding the behaviour of autobuild.<br>>> The Core question is: Should Autoproj warn if a package is locally newer<br>>> than the given tag/commit-id<br>>><br>>> Please follow the discussion, comment and help us to find a consens:<br>>> https://github.com/rock-core/autobuild/pull/23<br>>><br>>> Best,<br>>> Matthias<br>>><br>>><br>><br><br>_______________________________________________<br>Rock-dev mailing list<br>Rock-dev@dfki.de<br>http://www.dfki.de/mailman/cgi-bin/listinfo/rock-dev<br></p></blockquote></body></html>