[GitHub] [iceberg] kbendick commented on pull request #5186: [WIP] [Build/Core] - Add a Gradle task to create a build properties file for core module and accessor class

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

kbendick commented on PR #5186:
URL: https://github.com/apache/iceberg/pull/5186#issuecomment-1175249659

   > In general the approach makes sense to me. However, when looking at the spark-runtime jar I noticed that there's a `git.properties` file with a bunch of information being added (looks like this actually comes from the `apache-arrow` lib). Here's the content of that file:
   > 
   > ```
   > #Generated by Git-Commit-Id-Plugin
   > #Sat Jan 29 05:11:45 UTC 2022
   > git.build.user.email=
   > git.build.host=Mac-1643432868411.local
   > git.dirty=true
   > git.remote.origin.url=https\://github.com/apache/arrow
   > git.closest.tag.name=apache-arrow-7.0.0
   > git.commit.id.describe-short=apache-arrow-7.0.0-0-dirty
   > git.commit.user.email=szucs.krisztian@gmail.com
   > git.commit.time=29.01.2022 @ 00\:08\:24 UTC
   > git.commit.message.full=[Release] Update versions for 7.0.0
   > git.build.version=7.0.0
   > git.commit.message.short=[Release] Update versions for 7.0.0
   > git.commit.id.abbrev=e90472e
   > git.branch=e90472e35b40f58b17d408438bb8de1641bfe6ef
   > git.build.user.name=
   > git.closest.tag.commit.count=0
   > git.commit.id.describe=apache-arrow-7.0.0-0-ge90472e-dirty
   > git.commit.id=e90472e35b40f58b17d408438bb8de1641bfe6ef
   > git.tags=apache-arrow-7.0.0
   > git.build.time=29.01.2022 @ 05\:11\:45 UTC
   > git.commit.user.name=Krisztián Szűcs
   > ```
   > 
   > I looked at https://github.com/n0mer/gradle-git-properties and was wondering whether we could actually use that plugin to generate a `git.properties` file for Iceberg. It seems the plugin lets you control which info goes into that file. Also note that we could include the Iceberg version via a `customProperty`.
   > 
   > For completeness, here's how Arrow generates that file: https://github.com/apache/arrow/blob/master/java/pom.xml#L249-L290
   
   This is great. I'll try with this, as I can't find a clean way to deal with the additional file that's generated from the git tree.


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org