Skip to content

Commit 9d06907

Browse files
committed
Add more search/replace patterns in bump_dependency.bash
This also fixes some issues encountered while running the script for #1235 Signed-off-by: Addisu Z. Taddese <addisu@openrobotics.org>
1 parent d7bcf66 commit 9d06907

File tree

1 file changed

+14
-9
lines changed

1 file changed

+14
-9
lines changed

source-repo-scripts/bump_dependency.bash

Lines changed: 14 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
# When you say yes, the changes will be committed and pushed. Click on the link printed
3636
# by GitHub to open the pull request.
3737

38-
# TODO: Update collection list DSL on release-tools
38+
# TODO: Update gz-collection.yaml on release-tools
3939

4040
DEFAULT="\e[39m"
4141
DEFAULT_BG="\e[49m"
@@ -348,6 +348,7 @@ for ((i = 0; i < "${#LIBRARIES[@]}"; i++)); do
348348
startFromCleanBranch ${BUMP_BRANCH} master
349349

350350
# construct path with major version suffix
351+
FORMULA_BASE=$LIB
351352
FORMULA="Formula/${FORMULA_BASE}${VER}.rb"
352353
if [ ! -f "$FORMULA" ]; then
353354
echo -e "${GREEN}${LIB}: Creating ${FORMULA}${DEFAULT}"
@@ -356,8 +357,7 @@ for ((i = 0; i < "${#LIBRARIES[@]}"; i++)); do
356357

357358
# Collection
358359
if ! [[ $VER == ?(-)+([0-9]) ]] ; then
359-
# TODO(CH3): Change this to gz once we migrate the homebrew formulae
360-
cp Formula/ignition-${PREV_COLLECTION}.rb $FORMULA
360+
cp Formula/gz-${PREV_COLLECTION}.rb $FORMULA
361361
else
362362
cp Formula/${FORMULA_BASE}${PREV_VER}.rb $FORMULA
363363
fi
@@ -371,6 +371,9 @@ for ((i = 0; i < "${#LIBRARIES[@]}"; i++)); do
371371
# libN
372372
sed -i -E "s ((${LIB#"gz-"}))${PREV_VER} \1${VER} g" $FORMULA
373373
sed -i -E "s ((${LIB_#"gz_"}))${PREV_VER} \1${VER} g" $FORMULA
374+
# lib-N
375+
sed -i -E "s ((${LIB#"gz-"}-))${PREV_VER} \1${VER} g" $FORMULA
376+
sed -i -E "s ((${LIB_#"gz_"}-))${PREV_VER} \1${VER} g" $FORMULA
374377
# gz-libN -> main
375378
sed -i "s ${LIB}${PREV_VER} main g" $FORMULA
376379
# class GzLibN
@@ -388,6 +391,13 @@ for ((i = 0; i < "${#LIBRARIES[@]}"; i++)); do
388391
PREV_VER_NONNEGATIVE=$([[ "${PREV_VER}" -lt 0 ]] && echo "0" || echo "${PREV_VER}")
389392
sed -i "/ version /d" $FORMULA
390393
sed -i "/^ url.*/a\ version \"${PREV_VER_NONNEGATIVE}.999.999-0-`date +"%Y%m%d"`\"" $FORMULA
394+
# Collection
395+
if [[ "${LIB}" == "gz-${COLLECTION}" ]]; then
396+
PREV_COLLECTION_CAPITALIZED="${PREV_COLLECTION^}"
397+
COLLECTION_CAPITALIZED="${COLLECTION^}"
398+
sed -i -E "s/((Gz))${PREV_COLLECTION_CAPITALIZED}/\1${COLLECTION_CAPITALIZED}/g" $FORMULA
399+
sed -i -E "s/((gz-))${PREV_COLLECTION}/\1${COLLECTION}/g" $FORMULA
400+
fi
391401
# Remove extra blank lines
392402
cat -s $FORMULA | tee $FORMULA
393403

@@ -440,7 +450,7 @@ for ((i = 0; i < "${#LIBRARIES[@]}"; i++)); do
440450
startFromCleanBranch ${BUMP_BRANCH} main
441451

442452
# Check if main branch of that library is the correct version
443-
PROJECT_NAME="${LIB_}${VER}"
453+
PROJECT_NAME="${LIB}${VER}"
444454
PROJECT="project.*(${PROJECT_NAME}"
445455
if ! grep -q ${PROJECT} "CMakeLists.txt"; then
446456
echo -e "${RED}Wrong project name on [CMakeLists.txt], looking for [$PROJECT_NAME].${DEFAULT}"
@@ -474,11 +484,6 @@ for ((i = 0; i < "${#LIBRARIES[@]}"; i++)); do
474484
# with: "find_package(gz-cmake3)"
475485
find . -type f ! -name 'Changelog.md' ! -name 'Migration.md' -print0 | xargs -0 sed -i "s ${DEP_LIB}${DEP_PREV_VER} ${DEP_LIB}${DEP_VER} g"
476486

477-
# Replace collection yaml branch names with main
478-
if [[ "${LIB}" == "gz-${COLLECTION}" ]]; then
479-
find . -type f -name "collection-${COLLECTION}.yaml" -print0 | xargs -0 sed -i "s gz-${DEP_LIB}${DEP_VER} main g"
480-
fi
481-
482487
# Second run with _ instead of -, to support multiple variations of fuel-tools
483488
DEP_LIB=${DEP_LIB//-/_}
484489
find . -type f ! -name 'Changelog.md' ! -name 'Migration.md' -print0 | xargs -0 sed -i "s ${DEP_LIB}${DEP_PREV_VER} ${DEP_LIB}${DEP_VER} g"

0 commit comments

Comments
 (0)