VTK/Git/Develop Release: Difference between revisions
DaveDemarle (talk | contribs) mNo edit summary |
m (Minor tweaks) |
||
Line 1: | Line 1: | ||
VTK 6 | VTK 6 had several major changes, including the build system and library layout changes. Given the scope of the changes, it is important to support the 5.10 release for longer to enable those who are not yet ready to port to VTK 6+ to continue using the last release before these changes were merged. To that end, we have added some software development infrastructure that enables developers to maintain the 5.10 branch. We have added a release-5.10 branch to our VTK repositories. The branch was started from the v5.10.1 tag and is only intended for merging '''critical''' bug fixes, and '''conservative''' patches to allow VTK 5.10 to continue working on new platforms and with new compilers. | ||
== Submitting Patches == | == Submitting Patches == |
Revision as of 16:49, 28 October 2013
VTK 6 had several major changes, including the build system and library layout changes. Given the scope of the changes, it is important to support the 5.10 release for longer to enable those who are not yet ready to port to VTK 6+ to continue using the last release before these changes were merged. To that end, we have added some software development infrastructure that enables developers to maintain the 5.10 branch. We have added a release-5.10 branch to our VTK repositories. The branch was started from the v5.10.1 tag and is only intended for merging critical bug fixes, and conservative patches to allow VTK 5.10 to continue working on new platforms and with new compilers.
Submitting Patches
Any topic intended for the release-5.10 branch should be branches from the tip of this branch. This can be achieved with the following in a VTK source tree:
git fetch gerrit git checkout release-5.10 git checkout -b my-backported-bugfix
You can then work in that topic as you would normally, cherry picking commits or making changes and committing them. Please bear in mind that we will only consider conservative fixes/changes intended to fix bugs or compiler errors. These should already be present in master, and no new features will be considered for inclusion.
Reviewing and Submitting Changes
The normal set of core reviewers can be asked to review, but a much smaller set of people has submit privileges. At present this is limited to Dave DeMarle and Marcus D. Hanwell but the list may be expanded in the future. The topics are not tested by CDash@Home but nightly dashboards will be added. In order to push a topic for review the following (while in the topic) will work:
git push gerrit HEAD:refs/for/release-5.10/my-backported-bugfix
Where my-backported-bugfix is the name of the topic as it will appear in Gerrit. Please let us know on the VTK developers list if you experience any problems in submitting changes.