[Asterisk-code-review] Add remote url option and allow more stop conditions (repotools[master])

Kevin Harwell asteriskteam at digium.com
Mon Mar 28 14:27:06 CDT 2016


Kevin Harwell has uploaded a new patch set (#2).

Change subject: Add remote url option and allow more stop conditions
......................................................................

Add remote url option and allow more stop conditions

digium_git.py:

When branches and tags were created it would immediately push those items to
the remote, and then push subsequent changes/commits again later. This patch
makes it so any new items and/or changes are only pushed when the consuming
module is ready to do so (Namely once all work has been done - mkrelease.py
has been updated to account for these changes).

mkrelease.py & release_summary.py:

For the most part the mkrelease script would push changes upstream along the
way. The remote url was also hardcoded. Both of these things made testing the
script/release process hard.

This patch adds the ability to point the remote repo url to a "test" repo
if needed. It also adds more exit points during the process (when using the
interactive option). One of those points being "push changes", which means
everything is done locally up to that point (nothing has been pushed to the
remote yet). Also converted some of the [debug] logging (and added some) to
use python logger.

Change-Id: I51db48fa146d201b93a388a426ec3ba766e981ae
---
M digium_git.py
M mkrelease.py
M release_summary.py
3 files changed, 196 insertions(+), 172 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/repotools refs/changes/61/2461/2
-- 
To view, visit https://gerrit.asterisk.org/2461
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I51db48fa146d201b93a388a426ec3ba766e981ae
Gerrit-PatchSet: 2
Gerrit-Project: repotools
Gerrit-Branch: master
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Matthew Fredrickson <creslin at digium.com>



More information about the asterisk-code-review mailing list