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

Load all packages earlier and exit with helpful messages if any are missing #144

Open
empus opened this issue Mar 24, 2024 · 0 comments
Open
Assignees
Labels
improvement General improvement to code base
Milestone

Comments

@empus
Copy link
Owner

empus commented Mar 24, 2024

Overview

Missing TCL packages can result in a failed Armour load or unexpected behaviour. In some cases, a partial load can result in a half-initialised Armour, with some users opting to start over.

Loading all packages early and catching errors can prevent these issues. Currently, this is only done for some packages.

Detail

  1. Load all packages in arm-01_depends.tcl:
  • tcl 8.6
  • http
  • dns
  • sqlite3
  • md5
  • http
  • tls
  • json
  • sha1*
  • sha256*
  1. Output errors to ./armour/<botname>.log

  2. Kill the bot (via die) with helpful message

Considerations

Include suggested package based on the OS the bot is installed on.

@empus empus added the improvement General improvement to code base label Mar 24, 2024
@empus empus added this to the v4.1 milestone Mar 24, 2024
@empus empus self-assigned this Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement General improvement to code base
Projects
None yet
Development

No branches or pull requests

1 participant