Red Hat Bugzilla – Bug 1026873
fedora-review: doesn't seem to recognize that -docs package is for documentation
Last modified: 2013-12-01 11:34:36 EST
Description of problem:
In review request https://bugzilla.redhat.com/show_bug.cgi?id=1026337,
two packages are produced:
The second contains most of documentation, and the first one is actaully only 471KB. But fedora-review still says:
- Large documentation must go in a -doc subpackage. Large could be size (~1MB)
or number of files.
Note: Documentation size is 1822720 bytes in 10 files.
Version-Release number of selected component (if applicable):
I'm putting the binary packages and the source package for reference here:
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.