binutils: remove dependency to binutils-doc

[ailiop-git]

The binutils main package doesn't depend on the info files contained in
the documentation package. Remove it from being a dependency similar to
any other package whose documentation is split off into a subpackage.

General

• This is a new package and it conforms to the quality requirements

Have the results of the proposed changes been tested?

• I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
• I generally don't use the affected packages but briefly tested this PR

[ericonr]

binutils is 26MB, binutils-doc is less than 1MB. IMO it makes more sense to remove the subpackage completely and fold it into binutils (add replaces="binutils-doc>=0" to binutils).

[ericonr]

Ah, it isn't explained properly. It's split off because other packages depend on it: xbps-query -RX binutils-doc.

You could add a comment about it to the template instead, maybe.

[ailiop-git]

So I agree with folding it into the main package, if we are to follow the manual guideline given that the size of the docs is the deciding factor (we do have other packages with tiny docs split off into a subpkg, e.g. 66, which probably need fixing).

I'm assuming that the original reason was so that the cross binutils can share the doc, but they certainly don't depend on it (so we probably better remove the depends from them). Both cross-arm-none-eabi-binutils and cross-or1k-none-elf-binutils rm the info files at the post_install phase so there should be no conflict.

What do you think?

[ericonr]

The idea of the split is that it becomes necessary because we have many binutils packages which would otherwise conflict. But docs as a split package solve this. In that way, the fact that it's a subpackage could be considered an implementation detail (in order for all the packages to provide the complete deal, including docs) rather than something users should be worrying about, so I would go as far as to say there is nothing to change.

[ericonr]

People consciously added binutils-doc to depends for the other packages, so I expect there to have been actual demand.

[github-actions]

Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it.