FS#62885 - [swi-prolog] pkgver 8.0.2 adds too many mandatory dependencies

Attached to Project: Community Packages
Opened by Sacha Delanoue (Shaac) - Wednesday, 12 June 2019, 17:21 GMT
Last edited by Alexander F. Rødseth (xyproto) - Thursday, 13 June 2019, 09:19 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Alexander F. Rødseth (xyproto)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

Since new version of swi-prolog 8.0.2, there are many new dependencies: jdk-openjdk libedit libyaml qt5-tools

While some of them might be relevant, some are not. Why do I have to use the java JDK to run some prolog? I also find the need of qt5-tools very suspicious.

Java runtime is an optional dependency, so there is something related to java somewhere. Maybe openjdk is needed as a makedep dependency?

I’m not sure which dependencies should be mandatory here. And maybe I am wrong and they are all necessary. But something does not smell right.
This task depends upon

Closed by  Alexander F. Rødseth (xyproto)
Thursday, 13 June 2019, 09:19 GMT
Reason for closing:  None
Comment by Alexander F. Rødseth (xyproto) - Thursday, 13 June 2019, 07:27 GMT
Thanks for reporting. Yes, some of the dependencies could probably be moved from dependencies to make dependencies. I'm looking into it.
Comment by Alexander F. Rødseth (xyproto) - Thursday, 13 June 2019, 07:44 GMT
jdk-openjdk provides libjvm.so. When moving it from a dependency to a make dependency, namcap reports this warning:

swi-prolog W: Referenced library 'libjvm.so' is an uninstalled dependency

If this dependency "does not smell right", you could ask the upstream Prolog developers why it is needed.

jdk-openjdk could probably be listed as an optional dependency, though, with the risk of parts of swi-prolog not working.

qt5-base or qt5-tools is needed.
Comment by Alexander F. Rødseth (xyproto) - Thursday, 13 June 2019, 09:18 GMT
ok, changed

depends=(gmp jdk-openjdk libarchive libedit libyaml openssl qt5-tools readline)
makedepends=(cmake junit libjpeg libxft libxinerama libxpm ninja unixodbc)

to

depends=(libarchive libyaml qt5-base)
makedepends=(cmake jdk-openjdk junit libjpeg libxft libxinerama libxpm ninja unixodbc)

and added jdk-openjdk as an optdepend

Some packages could be removed as dependencies because they are already dependencies of dependencies, so it won't make much practical difference at installation-time.

The updated swi-prolog package will appear in [community] shortly.

Loading...