<div dir="ltr">You have right. file is not closed... This is strange. Perhaps server is in maintenance or something like that.<div><br></div><div>Well close it manually and set FIXED-IN to 5.4.0</div><div><br></div><div>Gilles<br><div class="gmail_extra"><br><div class="gmail_quote">2017-01-03 12:52 GMT+01:00 Gilles Caulier <span dir="ltr"><<a href="mailto:caulier.gilles@gmail.com" target="_blank">caulier.gilles@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">yes, the bug must be closed automatically. this is the goal of git hook scripts.<span class="m_-3128820654479741586HOEnZb"><font color="#888888"><div><br></div><div>Gilles</div></font></span></div><div class="m_-3128820654479741586HOEnZb"><div class="m_-3128820654479741586h5"><div class="gmail_extra"><br><div class="gmail_quote">2017-01-03 12:44 GMT+01:00 Markus Leuthold <span dir="ltr"><<a href="mailto:digikam@titlis.org" target="_blank">digikam@titlis.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Gilles<div><br></div><div>Happy new year for you too! I committed (along with some whitespace fixes and comments).</div><div><br></div><div>After git push, the server responds</div><div><br></div><div><span style="font-family:monospace"><span style="color:rgb(0,0,0)">remote: <a href="https://commits.kde.org/digikam/72c97cef18b260d2143261b72373e7ba53ba3dfb" target="_blank">https://commits.kde.org/digika<wbr>m/72c97cef18b260d2143261b72373<wbr>e7ba53ba3dfb</a> </span><br>remote: Closing bug 296580<br><br></span></div><div>Does that mean the bko entry is closed automatically by a post commit hook? or do I need to do it manually? Right now, the bug is still open on Bugzilla<br></div><div><br></div><div>best, Markus</div></div><div class="m_-3128820654479741586m_2426043754870455170HOEnZb"><div class="m_-3128820654479741586m_2426043754870455170h5"><div class="gmail_extra"><br><div class="gmail_quote">On 3 January 2017 at 09:36, Gilles Caulier <span dir="ltr"><<a href="mailto:caulier.gilles@gmail.com" target="_blank">caulier.gilles@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 dir="ltr">The patch is fine for me. You can commit.<div><br></div><div>Don't forget to include the bugzilla title at end of digikam core NEWS file for next release announcement. Close bugzilla entry with FIXED-IN set to 5.4.0</div><div><br></div><div>Best</div><span class="m_-3128820654479741586m_2426043754870455170m_311731239514675032HOEnZb"><font color="#888888"><div><br></div><div>Gilles</div><div><br></div><div><br><div><br></div><div><br></div></div></font></span></div><div class="m_-3128820654479741586m_2426043754870455170m_311731239514675032HOEnZb"><div class="m_-3128820654479741586m_2426043754870455170m_311731239514675032h5"><div class="gmail_extra"><br><div class="gmail_quote">2017-01-03 9:28 GMT+01:00 Gilles Caulier <span dir="ltr"><<a href="mailto:caulier.gilles@gmail.com" target="_blank">caulier.gilles@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Markus and happy new year 2017<div><br></div><div>reviewboard give duplicates feature than bugzilla. This last one is more powerfull and is the core of Agile project management.</div><div><br></div><div>I'm not favorable to use it. Bugzilla is enough, and optimum.</div><div><br></div><div>I will look in your patch today. Thanks for your contribution.</div><div><br></div><div>Best<span class="m_-3128820654479741586m_2426043754870455170m_311731239514675032m_1852666350014630559HOEnZb"><font color="#888888"><br></font></span></div><span class="m_-3128820654479741586m_2426043754870455170m_311731239514675032m_1852666350014630559HOEnZb"><font color="#888888"><div><br></div><div>Gilles Caulier</div></font></span></div><div class="m_-3128820654479741586m_2426043754870455170m_311731239514675032m_1852666350014630559HOEnZb"><div class="m_-3128820654479741586m_2426043754870455170m_311731239514675032m_1852666350014630559h5"><div class="gmail_extra"><br><div class="gmail_quote">2017-01-03 2:19 GMT+01:00 Markus Leuthold <span dir="ltr"><<a href="mailto:digikam@titlis.org" target="_blank">digikam@titlis.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi all</div><div><br></div><div>could you please review <a href="https://git.reviewboard.kde.org/r/129755/" style="font-size:12.8px;background-color:rgb(249,243,201)" target="_blank">https://git.reviewboard<wbr>.kde.org/r/129755/</a> ? May I commit this?</div><div><br></div><div>thanks!</div><div>Kusi</div><div><br></div><div>PS: How does the review process work nowadays? Is the reviewboard fine or do you have some pull request mechanism?</div><br></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div></div></div>