| Summary: | fedora-review: doesn't seem to recognize that -docs package is for documentation | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Zbigniew Jędrzejewski-Szmek <zbyszek> |
| Component: | fedora-review | Assignee: | Stanislav Ochotnicky <sochotni> |
| Status: | CLOSED WONTFIX | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | unspecified | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 20 | CC: | leamas.alec, pingou, sochotni |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2013-11-25 12:05:25 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
|
Description
Zbigniew Jędrzejewski-Szmek
2013-11-05 15:01:09 UTC
I'm gonna take a wild guess w/o having looked at the sources: could it be a difference between -doc and -docs? :) Indeed. It's scripts/check-large-docs.sh, it might need an "if" branch also for -docs. While the fix is relatively trivial, I don't think we should go this way. I'm in favour of being consistent across distribution for this kind of thing. The subpackage should be renamed IMO. From Packaging Guidelines: ... Or if there's a lot of documentation, consider putting it into a subpackage. In this case, it is recommended to use *-doc as the subpackage name. I'm closing as won't fix for the above reason. I don't want to hide inconsistencies with guidelines. The problem is that repoquery '*-docs'|wc lists 123 packages which would be flagged by fedora-review with this issue. I think we should strive to reduce the noise from the review tools as much as possible, and it seems that the ship has already sailed on the the split between -doc and -docs. Both are used in the wild, and unless the guidelines are changed to required just -doc, people will continue to keep adding packages which use -docs. Right now the review warnings are often all false positives, which makes people ignore legitimate warnings in the noise. That warnings tend to be false positives is really as expected. In situations which are not clear, the basic strategy is to notify the reviewer. IMHO, this is the only reasonable strategy. However, this bug is actually quite clear: fedora-review always strives to implement the current guidelines. If you want them to be changed, the proper action is to file a FPC ticket. We will not on purpose mis-interpret the GL. (In reply to Alec Leamas from comment #5) > That warnings tend to be false positives is really as expected. In > situations which are not clear, the basic strategy is to notify the > reviewer. IMHO, this is the only reasonable strategy. True, but this doesn't apply here. There's nothing unclear here: -docs and -doc packages are found in Fedora in large quantities. It's only fedora-review that has an issue with -docs. > However, this bug is actually quite clear: fedora-review always strives to > implement the current guidelines. If you want them to be changed, the proper > action is to file a FPC ticket. We will not on purpose mis-interpret the GL. I don't want to change the guidelines. The guidelines suggest -doc, but don't disallow -docs, and -docs are common practice. Thus, I think fedora-review should stop warning about -docs. |