https://bugzilla.redhat.com/show_bug.cgi?id=1069257 --- Comment #14 from Till Hofmann <hofmann@xxxxxxxxxxxxxxxxxxx> --- (In reply to Michael Schwendt from comment #13) > > > MPFR and GMP are not necessary but provide extra features, so I'm not > > sure if I should include them as requirement? > > As a rule of thumb for -devel package "Requires" one could say: (1) If *any* > of the headers within fparser-devel include headers from a different -devel > package, that one ought to be added as a "Requires". The rationale would > simply be that compiling with fparser-devel should never fail because of > missing headers. fparser_gmpint.hh includes files from gmp-devel, but clearly fparser_gmpint.hh is used only if the GMP Parser is to be used (similarly for MPFR). There are other parsers available which don't require GMP or MPFR. But GMP and MPFR support is determined at compile time, so if we don't add the dependency, there is no way we can provide GMP or MPFR support (see http://warp.povusers.org/FunctionParser/fparser.html#parsertypes ). Now I'm not sure what to do, either we force everyone to install mpfr-devel and gmp-devel or we omit the support for both completely. What do you think? I've asked upstream about Licensing and I got the following answer: > The reason is because of this clause at the beginning of the LGPL licence: > > "This version of the GNU Lesser General Public License incorporates > the terms and conditions of version 3 of the GNU General Public > License, supplemented by the additional permissions listed below." > > In other words, LGPL is GPL plus the additions mentioned in the LGPL > document. Thus you can't get the whole of the LGPL text if you don't > include the GPL text as well. This absolutely makes sense but it should apply to all LGPL-licensed packages, how is this usually handled? -- 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 _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review