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

Fix two small errors in abx-{readwise,spec-config} #1635

Merged
merged 2 commits into from
Jan 17, 2025

Conversation

ckiee
Copy link
Contributor

@ckiee ckiee commented Jan 17, 2025

Summary

Fixes various minor errors I got while testing NixOS/nixpkgs#354836 (on python 3.12 if that matters). See full commit messages for them.

Note: I am no longer affiliated with the NixOS project.

Changes these areas

  • Bugfixes
  • Feature behavior
  • Command line interface
  • Configuration options
  • Internal architecture
  • Snapshot data layout on disk

ckiee added 2 commits January 17, 2025 21:02
Fixes: attributeerror: 'list' object has no attribute 'SOURCES_DIR'
I was getting:
ImportError: cannot import name 'CONSTANTS' from partially initialized module 'archivebox' (most likely due to a circular import)
(/nix/store/6fy0wgy7r3ld3k590kxgxrc0r1cca347-archivebox-0.8.6rc3/lib/python3.12/site-packages/archivebox/__init__.py)
@pirate
Copy link
Member

pirate commented Jan 17, 2025

thanks!

@pirate pirate merged commit 6edcac6 into ArchiveBox:dev Jan 17, 2025
1 check passed
@ckiee
Copy link
Contributor Author

ckiee commented Jan 18, 2025

This one got me as far as archivebox without any arguments, but archivebox init is proving tougher. Is dev supposed to be usable at all?

@pirate
Copy link
Member

pirate commented Jan 18, 2025

no, dev is not supposed to be usable, I'm trying out a bunch of architecture changes on dev. stick to tagged branches if you want things working.

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.

2 participants