clobber pyodide packages with pip packages in freeze

[bollwyvl]

references

• fixes micropip.freeze fails when a pypi package would clobber a bundled pyodide package  #171

changes

• only add packages from pyodide-lock.json if they were not overloaded by some pip mechanism
• parametrize test_freeze_lockfile_compat over a package name and expected depeds, add pytest

[hoodmane]

Can you give example values for pyodide_packages and pip_packages where the old code was wrong?

[bollwyvl]

Over in #171 there's a console reproducer with `pyodide-http`... Really any wheel URL (even if it's the same version as what's in the pyodide-lock.json) will trigger the condition.

…

On Mon, Jan 6, 2025, 04:55 Hood Chatham ***@***.***> wrote: Can you give example values for pyodide_packages and pip_packages where the old code was wrong? — Reply to this email directly, view it on GitHub <#172 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAALCRFUKDWVYALABIC2FFD2JJOIVAVCNFSM6AAAAABUSEGLFGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNZSHA3DIMRZGY> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[bollwyvl]

Ran this here: https://github.com/bollwyvl/micropip/actions/runs/12726352585

[bollwyvl]

Here's a run with just the test vs main(-ish):

https://github.com/bollwyvl/micropip/actions/runs/12726465651/job/35474893463#step:7:315

[ryanking13]

Thanks. It looks good to me in general.

@hoodmane Anything you want to comment?

[hoodmane]

Thanks @bollwyvl!

[bollwyvl]

Thanks, folks! 😁

…

On Mon, Jan 13, 2025, 04:01 Hood Chatham ***@***.***> wrote: Merged #172 <#172> into main. — Reply to this email directly, view it on GitHub <#172 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAALCRC2JAVDFMUDE5PBF4T2KOFG5AVCNFSM6AAAAABUSEGLFGVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVHEYDEMZZGI4DMMI> . You are receiving this because you were mentioned.Message ID: ***@***.***>