This is an automated email from the ASF dual-hosted git repository.

github-merge-queue[bot] pushed a commit to branch 
gh-readonly-queue/main/pr-3563-6f23a2a7757a44d03f7cbfc9600a3bee6bd480a9
in repository https://gitbox.apache.org/repos/asf/iceberg-rust.git

commit 68a17aba6ee0c1500bcb65e881da4c71786bc42a
Author: Kevin Liu <[email protected]>
AuthorDate: Wed Oct 28 09:38:02 2026 +0000

    fix(python): don't install LICENSE and NOTICE into site-packages (#3563)
    
    * fix(python): stop installing LICENSE and NOTICE into site-packages
    
    Including LICENSE and NOTICE in the wheel puts them at the wheel root,
    so installing pyiceberg-core writes site-packages/LICENSE and
    site-packages/NOTICE, which clash with other packages that do the same.
    maturin already copies both into dist-info/licenses, so only include
    them in the sdist.
    
    Co-authored-by: Copilot App <[email protected]>
    
    * fix(python): note why LICENSE and NOTICE are sdist-only
    
    Co-authored-by: Copilot App <[email protected]>
    
    ---------
    
    Co-authored-by: Copilot App <[email protected]>
---
 bindings/python/pyproject.toml | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/bindings/python/pyproject.toml b/bindings/python/pyproject.toml
index 017ffaf04..714bac424 100644
--- a/bindings/python/pyproject.toml
+++ b/bindings/python/pyproject.toml
@@ -40,15 +40,10 @@ readme = "project-description.md"
 requires-python = ">=3.10,<4"
 
 [tool.maturin]
+# sdist only: maturin adds these to wheels automatically (dist-info/licenses/).
 include = [
-  { path = "LICENSE", format = [
-    "sdist",
-    "wheel",
-  ] },
-  { path = "NOTICE", format = [
-    "sdist",
-    "wheel",
-  ] },
+  { path = "LICENSE", format = "sdist" },
+  { path = "NOTICE", format = "sdist" },
 ]
 module-name = "pyiceberg_core.pyiceberg_core_rust"
 profile = "release"

Reply via email to