<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/121916/">https://git.reviewboard.kde.org/r/121916/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On January 11th, 2015, 8:42 a.m. MST, <b>Etienne Rebetez</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Ship It!</p></pre>
 </blockquote>




 <p>On January 11th, 2015, 12:19 p.m. MST, <b>Shalin Shah</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Thanks for giving me developer right. So I should merge my branch with master which is Applications/14.12, right? Also, can you help me out with merging patch? How should I go about it?</p></pre>
 </blockquote>





 <p>On January 11th, 2015, 2:15 p.m. MST, <b>Jeremy Whiting</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">master branch is called "master" Applications/14.12 branch is where point releases will come from until master is branched for Applications/15.03 or whatever the next major version is going to be called.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">As for how to merge, I typically check that my local master is up to date, then rebase the branch on master, then checkout master and merge the branch. Then push to master.</p></pre>
 </blockquote>





 <p>On January 11th, 2015, 2:17 p.m. MST, <b>Jeremy Whiting</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Note that since this is a bugfix it should go to Applications/14.12 so it will be in the next release from that branch which will be tagged on Jan 29th. But it also needs to go to master.</p></pre>
 </blockquote>





 <p>On January 11th, 2015, 2:21 p.m. MST, <b>Etienne Rebetez</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Yes, what Jeremy is saying:-)
I think you should read this: https://techbase.kde.org/Development/Git.</p></pre>
 </blockquote>





 <p>On January 12th, 2015, 10:54 a.m. MST, <b>Shalin Shah</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">So, I merged patch, committed and tried to push to master. But I get error. </p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">shalin@Shalin-MacBookPro:~/kde/kde/kalzium > git push git@git.kde.org:kalzium master
Counting objects: 34, done.
Compressing objects: 100% (4/4), done.
Writing objects: 100% (4/4), 526 bytes | 0 bytes/s, done.
Total 4 (delta 3), reused 0 (delta 0)
remote: Audit failure - Commit 1ec602534fd1d40f2b967027200d9b0a3af5bc72 - Non-full name: shalinshah1993
remote: Audit failure - Commit 1ec602534fd1d40f2b967027200d9b0a3af5bc72 - Non-full name: shalinshah1993
remote: FATAL: VREF/kde-update-checks: helper program exit status 256
remote: Push declined - commits failed audit
remote: error: hook declined to update refs/heads/master
To git@git.kde.org:kalzium
 ! [remote rejected] master -> master (hook declined)
error: failed to push some refs to 'git@git.kde.org:kalzium'</p></pre>
 </blockquote>








</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Yep, you need to set the author full name, set that up in your .gitconfig with git config user.name = "your name" or whatnot, then git commit --amend --reset-author to set the author to your new values.</p></pre>
<br />










<p>- Jeremy</p>


<br />
<p>On January 8th, 2015, 4:38 a.m. MST, Shalin Shah wrote:</p>









<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
 <tr>
  <td>

<div>Review request for KDE Edu and Etienne Rebetez.</div>
<div>By Shalin Shah.</div>


<p style="color: grey;"><i>Updated Jan. 8, 2015, 4:38 a.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
kalzium
</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Color of the pse elements is updated as soon as the preferences for gradient color or scheme color are changed. </p></pre>
  </td>
 </tr>
</table>



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>src/kalzium.cpp <span style="color: grey">(2e85cf8)</span></li>

</ul>

<p><a href="https://git.reviewboard.kde.org/r/121916/diff/" style="margin-left: 3em;">View Diff</a></p>






  </td>
 </tr>
</table>








  </div>
 </body>
</html>