Add Notice + License for binary

[ajantha-bhat]

Add License and notice file for the CLI runtime jar (needed for the upcoming release)

[ajantha-bhat]

@jbonofre: Please check this once.

[jbonofre]

@ajantha-bhat thanks. I will.

[jbonofre]

I'm resuming my review/work on this one.

[snazy]

IIRC we don't add LICENSE + NOTICE to the produced (non-bundle) jars as META-INF/LICENSE & META-INF/NOTICE. I think, that's not a problem unless we publish the individual artifacts to Maven Central.

From a brief look the LICENSE + NOTICE files look good, but I didn't cross check the whole files.

[snazy]

Looks like the rat configuration needs some love (CI failure).

[ajantha-bhat]

rat error seems to be for

Unapproved Licenses:
/Users/ajantha/Documents/workspace/polaris-tools/iceberg-catalog-migrator/cli/logs/catalog_migration.log
/Users/ajantha/Documents/workspace/polaris-tools/iceberg-catalog-migrator/api/logs/catalog_migration.log

This has not changed from this build. Probably it is flaky.
I fixed it by updating the rat config.

[snazy]

rat error seems to be for

Unapproved Licenses:
/Users/ajantha/Documents/workspace/polaris-tools/iceberg-catalog-migrator/cli/logs/catalog_migration.log
/Users/ajantha/Documents/workspace/polaris-tools/iceberg-catalog-migrator/api/logs/catalog_migration.log

This has not changed from this build. Probably it is flaky. I fixed it by updating the rat config.

Mind moving the logs/ directory into the build/ folder? In a separate PR though.

[ajantha-bhat]

Mind moving the logs/ directory into the build/ folder? In a separate PR though.

Thanks for this suggestion. But the final CLI also uses the same logback and I don't want to have build/logs folder for end user. For them logs is easy to find it.

I also tried console-log to use logs directory and root-log to use build/logs. So that CLI output will go to logs and normal output will go to build/logs. That didn't work. Everything is writing to build/logs.

So, can we keep it as it is?

[snazy]

So, can we keep it as it is?

Let's have an issue for this, so we don't forget.