Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-03-01 Thread via GitHub
github-actions[bot] closed pull request #11977: Build: Add plugin to generate license and notice files URL: https://github.com/apache/iceberg/pull/11977 -- 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

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-03-01 Thread via GitHub
github-actions[bot] commented on PR #11977: URL: https://github.com/apache/iceberg/pull/11977#issuecomment-2692483167 This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-02-22 Thread via GitHub
github-actions[bot] commented on PR #11977: URL: https://github.com/apache/iceberg/pull/11977#issuecomment-2676463563 This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think thatโ€™s incorrect or this pul

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-23 Thread via GitHub
jbonofre commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1926640052 ## aws-bundle/NOTICE: ## @@ -5,92 +5,22 @@ Copyright 2017-2024 The Apache Software Foundation This product includes software developed at The Apache Software Found

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-23 Thread via GitHub
jbonofre commented on PR #11977: URL: https://github.com/apache/iceberg/pull/11977#issuecomment-2609282280 @bryanck thanks a lot for working on that ! I will take a look as I'm doing a complete pass on artifacts `LICENSE`/`NOTICE`. Generally speaking, by experience on Apache projects,

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-21 Thread via GitHub
bryanck commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1924086012 ## license.gradle: ## @@ -0,0 +1,196 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-20 Thread via GitHub
ajantha-bhat commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1922300103 ## license.gradle: ## @@ -0,0 +1,196 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the N

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-17 Thread via GitHub
RussellSpitzer commented on PR #11977: URL: https://github.com/apache/iceberg/pull/11977#issuecomment-2599272113 I would like to be clear that it is extremely important that we assume all other engineers are contributing in good faith and the interaction above is failing to meet that standa

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
snazy commented on PR #11977: URL: https://github.com/apache/iceberg/pull/11977#issuecomment-2597552634 > This was not copied from another project, we have used this in Iceberg before Polaris existed, but simply didn't contribute the code. [#8261 (comment)](https://github.com/apache/iceberg

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
bryanck commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919376400 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact contain

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
bryanck commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919358126 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact contain

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
bryanck commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919358126 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact contain

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
bryanck commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919356956 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact contain

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
RussellSpitzer commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919352795 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
RussellSpitzer commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919352050 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
bryanck commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919349968 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact contain

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
bryanck commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919347703 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact contain

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
RussellSpitzer commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919322126 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
bryanck commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919307214 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact contain

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
RussellSpitzer commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919306097 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
bryanck commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919239714 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact contain

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
bryanck commented on PR #11977: URL: https://github.com/apache/iceberg/pull/11977#issuecomment-2596931879 > I'm requesting changes to this PR, because as it is based on code of Apache Polaris (I think @jbonofre mentioned it on the last Iceberg community sync) it requires a mention - and I s

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
RussellSpitzer commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919194287 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
RussellSpitzer commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1919189350 ## aws-bundle/LICENSE: ## @@ -203,320 +203,468 @@ -This binary artifact

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
bryanck commented on PR #11977: URL: https://github.com/apache/iceberg/pull/11977#issuecomment-2596867076 > This is looking great @bryanck, thanks for working on this ๐Ÿ‘ > > How about documenting the process of checking if the NOTICE files are up to date as part of the [how-to-release

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-16 Thread via GitHub
Fokko commented on PR #11977: URL: https://github.com/apache/iceberg/pull/11977#issuecomment-2596850981 This is looking great @bryanck, thanks for working on this ๐Ÿ‘ How about documenting the process of checking if the NOTICE files are up to date as part of the [how-to-release](https

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-15 Thread via GitHub
bryanck commented on PR #11977: URL: https://github.com/apache/iceberg/pull/11977#issuecomment-2594424405 > I liked the idea that Ryan suggested this morning, where our plugin would always generate and check the generated files against a set of committed existing files and report an error i

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-15 Thread via GitHub
bryanck commented on PR #11977: URL: https://github.com/apache/iceberg/pull/11977#issuecomment-2594205352 > I liked the idea that Ryan suggested this morning, where our plugin would always generate and check the generated files against a set of committed existing files and report an error i

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-15 Thread via GitHub
RussellSpitzer commented on PR #11977: URL: https://github.com/apache/iceberg/pull/11977#issuecomment-2594075737 I liked the idea that Ryan suggested this morning, where our plugin would always generate and check the generated files against a set of committed existing files and report an er

Re: [PR] Build: Add plugin to generate license and notice files [iceberg]

2025-01-15 Thread via GitHub
bryanck commented on code in PR #11977: URL: https://github.com/apache/iceberg/pull/11977#discussion_r1917278321 ## buildSrc/build.gradle: ## @@ -0,0 +1,26 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the