<HTML>
<HEAD>
<TITLE>Re: [vtk-developers] Best way to contribute to VTK ? - Fwd: [GitHub] nolden sent you a pull request from nolden/VTK</TITLE>
</HEAD>
<BODY>
<FONT FACE="Calibri, Verdana, Helvetica, Arial"><SPAN STYLE='font-size:11pt'>Is submitting patches to the mailing list now the officially sanctioned method of providing contributions (as opposed to publishing on a public repository such as Marco has done)? Will this change in the near future when we plan to move to a branchy workflow (which as I understand it now has plans for a more open “staging” repository)? <BR>
<BR>
Perhaps this would be a good topic for the next ARB.<BR>
<BR>
-Ken<BR>
<BR>
<BR>
On 7/6/10 7:38 AM, "Dave Partyka" <<a href="dave.partyka@kitware.com">dave.partyka@kitware.com</a>> wrote:<BR>
<BR>
</SPAN></FONT><BLOCKQUOTE><FONT FACE="Calibri, Verdana, Helvetica, Arial"><SPAN STYLE='font-size:11pt'>The most visible way is to send a patch to the vtk list. If you use the git patch commands you can create patches that will preserve the author (Marco in this case) as git has separate Author and Committer fields. See this wiki at the bottom on how to create a patch.<BR>
<BR>
<a href="http://vtk.org/Wiki/VTK/Git#Submitting_a_patch">http://vtk.org/Wiki/VTK/Git#Submitting_a_patch</a><BR>
<BR>
On Tue, Jul 6, 2010 at 9:24 AM, Jean-Christophe Fillion-Robin <<a href="jchris.fillionr@kitware.com">jchris.fillionr@kitware.com</a>> wrote:<BR>
</SPAN></FONT><BLOCKQUOTE><FONT FACE="Calibri, Verdana, Helvetica, Arial"><SPAN STYLE='font-size:11pt'>Hi Folks, <BR>
<BR>
Since VTK souce code is available using Git, the repository can be forked very easily and people can contribute quite easily. <BR>
<BR>
Being also available on github, Marco from DKFZ (German research cancer institute) forked it and added a patch. As he mentioned, "Pull request" doesn't seem to "work". What could be the best way for him to share his patch. <BR>
<BR>
Beside of the following page <a href="http://www.vtk.org/VTK/project/getinvolved.html">http://www.vtk.org/VTK/project/getinvolved.html</a> , <BR>
Is there any documentation available regarding the "contribution" workflow ? <BR>
Is the mailing list the preferred way to share contribution ? <BR>
Will the ownership of the original "patcher" be maintained ? <BR>
<BR>
I added Marco in CC, feel free to answer that email and let him know the best way to share his work. <BR>
<BR>
Thks<BR>
Jc<BR>
<BR>
<BR>
---------- Forwarded message ----------<BR>
From: <B>GitHub</B> <<a href="noreply@github.com">noreply@github.com</a>><BR>
Date: Wed, Jun 30, 2010 at 9:17 AM<BR>
Subject: [GitHub] nolden sent you a pull request from nolden/VTK<BR>
To: <a href="jchris.fillionr@kitware.com">jchris.fillionr@kitware.com</a><BR>
<BR>
<BR>
<BR>
<BR>
nolden wants you to pull from nolden/VTK at 9ba9588<BR>
<BR>
Body: Hi,<BR>
<BR>
this is a bugfix for the rendering of utf8 characters. The handling of signed / unsigned was incorrect. As a result only 7bit characters were rendered, though in general the code was ok for 8bit characters.<BR>
<BR>
cc Steven and JC, since I don't know if the "KItware" user reads pull requests.<BR>
<BR>
View repository: <a href="http://github.com/nolden/VTK/tree/9ba958806256e38ae4ae4fc424c37f8a7eea2ae9">http://github.com/nolden/VTK/tree/9ba958806256e38ae4ae4fc424c37f8a7eea2ae9</a><BR>
<BR>
<BR>
</SPAN></FONT></BLOCKQUOTE></BLOCKQUOTE><FONT FACE="Calibri, Verdana, Helvetica, Arial"><SPAN STYLE='font-size:11pt'><BR>
</SPAN></FONT><FONT SIZE="2"><FONT FACE="Consolas, Courier New, Courier"><SPAN STYLE='font-size:10pt'><BR>
**** Kenneth Moreland<BR>
*** Sandia National Laboratories<BR>
*********** <BR>
*** *** *** email: <a href="kmorel@sandia.gov">kmorel@sandia.gov</a><BR>
** *** ** phone: (505) 844-8919<BR>
*** web: <a href="http://www.cs.unm.edu/~kmorel">http://www.cs.unm.edu/~kmorel</a><BR>
</SPAN></FONT></FONT><FONT FACE="Calibri, Verdana, Helvetica, Arial"><SPAN STYLE='font-size:11pt'><BR>
</SPAN></FONT>
</BODY>
</HTML>