calebmadrigal / trackerjacker

Like nmap for mapping wifi networks you're not connected to, plus device tracking
MIT License
2.64k stars 188 forks source link

import yaml instead of pyaml? #3

Closed obilodeau closed 6 years ago

obilodeau commented 6 years ago

On Python 3.6 with Arch Linux:

$ sudo trackerjacker -h
Traceback (most recent call last):
  File "/usr/sbin/trackerjacker", line 11, in <module>
    load_entry_point('trackerjacker==1.1.0', 'console_scripts', 'trackerjacker')()
  File "/usr/lib/python3.6/site-packages/pkg_resources/__init__.py", line 572, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2755, in load_entry_point
    return ep.load()
  File "/usr/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2408, in load
    return self.resolve()
  File "/usr/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2414, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/usr/lib/python3.6/site-packages/trackerjacker/__main__.py", line 15, in <module>
    from . import dot11_mapper
  File "/usr/lib/python3.6/site-packages/trackerjacker/dot11_mapper.py", line 10, in <module>
    import pyaml
ModuleNotFoundError: No module named 'pyaml'

Based on PyYAML's doc the import should be yaml I think: http://pyyaml.org/wiki/PyYAML

$ pip3 list | grep PyYAML
[...]
PyYAML (3.12)
[...]

$ ipython3 
Python 3.6.4 (default, Jan  5 2018, 02:35:40) 
Type 'copyright', 'credits' or 'license' for more information
IPython 6.2.1 -- An enhanced Interactive Python. Type '?' for help.

In [1]: import pyaml
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-1-1eb0b043bec3> in <module>()
----> 1 import pyaml

ModuleNotFoundError: No module named 'pyaml'

In [2]: import yaml

In [3]: 
calebmadrigal commented 6 years ago

@obilodeau Thanks for finding this - I forgot to update the requirements.txt when I had moved from PyYAML (https://pypi.python.org/pypi/PyYAML) to pyaml (https://pypi.python.org/pypi/pyaml). Fix is in version 1.1.1 (https://github.com/calebmadrigal/trackerjacker/commit/eff9fd4e55e54dfe86f42f6ca471bb7293814668).