Move atomic_ops libs into Requires.private
Acturally, bdwgc it self may requires atomic_ops, but it does not require downstream packages (like guile) requires atomic_ops. So I would suggest that move @ATOMIC_OPS_LIBS@ into Requires.private, so that won't confuse downstream packages, especially when performing check.
https://people.freedesktop.org/~dbn/pkg-config-guide.html#faq
My library
zuseslibxinternally, but does not exposelibxdata types in its public API. What do I put in myz.pcfile? Again, add the module toRequires.privateif it supportspkg-config. In this case, the compiler flags will be emitted unnecessarily, but it ensures that the linker flags will be present when linking statically. Iflibxdoes not supportpkg-config, add the necessary linker flags toLibs.private.
So I think, move atomic_ops is a more appropriate solution.
Please rebase to fresh master and explain the difference from the previous PR I just merged.
I suddenly realized a problem. atomic_ops shipped atomic_ops.pc starting from 7.2. If bdwgc is built upon atomic_ops 7.1, then we should not put it in bdwgc.pc file, because there is no atomic_ops.pc at that time.
Maybe we should leave it into Libs.private for now, till the time of bdwgc relies on atomic_ops >= 7.2.
Maybe we should leave it into Libs.private for now, till the time of bdwgc relies on atomic_ops >= 7.2.
Although, 7.1 or 7.2 is not hard-coded in bdwgc source. I agree that we could leave this change for the future - I will create the relevant issue and close this PR.