Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Release] Address spurious Apache RAT failures in source release script #19890

Closed
asfimport opened this issue Oct 20, 2018 · 8 comments
Closed
Assignees
Milestone

Comments

@asfimport
Copy link
Collaborator

Something about the untarring / retarring process is offending Apache RAT and it's reporting a number of unapproved licenses incorrectly

https://gist.github.com/wesm/490dc50c494cc914ae28b4fd897a73eb

Reporter: Wes McKinney / @wesm
Assignee: Kouhei Sutou / @kou

PRs and other links:

Note: This issue was originally created as ARROW-3578. Please see the migration documentation for further details.

@asfimport
Copy link
Collaborator Author

Wes McKinney / @wesm:
Seems this could be caused by #2417

@asfimport
Copy link
Collaborator Author

Krisztian Szucs / @kszucs:
Steps to reproduce?

@asfimport
Copy link
Collaborator Author

Wes McKinney / @wesm:
Run the source release script. It fails here: https://github.com/apache/arrow/blob/master/dev/release/02-source.sh#L114

@asfimport
Copy link
Collaborator Author

Uwe Korn / @xhochy:
Are these symlinks in the tar? At least cmake_modules were a symlink once that created these problems.

@asfimport
Copy link
Collaborator Author

Wes McKinney / @wesm:
I don't think so

@asfimport
Copy link
Collaborator Author

Krisztian Szucs / @kszucs:
Caused by the prepare script: https://github.com/apache/arrow/blob/master/dev/release/00-prepare.sh#L147-L171

On the release branch the rat exclude file is updated, and the 02-source.sh uses the updated exclude file, not the one which belongs to the release.

 

Here is the boundary: https://github.com/apache/arrow/blob/master/dev/release/00-prepare.sh#L141

So We need to run RAT with the exclude file within the archive.

@asfimport
Copy link
Collaborator Author

Uwe Korn / @xhochy:
These are things that are symlinked to in the archive. It probably depends on what comes first.

@asfimport
Copy link
Collaborator Author

Kouhei Sutou / @kou:
Issue resolved by pull request 4028
#4028

@asfimport asfimport added this to the 0.13.0 milestone Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants