git cherry-pick should be ok for the back port.<br><br>Cheers,<br>Jammy<br><br><div class="gmail_quote">On Thu, Mar 10, 2011 at 6:55 AM, Patrick Spendrin <span dir="ltr"><<a href="mailto:ps_ml@gmx.de">ps_ml@gmx.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">Am 09.03.2011 22:26, schrieb Wolfgang Rohdewald:<br>
<div class="im">> Hi,<br>
><br>
><br>
> when compiling kdelibs 4.6 on Windows, I get an<br>
><br>
> error in kinit_win.cpp at line 110 (I am afraid I deleted it)<br>
><br>
><br>
> but the commit in kdelibs below obviously fixes it. So I<br>
><br>
> want to backport it to kdelibs.<br>
><br>
><br>
> Would that be OK, and how would I do it? Like this:<br>
><br>
><br>
> git checkout --track origin/KDE/4.6<br>
><br>
> git cherry-pick fd16288<br>
><br>
> git push<br>
><br>
><br>
> or would I violate dozens of rules by doing so?<br>
<br>
</div>No idea about the git commands, but backporting would actually be good.<br>
you can always just apply the patch by hand as we did with svn back then.<br>
<br>
regards,<br>
<font color="#888888">Patrick<br>
</font><div><div></div><div class="h5"><br>
><br>
><br>
><br>
> i5:~/src/kdelibs.was/kinit$ git show fd16288<br>
><br>
> commit fd16288cedadf8ccf469df8a7be296763f5003d8<br>
><br>
> Author: Patrick Spendrin <<a href="mailto:ps_ml@gmx.de">ps_ml@gmx.de</a>><br>
><br>
> Date: Sun Feb 20 21:39:20 2011 +0100<br>
><br>
><br>
> we need to cast by hand from wchar to ushort<br>
><br>
><br>
> diff --git a/kinit/kinit_win.cpp b/kinit/kinit_win.cpp<br>
><br>
> index 17266a4..9c1521a 100644<br>
><br>
> --- a/kinit/kinit_win.cpp<br>
><br>
> +++ b/kinit/kinit_win.cpp<br>
><br>
> @@ -107,7 +107,7 @@ QString toString(PSID sid)<br>
><br>
> if (!ConvertSidToStringSid(sid, &s))<br>
><br>
> return QString();<br>
><br>
> - QString result = QString::fromUtf16(s);<br>
><br>
> + QString result = QString::fromUtf16(reinterpret_cast<ushort*>(s));<br>
><br>
> LocalFree(s);<br>
><br>
> return result;<br>
><br>
> }<br>
><br>
><br>
><br>
> --<br>
><br>
> Wolfgang<br>
><br>
<br>
</div></div></blockquote></div><br>