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

A more complete C example #78

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jonwoodring
Copy link

A more complete C example:

  • adds the threadsafe cleanup code
  • signal handlers for SIGINT and SIGTERM (same exit code)
  • modified README for a bit more detail
  • added a simple level logger
  • adds a Makefile and two ways to make through pkg-config and without
  • changed the environment variables to DELTACHAT_C_ECHOBOT_EMAIL and DELTACHAT_C_ECHOBOT_PASSWORD
  • changed if-else chains to switch-case

@adbenitez
Copy link
Contributor

Hi @jonwoodring thanks a lot for contributing!!! I feel like this is changing too much things in a single pull request but I am no C developer, maybe our C expert @r10s can review your PR

@jonwoodring
Copy link
Author

That's fine.

If it's too much, I can just keep it alive on my page, and maybe put it link in the README or so someone else can find it. Or maybe put is as an alternate example in a second directory?

Either way, no skin off my back. Thanks.

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