Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

meson.build: Install dist-info METADATA #1303

Merged

Conversation

zmedico
Copy link
Member

@zmedico zmedico commented Mar 8, 2024

Install dist-info METADATA for pip to resolve dependencies:

# pip freeze | grep portage
portage==3.0.63

Suggested-by: Eli Schwartz eschwartz93@gmail.com
Bug: https://bugs.gentoo.org/920330

@zmedico zmedico force-pushed the bug_920330_install_dist-info_METADATA branch 3 times, most recently from e441ad9 to 041b289 Compare March 8, 2024 04:36
Install dist-info METADATA for pip to resolve dependencies:

    # pip freeze | grep portage
    portage==3.0.63

Suggested-by: Eli Schwartz <eschwartz93@gmail.com>
Bug: https://bugs.gentoo.org/920330
Signed-off-by: Zac Medico <zmedico@gentoo.org>
@zmedico zmedico force-pushed the bug_920330_install_dist-info_METADATA branch from 041b289 to 32e4a58 Compare March 8, 2024 05:05
Copy link
Member

@chewi chewi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for keeping it simple in the end. 🙂

@mgorny
Copy link
Member

mgorny commented Mar 8, 2024

Please give me 24 hours or so to attempt to rewrite sys-apps/portage ebuild to use plain PEP517 build instead of the awful pile of hacks it's doing right now.

@chewi
Copy link
Member

chewi commented Mar 8, 2024

Please give me 24 hours or so to attempt to rewrite sys-apps/portage ebuild to use plain PEP517 build instead of the awful pile of hacks it's doing right now.

I honestly don't believe my work was an awful pile of hacks. Portage does not abuse Meson's Python support, and Meson installs the files to the same locations that PEP517 would. You're effectively saying that Meson's Python support is worthless unless used together with meson-python. Feel free to try something different, but I think it's good that Portage can currently be installed system-wide using the regular Meson commands.

@mgorny
Copy link
Member

mgorny commented Mar 8, 2024

Nevermind. I didn't realize that mesonpy is so strict about wheel specification, and doesn't allow installing to other paths like setuptools does. This is hopeless.

@gentoo-bot gentoo-bot merged commit 32e4a58 into gentoo:master Mar 8, 2024
12 checks passed
@zmedico zmedico deleted the bug_920330_install_dist-info_METADATA branch March 8, 2024 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants