Skip to content

Commit 0128bf6

Browse files
dapr-botjavier-aliagasiri-varma
authored
fix(release): install dapr-sdk-bom locally before bumping dapr-spring-bom (#1744) (#1745)
The release script bumps the dapr.sdk.version property in all reactor POMs (including dapr-spring-bom) before bumping the standalone BOMs. The later 'mvn versions:set -f dapr-spring/dapr-spring-bom/pom.xml' runs in single-POM mode (no reactor) and tries to resolve the import dapr-sdk-bom at the new version, which is not yet installed. Install sdk-bom into the local repo between the two BOM updates so the import resolves. (cherry picked from commit 2c038f4) Signed-off-by: Javier Aliaga <javier@diagrid.io> Co-authored-by: Javier Aliaga <javier@diagrid.io> Co-authored-by: Siri Varma Vegiraju <siri.varma@outlook.com>
1 parent ed8d6a0 commit 0128bf6

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

.github/scripts/update_sdk_version.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@ mvn versions:set-property -Dproperty=dapr.sdk.version -DnewVersion=$DAPR_JAVA_SD
1515
# BOMs are standalone (no parent), so versions:set skips them — update explicitly.
1616
mvn versions:set -DnewVersion=$DAPR_JAVA_SDK_VERSION -f sdk-bom/pom.xml
1717
mvn versions:set-property -Dproperty=dapr.sdk.version -DnewVersion=$DAPR_JAVA_SDK_VERSION -f sdk-bom/pom.xml
18+
# Install dapr-sdk-bom locally so dapr-spring-bom's import can resolve when loaded with -f (no reactor).
19+
mvn install -N -DskipTests -f sdk-bom/pom.xml
1820
mvn versions:set -DnewVersion=$DAPR_JAVA_SDK_VERSION -f dapr-spring/dapr-spring-bom/pom.xml
1921
mvn versions:set-property -Dproperty=dapr.sdk.version -DnewVersion=$DAPR_JAVA_SDK_VERSION -f dapr-spring/dapr-spring-bom/pom.xml
2022
mvn versions:set-property -Dproperty=dapr.sdk.alpha.version -DnewVersion=$DAPR_JAVA_SDK_ALPHA_VERSION -f sdk-tests/pom.xml

0 commit comments

Comments
 (0)