Skip to content

Build: exclude site/ from the source release archive - #18193

Open
nssalian wants to merge 1 commit into
apache:mainfrom
nssalian:fix-18191
Open

nssalian wants to merge 1 commit into
apache:mainfrom
nssalian:fix-18191

Conversation

@nssalian

@nssalian nssalian commented Sep 21, 2026

Copy link
Copy Markdown
Collaborator

Closes #18191

The source release tarball bundled third-party doc-site assets under site/docs/assets/ (Bootstrap, termynal, Lottie player) whose license texts aren't included and that aren't listed in the root LICENSE.

These are rendered-docs assets, published separately and not needed to build Iceberg from source. This adds site/ to .gitattributes export-ignore, mirroring the existing docs/ rule.

I verified on this branch with:

git archive HEAD --prefix t/ -o /tmp/site-check.tgz && tar tzf /tmp/site-check.tgz | grep -c '/site/'
0

@github-actions github-actions Bot added the INFRA label Sep 21, 2026
@nssalian
nssalian marked this pull request as ready for review September 21, 2026 01:55
@nssalian
nssalian marked this pull request as draft September 21, 2026 03:07
@nssalian
nssalian force-pushed the fix-18191 branch 2 times, most recently from a1198bc to e428a81 Compare September 21, 2026 03:25
@nssalian
nssalian marked this pull request as ready for review September 21, 2026 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Source release bundles third-party doc-site assets not accounted for in LICENSE

1 participant