FedoraReview
FedoraReview copied to clipboard
Add ada plugin
Note that our development happens on devel branch, not master branch (as described in CONTRIBUTE file). Current devel branch no longer uses "is_applicable" function in the same way.
But the most important thing is actually, that you duplicated checks already present (CheckLDConfig, CheckSoFiles and few more). I see no reason why that should be done.
So my suggestions for reworking before I can merge:
- base this on current devel
- remove all non-ada specific checks. If you want to improve checks already in f-r please do it in a different pull request
- name all ada checks in sensible and unique way so they won't clash with other checks. My suggestion is "CheckAdaXXX"
- If your check is supposed to "override" original check use "deprecates" (see for example RCheckLatestVersionIsPackaged)
Ada packages use *.so and ldconfig as well as C++. Why I cannot check LDConfig for ada packages?
It's actually pretty simple. Checks you duplicated should run for packages that need them automatically already. If you have a package that has *so files and we don't list the ldconfig check, then by all means file a bugreport. With your duplication that test would be listed twice in the resulting template.
I can prepare a branch with fixes I proposed which you could sanity-check, since I am not familiar with Ada guidelines myself.