https://bugzilla.redhat.com/show_bug.cgi?id=1754964
--- Comment #6 from Neal Gompa <ngompa13(a)gmail.com> ---
(In reply to Miro Hrončok from comment #5)
(In reply to Neal Gompa from comment #4)
> Notes through initial pass of review:
>
> > %global _pkgdocdir %{_docdir}/%{name}-%{version}
>
> Why are we overriding pkgdocdir this way?
Copypasta from bzr, will drop it.
Okay, sounds good!
> > Suggests: %{name} = %{version}-%{release}
>
> This should be using "Enhances" instead of "Suggests", as this
is a reverse
> relationship.
IIRC we don't do reverse dependencies. If I wanted breezy to suggest the
docs, I'd do that directly.
This was adapted from bzr, where the docs required bzr (and I found that too
strong). In fact, it can go away totally.
That's fine with me as well. But we do in fact do reverse dependencies all the
time. Supplements and Enhances have been allowed and used for a couple of years
now. In fact, we used them first before forward weak dependencies, because YUM
ignores them rather than breaking on them.
Enhances is the correct relationship for this, and for things that use Suggests
and Enhances for helping users discover complementary packages, it's quite
helpful as long as the relationships are correct.
> > ln -s brz.1 %{buildroot}%{_mandir}/man1/bzr.1
>
> This doesn't do *exactly* the thing you expect. Do something like this:
>
> > # Link bzr(1) to brz(1)
> > echo ".so man1/brz.1" > %{buildroot}%{_mandir}/man1/bzr.1
>
> This ensures man-db does the correct thing for references.
Could you please describe the difference? It sound interesting, I've always
just linked the manual page (I remember it was broken before due to the way
the Builrtoot Policy handled compression, but that works fine now).
If you do a "man 1 bzr" with this, man's cached references will pull brz(1).
It
also allows manpage indexers to Do The Right Thing(TM) and re-point references.
For example:
https://www.mankier.com/1/pkg-config will actually redirect to
https://www.mankier.com/1/pkgconf instead of just being a copy.
There's a few other benefits, mostly related to the web of man page
dependencies and how they're handled by man page parsers.
I don't know why this isn't documented or made into a macro, but it probably
should be. It's a very useful trick!
> > %doc %{_pkgdocdir}/COPYING.txt
>
> Please relocate the license properly and install it via "%license"
Sure thing. Again, a (wrong) copypasta from bzr.
Okay, sounds good!
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component