mirror of
https://github.com/ArchiveBox/ArchiveBox.git
synced 2026-04-26 01:26:00 +03:00
[GH-ISSUE #372] DJango branch: circular import problem: crontab and logging #3278
Labels
No labels
expected: maybe someday
expected: next release
expected: release after next
expected: unlikely unless contributed
good first ticket
help wanted
pull-request
scope: all users
scope: windows users
size: easy
size: hard
size: medium
size: medium
status: backlog
status: blocked
status: done
status: idea-phase
status: needs followup
status: wip
status: wontfix
touches: API/CLI/Spec
touches: configuration
touches: data/schema/architecture
touches: dependencies/packaging
touches: docs
touches: js
touches: views/replayers/html/css
why: correctness
why: functionality
why: performance
why: security
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ArchiveBox#3278
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @terxw on GitHub (Jul 19, 2020).
Original GitHub issue: https://github.com/ArchiveBox/ArchiveBox/issues/372
Describe the bug
Steps to reproduce
Screenshots or log output
Software versions
848977e@cdvv7788 commented on GitHub (Jul 20, 2020):
@terxw what page are you trying to index? I ran
archivebox add https://example.comand it worked.I will review it anyway, but if you can provide a little more information to reproduce the issue, that would be helpful.
@cdvv7788 commented on GitHub (Jul 20, 2020):
I was able to reproduce it!
@cdvv7788 commented on GitHub (Jul 20, 2020):
@terxw I had the issue with circular imports, but not from crontab. Can you please pull the latest
djangobranch and try again.@terxw commented on GitHub (Jul 22, 2020):
The error appeared at any command e.g. archivebox add, update, and with any url.
I tried updating to the new version
0965031after fix by cdvv7788 and I got this error:i used pew for venv:
It looks to my like urllib3 is trying to run getLogger() from ArchiveBox/archivebox/logging.py instead of python lgger module
@cdvv7788 commented on GitHub (Jul 22, 2020):
@terxw How are you running the commands?
Can you try installing with
pip install -e .in the folder, and then running the commands witharchivebox add http://example.com?I am having the same issue you are having when running directly with
bin/archivebox(checking that atm).@cdvv7788 commented on GitHub (Jul 22, 2020):
@terxw We renamed the module to prevent the issue. Once #379 is merged, you should be able to use it as you normally do.
@terxw commented on GitHub (Jul 25, 2020):
Tried latest commit, installed in venv with pip3 install -e . and no import problem!