[GitHub] [calcite-avatica] zabetak commented on issue #104: WIP: CALCITE-3158: Avatica: migrate build system to Gradle

[GitBox <gi...@apache.org>]

zabetak commented on issue #104: WIP: CALCITE-3158: Avatica: migrate build system to Gradle
URL: https://github.com/apache/calcite-avatica/pull/104#issuecomment-550197516
 
 
   I performed a dryRun of the release with gradle and then back to back a dryRun of the release using mvn. I know it is a bit out of scope since mvn will be completely replaced but I noticed that in the second case there are the following problems when performing the RAT check:
   ```
   cat target/rat.txt | grep "\!?????"
    !????? gradlew.bat
    !????? .ratignore
    !????? gradlew
    !????? release/build/licenses/getLicenses/license.xml
    !????? release/build/distributions/apache-calcite-avatica-1.16.0-src.tar.gz.asc
    !????? release/build/distributions/apache-calcite-avatica-1.16.0-src.zip.asc
    !????? release/build/distributions/apache-calcite-avatica-1.16.0-src.tar.gz.sha512
    !????? release/build/distributions/apache-calcite-avatica-1.16.0-src.zip.sha512
    !????? src/main/config/licenses/hsqldb/LICENSE-hsqldb
    !????? src/main/config/licenses/hsqldb/LICENSE-hypersonic
    !????? .gradle/vcs-1/gc.properties
    !????? .gradle/buildOutputCleanup/cache.properties
    !????? .gradle/5.5.1/gc.properties
   ```
   I suppose that these come up because .ratignore is not used by maven right?
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services