New comment by tornaria on void-packages repository

https://github.com/void-linux/void-packages/pull/47888#issuecomment-1869029316

Comment:
Pushed a new version. Now `show-build-deps` will include checkdepends when using -Q or -K.

This means `sort-dependencies` will do the same when using -Q or -K.

As is this won't fix CI, since `common/travis/changed_templates.sh` doesn't pass `-Q` to sort-dependencies.

It's also not 100% correct since there are a number of situations where `XBPS_CHECK_PKGS` can be set but check will not run. See here: https://github.com/void-linux/void-packages/pull/46207/files#diff-2553330b694a471fd9a77f9645b03f16c9c92e8e656a5b44634d816cbdaa2a5dR140-R144 for a set of conditions which imply no testing for a particular package. Maybe this can be refactored somewhere and used for this PR and also to fix #46207.

The idea is: read the pkg, determine if checks would be run for this pkg (based on `make_check`, value of `XBPS_CHECK_PKGS`, ci-skip, etc) and only include checkdepends together with build depends if the checks will be run.

This also means it's easy to use `common/scripts/xbps-cycles.py` to catch build-check cycles by doing something like `XBPS_CHECK_PKGS=yes common/scripts/xbps-cycles.py`.

Proposal (again): consider build-check cycles with `XBPS_CHECK_PKGS=yes` to be a bug; only allow build-check cycles when `XBPS_CHECK_PKGS=full`.