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

pyproject.toml: six is a required dependency #14

Closed
wants to merge 2 commits into from

Conversation

twoerner
Copy link
Collaborator

When not installed, a clean/fresh install results in:

$ bmaptool --version
Traceback (most recent call last):
  File "/2opt/devel/bmaptool/venv.python3.11/bin/bmaptool", line 5, in <module>
    from bmaptool.CLI import main
  File "/2opt/devel/bmaptool/venv.python3.11/lib64/python3.11/site-packages/bmaptool/CLI.py", line 42, in <module>
    from . import BmapCreate, BmapCopy, BmapHelpers, TransRead
  File "/2opt/devel/bmaptool/venv.python3.11/lib64/python3.11/site-packages/bmaptool/TransRead.py", line 36, in <module>
    from six.moves.urllib import parse as urlparse
ModuleNotFoundError: No module named 'six'

When not installed, a clean/fresh install results in:

	$ bmaptool --version
	Traceback (most recent call last):
	  File "/2opt/devel/bmaptool/venv.python3.11/bin/bmaptool", line 5, in <module>
	    from bmaptool.CLI import main
	  File "/2opt/devel/bmaptool/venv.python3.11/lib64/python3.11/site-packages/bmaptool/CLI.py", line 42, in <module>
	    from . import BmapCreate, BmapCopy, BmapHelpers, TransRead
	  File "/2opt/devel/bmaptool/venv.python3.11/lib64/python3.11/site-packages/bmaptool/TransRead.py", line 36, in <module>
	    from six.moves.urllib import parse as urlparse
	ModuleNotFoundError: No module named 'six'

Signed-off-by: Trevor Woerner <[email protected]>
@twoerner
Copy link
Collaborator Author

I think this is required after #12

The unit tests shouldn't depend on the dev dependencies, install normal
dependencies to ensure the dependecy list is accurate for regular
users/installs.

Signed-off-by: Trevor Woerner <[email protected]>
@twoerner twoerner closed this Mar 27, 2024
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.

1 participant