Add missing init/deinit calls

[cottsay]

For the Python extension, de-initialization of Assuan as missing. For both the Python extension and the agent executable, initialization and de-initialization of the createrepo_c parser and dumper were missing, leading to a possible segfault when the dumper was lazy initialized by multiple threads simultaneously.

The changes to agent.c involve a small refactor to move the bulk of the code out of main and into a new run function. The main function is still responsible for argument parsing and invocation of the global init/deinit routines, and the actual operations are called from run now. This reduced a lot of the deinit code duplication, especially given that we're now adding two more of them.

Closes #41

[codecov-commenter]

Codecov Report

❌ Patch coverage is 28.57143% with 65 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.54%. Comparing base (955efe9) to head (121b3c4).
⚠️ Report is 9 commits behind head on main.

Files with missing lines	Patch %	Lines
src/createrepo-agent/agent.c	13.55%	49 Missing and 2 partials ⚠️
src/python/init.c	35.29%	9 Missing and 2 partials ⚠️
test/integration_main.cpp	62.50%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
+ Coverage   48.97%   55.54%   +6.57%     
==========================================
  Files          20       22       +2     
  Lines        2638     2830     +192     
  Branches      538      601      +63     
==========================================
+ Hits         1292     1572     +280     
+ Misses       1039      936     -103     
- Partials      307      322      +15     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[j-rivero]

One question test/integration_main.cpp there is an assuan_sock_init(); call, do we need to there the cr_xml_dump_init(); and cr_package_parser_init(); ?

Same for deinit and cleanups in that same file.

[cottsay]

One question test/integration_main.cpp there is an assuan_sock_init(); call, do we need to there the cr_xml_dump_init(); and cr_package_parser_init(); ?

Yes! Good call. Done in c0f4dfa.

[j-rivero]

Another round and I think we are good to go

[j-rivero]

Good to go. All the big problems are patched as far as I can tell. I left one minor patch.

[cottsay]

Thanks!

I left one minor patch.

I'm not seeing this, and I don't wanna miss it. Is there a chance that GitHub dropped a comment somewhere?

[j-rivero]

ouch, missed to press the button sorry