jdlorimer / incremental-reading

Anki add-on providing incremental reading features
https://ankiweb.net/shared/info/935264945
ISC License
216 stars 38 forks source link

import cgi (anki beta) #87

Closed khonkhortisan closed 5 years ago

khonkhortisan commented 5 years ago

Describe the bug Mod doesn't load, errors on importing cgi from feedparser

To Reproduce Use anki-2.1.6-beta1-windows.exe or anki-2.1.6-beta2-windows.exe ?

Expected behavior

# ---------- required modules (should come with any Python distribution) ----------
import cgi

It should work, as the comment says.

Screenshots image

---------------------------
Anki
---------------------------
Una extensión que has instalado ha fallado al cargarse. Si los problemas persisten, por favor ve a Herramientas> Menú de extensiones o deshabilita esta extensión.

Mientras cargando 'Incremental Reading v4913':
Traceback (most recent call last):
  File "aqt\addons.py", line 60, in loadAddons
  File "C:\Users\Khonkhortisan\AppData\Roaming\Anki2\addons21\935264945\__init__.py", line 17, in <module>
    from .main import ReadingManager
  File "C:\Users\Khonkhortisan\AppData\Roaming\Anki2\addons21\935264945\main.py", line 28, in <module>
    from .importer import Importer
  File "C:\Users\Khonkhortisan\AppData\Roaming\Anki2\addons21\935264945\importer.py", line 43, in <module>
    from .lib.feedparser import parse
  File "C:\Users\Khonkhortisan\AppData\Roaming\Anki2\addons21\935264945\lib\feedparser.py", line 133, in <module>
    import cgi
ModuleNotFoundError: No module named 'cgi'

---------------------------
Aceptar   
---------------------------

Specs (please complete the following information):

Additional context Without that comment above the import, I would assume I would have to install a python module on windows. It's probably that 2.1.6 changed how it sets up the python path or loads addons.

jdlorimer commented 5 years ago

Thanks, I'll look into this tomorrow and bundle cgi if I need to.

jdlorimer commented 5 years ago

This should be fixed now thanks to Morgan.