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

Exclude doc/ from wheel #423

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

penguinpee
Copy link
Contributor

The automatic discovery adds doc/ to the wheel triggered by the
presence of conf.py in that directory.

The automatic discovery adds doc/ to the wheel triggered by the
presence of `conf.py` in that directory.
@penguinpee
Copy link
Contributor Author

Alternatively, disabling namespaces also works: find = {namespaces = false} instead of find {}.

@leouieda
Copy link
Member

Thanks for that @penguinpee. We're discussing changing our packaging practices to remove all of these from our built packages here: fatiando/community#154

@carlosal1015
Copy link

Here https://archlinux.org/packages/extra/any/python-pooch/files same error

error: failed to commit transaction (conflicting files)
python-pooch: /usr/lib/python3.12/site-packages/doc/conf.py exists in filesystem (owned by python-codepy)

@penguinpee
Copy link
Contributor Author

Here https://archlinux.org/packages/extra/any/python-pooch/files same error

error: failed to commit transaction (conflicting files)
python-pooch: /usr/lib/python3.12/site-packages/doc/conf.py exists in filesystem (owned by python-codepy)

In your case the file is already claimed by codepy. I looked at the sdist from PyPI and codepy is making the same mistake by using packages=find_namespace_packages() in setup.py, which adds doc as a top level module with conf.py in it.

That should be reported to codepy upstream. For now you could apply this PR as a patch to pooch allowing you to get on with the installation.

@carlosal1015
Copy link

I will do it, many thanks for the feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants