[Rpm-maint] [rpm-software-management/rpm] scripts/pkgconfigdeps.sh fixes (#411)

[kloczek]

1) remove --print-requires-private from $pkconfig parameters
It generates list of Requires when static linking is used:

$ pkg-config --help | grep -A1 -- --print-requires-private
  --print-requires-private          print required dependency frameworks for static
                                    linking to stdout
By using this option generated list Requires is flooded static linking requirements
when all distributions now are trying to avoid even build and provide any possible
static libraries.

2) Remove add pkgconfig to list of generated Requires is at least one .p…  …
…c file was found.

Provide .pc file by any package does not mean that it will be used.
Usually .pc files are used on building other packages.
By this redundant Requires in Fedora almost 750 packages packages requires pkgconfig.

$ dnf -qC repoquery --whatrequires pkgconfig --qf="%{name}"| wc -l
748

pkgconfig should be only in Requires if for example in autoconf .m4 macros is used
pkg-config (to provide the list of libs or cflags). In any other case pkgconfig or pkg-config
should be used only in BuildRequires.
You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/411

-- Commit Summary --

  * remove --print-requires-private frpm $pkconfig parameters
  * Remove add pkgconfig to list of generated Requires is at least one .pc file was found.

-- File Changes --

    M scripts/pkgconfigdeps.sh (4)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/411.patch
https://github.com/rpm-software-management/rpm/pull/411.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/411
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20180311/cfd4b974/attachment.html>