VTK/Git/Develop Release: Difference between revisions

From KitwarePublic
< VTK‎ | Git
Jump to navigationJump to search
m (Minor tweaks)
Line 13: Line 13:
== Reviewing and Submitting Changes ==
== Reviewing and Submitting Changes ==


The normal [http://review.source.kitware.com/#/admin/groups/11,members 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:
The normal [http://review.source.kitware.com/#/admin/groups/11,members 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 or the continuous submitters but nightly dashboard tests from mac, windows and linux machines are present.  
 
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
  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.
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.

Revision as of 22:22, 3 December 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 or the continuous submitters but nightly dashboard tests from mac, windows and linux machines are present.

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.