8000 release-version: reintroduce sed massaging · hadim/scijava-scripts@bc1c1fb · GitHub
[go: up one dir, main page]

Skip to content

Commit bc1c1fb

Browse files
committed
release-version: reintroduce sed massaging
This reverts commit b21af25, and tweaks the command to return successfully with both sed flavors.
1 parent b21af25 commit bc1c1fb

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

release-version.sh

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -194,6 +194,14 @@ fi &&
194194

195195
# rewrite the tag to include release.properties
196196
test -n "$tag" &&
197+
# HACK: SciJava projects use SSH (git@github.com:...) for developerConnection.
198+
# The release:perform command wants to use the developerConnection URL when
199+
# checking out the release tag. But reading from this URL requires credentials
200+
# which we would rather Travis not need. So we replace the scm.url in the
201+
# release.properties file to use the read-only (git://github.com/...) URL.
202+
# This is OK, since release:perform does not need write access to the repo.
203+
sed -i.bak -e 's|^scm.url=scm\\:git\\:git@github.com\\:|scm.url=scm\\:git\\:git\\://github.com/|' release.properties &&
204+
rm release.properties.bak &&
197205
$DRY_RUN git checkout "$tag" &&
198206
$DRY_RUN git add release.properties &&
199207
$DRY_RUN git commit --amend --no-edit &&

0 commit comments

Comments
 (0)
0