Minimize use of is_package_installed at runtime

[kiwifb]

This is helpful for sage-on-distributions. Typically sage-on-distribution do not want, or need, to call sage's internal packaging system.

I raised this as part of Sage days 77.

Related tickets:

• Replace is_package_installed with Features #20382 Replace is_package_installed with Features
• Don't use is_package_installed('bliss') #21289 Don't use is_package_installed('bliss')
• speed regression in is_package_installed #21291 speed regression in is_package_installed
• Timeout in graphs/tutte_polynomial.py #21355 Timeout in graphs/tutte_polynomial.py

CC: @slel

Component: packages: optional

Keywords: sagedays 77

Author: François Bissey

Branch/Commit: u/fbissey/is_pkg_installed @ aba00ce

Issue created by migration from https://trac.sagemath.org/ticket/20377

[mezzarobba]

comment:2

Your branch points to the latest beta, did you forget to push or something?

[kiwifb]

comment:4

I forgot to commit :(

[sagetrac-git]

Changed commit from 7ffcdd1 to a8e2087

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

a8e2087

first draft of minimizing is_package_installed

[videlec]

comment:7

You forget a colon

  File "/home/worker/sage-patchbot/local/lib/python2.7/site-packages/sage/groups/generic.py", line 1407
    if not os.path.isfile(os.path.join(SAGE_LOCAL, "gap", "latest", "small", "readsml.g"))

[sagetrac-git]

Changed commit from a8e2087 to d7d8483

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

d7d8483

Add forgotten ":"

[kiwifb]

comment:9

If that's all I forgot that's not a bad score. This particular one was different in sage-on-gentoo because I don't ship this database separately from gap like it is done in sage so it doesn't need testing.
I'll still need to patch that one one way or another because the path will be different on gentoo. Alternative proposal welcome.

[sagetrac-git]

Changed commit from d7d8483 to aba00ce

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

aba00ce

Merge branch 'develop' into is_pkg_installed

[saraedum]

comment:11

It seems we are working on the same issue ;)
#20382 does essentially the same, maybe you want to have a look.

It seems to me that my approach is slightly more generic but I am open for discussion.

[jdemeyer]

comment:13

Is this still relevant or should it be closed in the light of #20382?

[kiwifb]

comment:14

Replying to @jdemeyer:

Is this still relevant or should it be closed in the light of #20382?

I think I should have marked this one duplicate months ago. There is nothing in here that #20382 doesn't do better.