Warn about transitive dependencies

This commit change the way how namcap handles dependencies to meet with our recent guideline to not rely on transitive dependencies, as they might break, if one of the dependencies is updated.

Previously, namcap warned about dependencies if they are included but already satisfied, even if these are transitive dependencies. From now, namcap will warn if a dependency is not explicitly specified, and won't show a warning if these are already satisfied by other dependencies.

This commit also simplifies and fixes various issues in the dependency handling logic, e.g.:

• Strip the version numbers when searching what provides a package, otherwise it doesn't match any.
• Remove some slow recursive dependency search. With the new logic we only need it in case of the explicitly specified dependencies.
• Always show at least an info message when a dependency satisfied. Useful to see which components need a library.

added 1 commit

• 78184bed - Warn about transitive dependencies

Compare with previous version

added 1 commit

• a3f7ed56 - Warn about transitive dependencies

Compare with previous version

added 1 commit

• 0e1fe1c5 - Warn about transitive dependencies

Compare with previous version

I had another WIP branch for reporting transitive dependencies. Compared to this branch, it

• Drops recursive dependency resolution altogether
• Treats implicit transitive dependencies the same as missing dependencies, and thus they become errors instead of warnings.

I'm not sure if reporting such issues as errors will be confusing or not. This branch may be better.

approved this merge request

This looks good to me but I haven't been able to compare it in depth to @yan12125's related branch. It would be useful to get some more feedback from @bgyorgy on that comment and from anybody else with an opinion here.

added 13 commits

• 0e1fe1c5...c3eb177a - 12 commits from branch pacman:master
• 2771841b - Warn about transitive dependencies

Compare with previous version

enabled an automatic merge when the pipeline for 2771841b succeeds

aborted the automatic merge because target branch was updated

resolved all threads

added 4 commits

• 2771841b...4bbe56e4 - 3 commits from branch pacman:master
• 0eae4544 - Warn about transitive dependencies

Compare with previous version

unapproved this merge request

This MR is going to require some updated test expectations before we can merge.

added 1 commit

• 37c05449 - Update tests to follow changes in depends.py

Compare with previous version

I updated the tests to meet with the new behaviour.

approved this merge request

merged