Change in ...libosmocore[master]: osmo-release.sh: update TODO-RELEASE for non-lib projects too

[laforge]

laforge has submitted this change and it was merged. ( https://gerrit.osmocom.org/c/libosmocore/+/15577 )

Change subject: osmo-release.sh: update TODO-RELEASE for non-lib projects too
......................................................................

osmo-release.sh: update TODO-RELEASE for non-lib projects too

Projects not containing libraries may also want to contain a
TODO-RELEASE in order to write down when a new API available only on
libosmoXYZ current master (hence configure.ac and debian cannot be
updated during the patch using the API until the APIs are available in a
new release). This way, during release process of the project, the
maintainer can see that a release of libosmoXYZ is needed beforehand and
then update configure.ac of project accordingly with the new version of
libosmoXYZ.

Furthermore, we want to update the file only if mode DRY_RUN is not
selected.

Change-Id: I409b7eb8c23d21473f25dd2000f5d4447b24adb9
---
M osmo-release.sh
1 file changed, 7 insertions(+), 5 deletions(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/osmo-release.sh b/osmo-release.sh
index 1effe7e..4450436 100755
--- a/osmo-release.sh
+++ b/osmo-release.sh
@@ -156,16 +156,18 @@
 	if [ "z$DRY_RUN" != "z0" ]; then
 		exit 0
 	fi
-	if [ -f "TODO-RELEASE" ]; then
-		grep '#' TODO-RELEASE > TODO-RELEASE.clean
-		mv TODO-RELEASE.clean TODO-RELEASE
-		git add TODO-RELEASE
-	fi
 fi
 
 if [ "z$DRY_RUN" != "z0" ]; then
 	exit 0
 fi
+
+if [ -f "TODO-RELEASE" ]; then
+	grep '#' TODO-RELEASE > TODO-RELEASE.clean
+	mv TODO-RELEASE.clean TODO-RELEASE
+	git add TODO-RELEASE
+fi
+
 gbp dch --debian-tag='%(version)s' --auto --meta --git-author --multimaint-merge --ignore-branch --new-version="$NEW_VER"
 dch -r -m --distribution "unstable" ""
 git add ${GIT_TOPDIR}/debian/changelog

-- 
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/15577
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I409b7eb8c23d21473f25dd2000f5d4447b24adb9
Gerrit-Change-Number: 15577
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at gnumonks.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190920/e0463146/attachment.htm>