Remove python 2 init symbols to get compilation to work on 3.12.4

Pritam Dodeja · Pritam Dodeja · commit 3f56c2c6464a · 2026-02-23T04:54:06.000-05:00
Stricter llvm linker was not allowing compilation to succeed.
diff --git a/ml_metadata/ml_metadata.bzl b/ml_metadata/ml_metadata.bzl
@@ -221,11 +221,9 @@ def ml_metadata_pybind_extension(
         "PyInit_%s" % sname,
     ]
 
-    # For Linux, include Python 2 symbols for compatibility
+    # For Linux, do not include Python 2 symbols for compatibility
     # (version script allows undefined symbols)
     exported_symbols_linux = [
-        "init%s" % sname,
-        "init_%s" % sname,
         "PyInit_%s" % sname,
     ]
 

Original file line number	Diff line number	Diff line change
`@@ -221,11 +221,9 @@ def ml_metadata_pybind_extension(`
`221`	`221`	`"PyInit_%s" % sname,`
`222`	`222`	`]`
`223`	`223`
`224`		`- # For Linux, include Python 2 symbols for compatibility`
	`224`	`+ # For Linux, do not include Python 2 symbols for compatibility`
`225`	`225`	`# (version script allows undefined symbols)`
`226`	`226`	`exported_symbols_linux = [`
`227`		`- "init%s" % sname,`
`228`		`- "init_%s" % sname,`
`229`	`227`	`"PyInit_%s" % sname,`
`230`	`228`	`]`
`231`	`229`