You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Julian Hyde (Jira)" <ji...@apache.org> on 2022/10/26 01:07:00 UTC

[jira] [Commented] (CALCITE-5197) Bump gradle to 7.4.2 and add checksum autoupdate

    [ https://issues.apache.org/jira/browse/CALCITE-5197?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17624102#comment-17624102 ] 

Julian Hyde commented on CALCITE-5197:
--------------------------------------

Reviewing and testing now. If possible I'll bump up to Gradle 7.5.1, the latest.

> Bump gradle to 7.4.2 and add checksum autoupdate
> ------------------------------------------------
>
>                 Key: CALCITE-5197
>                 URL: https://issues.apache.org/jira/browse/CALCITE-5197
>             Project: Calcite
>          Issue Type: Improvement
>          Components: build
>    Affects Versions: 1.30.0
>            Reporter: Sergey Nuyanzin
>            Assignee: Sergey Nuyanzin
>            Priority: Trivial
>              Labels: pull-request-available
>             Fix For: 1.33.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> The problem with gradle update via
> {code}
> ./gradlew wrapper --gradle-version <new_version> && ./gradlew autostyleApply
> {code}
> it removes checksum from {{gradle/wrapper/gradle-wrapper.properties}}
> there could be added a task which could update checksum as well, e.g.
> {code}
> tasks.wrapper {
>     distributionType = Wrapper.DistributionType.BIN
>     doLast {
>         val sha256Uri = URI("$distributionUrl.sha256")
>         val sha256Sum = String(sha256Uri.toURL().readBytes())
>         propertiesFile.appendText("distributionSha256Sum=${sha256Sum}\n")
>     }
> }
> {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)