<table><tr><td style="">ltoscano added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D9715" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D9715#188743" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;" rel="noreferrer">D9715#188743</a>, <a href="https://phabricator.kde.org/p/poboiko/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@poboiko</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>a) is the fix relevant also for Applications/17.12?</p></blockquote>

<p>I guess so. I have experienced this for some time, it doesn't look like a recent regression.</p></div>
</blockquote>

<p>Oki, so let's see what <a href="https://phabricator.kde.org/p/dvratil/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@dvratil</a> says, if it's worth to cherry-pick it.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;">

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>b) please try to use arc land next time. This way the commit message has no context at all and having to check the review even for basic information about the change is not nice.</p></blockquote>

<p>Sorry! Will provide more details in commit message next time...<br />
 Haven't used the Arcanist before, I'm just more familiar with good old git diff & web interface. I've just found <a href="https://community.kde.org/Infrastructure/Phabricator#Using_Arcanist" class="remarkup-link" target="_blank" rel="noreferrer">this</a> guide; I assume this is the right one.</p></blockquote>

<p>Yes, that's the page. There is also a git-phab command from a 3rd-party, but I think that no one from KDE so far tested it.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R44 KDE PIM Runtime</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9715" rel="noreferrer">https://phabricator.kde.org/D9715</a></div></div><br /><div><strong>To: </strong>poboiko, KDE PIM, dvratil<br /><strong>Cc: </strong>ltoscano, dvratil, mlaurent, dvasin, winterz, vkrause, knauss<br /></div>