Skip to content

BUG: Fix meson-python failing on installing install_subdirs#147

Merged
rgommers merged 2 commits intomesonbuild:mainfrom
lithomas1:fix-install-subdirs
Sep 28, 2022
Merged

BUG: Fix meson-python failing on installing install_subdirs#147
rgommers merged 2 commits intomesonbuild:mainfrom
lithomas1:fix-install-subdirs

Conversation

@lithomas1
Copy link
Member

You can only write a single file, or a directory that you can't specify the destination for with wheel.

I copy pasted the code and adjusted from wheel to do it.

cc @rgommers @FFY00

@rgommers rgommers added the bug Something isn't working label Sep 25, 2022
rgommers added a commit to rgommers/meson-python that referenced this pull request Sep 25, 2022
This works with the fix in mesonbuildgh-147.

[ci skip]
Copy link
Contributor

@rgommers rgommers left a comment

Choose a reason for hiding this comment

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

Thanks @lithomas1! I have tested that this works with the test case for install_subdir in gh-145.

@rgommers
Copy link
Contributor

Mypy and the pre-commit check seem to be a bit unhappy still.

@rgommers rgommers added this to the v0.9.0 milestone Sep 27, 2022
rgommers added a commit to rgommers/meson-python that referenced this pull request Sep 27, 2022
This works with the fix in mesonbuildgh-147.

[ci skip]
lithomas1 and others added 2 commits September 28, 2022 00:05
You can only write a single file, or a directory that you can't specify the destination for with wheel.

I copy pasted the code and adjusted from wheel to do it.

cc @rgommers @FFY00
@rgommers rgommers marked this pull request as ready for review September 27, 2022 22:36
@rgommers
Copy link
Contributor

This is happy now, and also works in combination with the scipy-like test in gh-148. So in it goes. Thanks @lithomas1!

@rgommers rgommers merged commit 3a36c8f into mesonbuild:main Sep 28, 2022
rgommers added a commit to rgommers/meson-python that referenced this pull request Sep 28, 2022
This works with the fix in mesonbuildgh-147.

[ci skip]
@lithomas1 lithomas1 deleted the fix-install-subdirs branch November 19, 2022 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants