Skip to content
New issue

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

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

Already on GitHub? # to your account

Install submodules. #224

Merged
merged 1 commit into from
May 13, 2020
Merged

Conversation

gordonmessmer
Copy link
Contributor

I see the the builders and callback plugins are included in the wheel, but they aren't installed by "python3 setup.py install". I think they need to be added to the packages list in setup.cfg, but I'm curious how the wheel was built, so that they were included.

@TomasTomecek
Copy link
Collaborator

honestly, it's been years since I used setup.py install, I usually install things with pip

setup.cfg Outdated
@@ -31,7 +31,7 @@ keywords =

[options]
python_requires = >=3.6
packages = ansible_bender
packages = ansible_bender, ansible_bender.builders, ansible_bender.callback_plugins
Copy link
Collaborator

Choose a reason for hiding this comment

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

I took look in the python packages docs and found this:

packages = find:

https://setuptools.readthedocs.io/en/latest/setuptools.html#configuring-setup-using-setup-cfg-files

could we try that instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That also works.

@gordonmessmer
Copy link
Contributor Author

honestly, it's been years since I used setup.py install, I usually install things with pip

I might be missing something... what I mean is, how did you build the wheel that pip installed?

@TomasTomecek
Copy link
Collaborator

probably with python3 setup.py sdist

@TomasTomecek TomasTomecek merged commit e1a501c into ansible-community:master May 13, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants