I would prefer publishing on a public repository, but am not informed about pull requests on Github/Gitorious. I would certainly preserve authorship information, and recently integrated a patch from David Feng with preserved authorship. We also worked with David Gobbi on the wrapping branch.<div>

<br></div><div>Sorry if I missed this one, the change looks reasonable and I can take a look at it. It would be good to see improved unicode support too. It is something that the ARB should probably be discussing, but it would be great to involve the community as far as is possible.</div>

<div><br></div><div>Marcus</div><div><br><div class="gmail_quote">On Tue, Jul 6, 2010 at 12:06 PM, Moreland, Kenneth <span dir="ltr"><<a href="mailto:kmorel@sandia.gov">kmorel@sandia.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">





<div>
<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<div class="im"><br>
<br>
<br>
On 7/6/10 7:38 AM, "Dave Partyka" <<a href="http://dave.partyka@kitware.com" target="_blank">dave.partyka@kitware.com</a>> wrote:<br>
<br>
</div></span></font><div class="im"><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" target="_blank">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="http://jchris.fillionr@kitware.com" target="_blank">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" target="_blank">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="http://noreply@github.com" target="_blank">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="http://jchris.fillionr@kitware.com" target="_blank">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" target="_blank">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></div><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="http://kmorel@sandia.gov" target="_blank">kmorel@sandia.gov</a><br>
**  ***  **  phone: (505) 844-8919<br>
    ***      web:   <a href="http://www.cs.unm.edu/~kmorel" target="_blank">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>
</div>


<br>_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
<br></blockquote></div><br></div>