Skip to content

Commit

Permalink
[#5535] fix(build,license): Fix the gradle publish issue and several …
Browse files Browse the repository at this point in the history
…license issues (#5544)

### What changes were proposed in this pull request?

1. Fix the release script to also publish `.module` file used for
gradle.
2. Update the original place for gradle script.
3. Update the paths for codes referring from Ranger.

### Why are the changes needed?

Fix the publish and license issue

Fix: #5535 

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Local verification.
  • Loading branch information
jerryshao authored Nov 13, 2024
1 parent b453b07 commit 4de7084
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 2 deletions.
5 changes: 4 additions & 1 deletion LICENSE
Original file line number Diff line number Diff line change
Expand Up @@ -206,6 +206,10 @@
This product bundles various third-party components also under the
Apache Software License 2.0.

Gradle (https://github.com/gradle/gradle)
./gradlew
./gradle/wrapper/gradle-wrapper.properties

Apache Spark
./api/src/main/java/org/apache/gravitino/SchemaChange.java
./api/src/main/java/org/apache/gravitino/SupportsSchemas.java
Expand Down Expand Up @@ -249,7 +253,6 @@
./clients/client-java/src/main/java/org/apache/gravitino/client/OAuth2ClientUtil.java
./iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/service/IcebergExceptionMapper.java
./iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/utils/IcebergHiveCachedClientPool.java
./gradlew

Apache Paimon
./catalogs/catalog-lakehouse-paimon/src/main/java/org/apache/gravitino/catalog/lakehouse/paimon/utils/TypeUtils.java
Expand Down
2 changes: 1 addition & 1 deletion dev/release/release-build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -339,7 +339,7 @@ if [[ "$1" == "publish-release" ]]; then
pushd $tmp_repo/org/apache/gravitino

# Remove any extra files generated during install
find . -type f |grep -v \.jar |grep -v \.pom |grep -v cyclonedx | xargs rm
find . -type f |grep -v \.jar |grep -v \.pom |grep -v cyclonedx | grep -v \.module | xargs rm

echo "Creating hash and signature files"
# this must have .asc, .md5 and .sha1 - it really doesn't like anything else there
Expand Down
1 change: 1 addition & 0 deletions gradle/wrapper/gradle-wrapper.properties
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
# specific language governing permissions and limitations
# under the License.
#
# Refer from https://github.com/gradle/gradle/blob/master/gradle/wrapper/gradle-wrapper.properties
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
# checksum was taken from https://gradle.org/release-checksums
Expand Down
2 changes: 2 additions & 0 deletions gradlew
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,8 @@
#
##############################################################################

# Refer from https://github.com/gradle/gradle/blob/master/gradlew

# Attempt to set APP_HOME

# Resolve links: $0 may be a link
Expand Down

0 comments on commit 4de7084

Please sign in to comment.