tree d718d19be6ffde581dfbf0eebdc3e797922805ab
parent cc46fb26d629b9e440371861f031cb2a85fd9c55
author Ignas Anikevicius <240938+aignas@users.noreply.github.com> 1745144279 +0900
committer GitHub <noreply@github.com> 1745144279 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJoBMnnCRC1aQ7uu5UhlAAAzW4QAJI32V8VpGCS0aUgpTNRA3UF
 HuWKsIsRqL6V+qMpCqNvYUNjB9yEBMJq420feIdf4Ak3d3HbBYuzb3gdbmT2NRuf
 ZxkroMVRsbXciaC/s9x4z9j4cMyigv3w4r+pU9PuPUqqMyLPXzMX41QPWBg3+AK4
 Z61D/Wp47++htbj4rQT7RDTzsH6u8nRze0KRJcwiemZdhdlS6doULquf2SKTVZEa
 Yn0mgy030M4QGehR2dnAnCLQbpfyMZ/gS9BerBlHELL07bggw8uWnfwH934eYjgh
 tP9oOzIPHgX9oRLNY4yDX+GR+La9Z1ziCJmxxLBPx2EhgOjUsBQeWStDiB9Jgun2
 PTE26nS1Q9Rhby+NyyNgNAu/uP5pXAWzITAtFTWiWZOJbWnfpd0XvfWfPxcDePpr
 CEbqTJtxgGtQ3qkpQQycBVSrypcH4vbEic9kZDSiBuvdBDH8Ye1i99NQ4FgQ7SZ2
 R5Hg2STKqt1fEtMv12iNlh6aqoteuUm3AKoZmh6I3iBmAsgmqBAkiemVxGHphb38
 vW7BmsP8PM4Xrh4vSbxZ7z8aggErpLqecFyew0+V3WB+sKtGep3AtJSJrbr/AyWk
 ABIpE5dS95UFDGgyk1COD1utB+QOo0IPuEGkwytl+E3ddk58CUhD5sw/xMZEnTSj
 QTWaPSxPMEe9QY6w6ik2
 =mZVa
 -----END PGP SIGNATURE-----
 

fix: load target_platforms through the hub (#2781)

This PR moves the parsing of `Requires-Dist` to the loading phase
within the `whl_library_targets_from_requires` macro. The original
`whl_library_targets` macro has been left unchanged so that I don't have
to reinvent the unit tests - it is well covered under tests.

Before this PR we had to wire the `target_platforms` via the
`experimental_target_platforms` attr in the `whl_library`, which means
that whenever this would change (e.g. the minor Python version changes),
the wheel would be re-extracted even though the final result may be the
same.

This refactor uncovered that the dependency graph creation was incorrect
if we had multiple target Python versions due to various heuristics that
this had. In hindsight I had them to make the generated `BUILD.bazel`
files more readable when the unit test coverage was not great. Now this
is unnecessary and since everything is happening in Starlark I thought
that having a simpler algorithm that does the right thing always is the
best way.

This also cleans up the code by removing left over TODO notes or code
that no longer make sense.

Work towards #260, #2319