* Move core library sources into src/lib/core
* Move command-line programs into src/cli

* Fix up shell() behavior

* Removes dead code after execl()
* Return the return value of execl() immediately
* Output redirection fix: if f_stderr and redirect_stderr were both set, stderr was not redirected
* Move the file handles into the child process
* Bash now executes with --norc to avoid clobbering environment variables. This mimics /bin/sh behavior.

* Fix test script environment

* shell() already provides a temporary script. Not jumping into another sub-shell should preserve help preserve the top-level environment.

* Try a different approach using declare -f

* Revert "Try a different approach using declare -f"

This reverts commit ea8ec855c3c6870d29c55afe3787afb2c05026a1.

* Revert "Fix test script environment"

This reverts commit 0a4efe972a78378eba5c5fbc6819c39b3cc6c9cb.

* Change script permissions: 0700

* Force conda reactivation in test script

* Switch to the usual environment reactivation method

* Initial commit of unit tests [WIP]

* Address shortcomings and bugs flushed out by unit tests

* Enable unit testing in CI workflow

* Enable verbose ctests

* Handle lack of __FILE_NAME__ define

* Only podman support `run --arch` argument

* Skip docker build testing if CI system cannot pull an image

* Remove errant call to puts()

* Identify local repo user

* Fix missing xmllint

* NULL terminate arrays

* Fix filename assignment in is_url mode

* Break loop when expected lines are exhausted

* strcmp_array expects NULL terminated array. Iterating by size in this case passes NULL to strcmp leading to an invalid read

* Remove debug printf statements

* Disable a few warnings for tests

* Workaround for ctest junit xml truncation

* Update checkout@v4

* Prevent false-positive result

* Return zero on error

* Fix strlist_remove function

* Value argument can be constant

* Fix test to match changes to startswith and endswith

* Add test_ini.c

* Fix redaction code to accept NULL pointers in array

* And let the caller specify the length of the array of strings to redact.

* Redactions now occur directly on authentication strings rather than their command line arguments

* Fix BUILD_TESTING_DEBUG

* Adds missing -D argument

* When the result string grew larger than the initial_size, the function would return NULL even if realloc succeeded
* Replaces fgets with fread to ensure all data returned by the program is consumed.
* The line buffer is zeroed for each iteration

* add usage comment function