ARPA-SIMC / wreport

C++ library and applications to work with weather reports. The library provides featureful BUFR and CREX encoding and decoding.
Other
9 stars 9 forks source link

Fix libwreport.pc dependencies #19

Closed edigiacomo closed 6 years ago

edigiacomo commented 6 years ago

See https://people.freedesktop.org/~dbn/pkg-config-guide.html#faq

wreport is compiled against the following libraries:

edigiacomo commented 6 years ago

Moreover, Requires: lua >= 5.1.1 could be removed from RPM dependencies (because Lua supports pkg-config in RH-based distros).

edigiacomo commented 6 years ago

In-depth explanation following.

Tthe .pc file should observe with the following rule:

See https://people.freedesktop.org/%7Edbn/pkg-config-guide.html#concepts

The .spec file should observe the following rules:

See https://fedoraproject.org/wiki/Packaging:PkgConfigBuildRequires and https://fedoraproject.org/wiki/Archives:PackagingDrafts/PkgconfigAutoRequires?rd=PackagingDrafts/PkgconfigAutoRequires

edigiacomo commented 6 years ago
  • A required library which doesn't support pkg-config will be listed in BuildRequires using the name its -devel package and in Requires of the -devel package.

Moreover: the library will be in the Requires of the -devel package only if it's in Libs of the .pc file (i.e. not in Libs.private).