Remove the ore_algebra optional package

[sagetrac-tmonteil]

The version shipped by Sage has been out of date for a while and periodically breaks when Sage makes incompatible changes, while up-to-date versions can easily be installed from pip.

Now pip-available from https://github.com/mkauers/ore_algebra

CC: @mkauers @mezzarobba @slel

Component: packages: optional

Keywords: upgrade, ore_algebra

Author: Thierry Monteil, Marc Mezzarobba

Branch/Commit: 1f64d51

Reviewer: Frédéric Chapoton

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

[sagetrac-tmonteil]

Branch: u/tmonteil/fetch_ore_algebra_directly_from_upstream

[sagetrac-tmonteil]

Commit: ff802e2

[sagetrac-tmonteil]

New commits:

ff802e2

26332 : fetch ore_algebra directly from upstream

[jdemeyer]

Replying to @sagetrac-tmonteil:

(A) ore_algebra 0.3 is not working with Sage for a wile and upstream does not release versions anymore.

(B) Hence, we should [...] fetch ore_algebra directly from upstream.

I don't see how premise (A) implies conclusion (B). There are other packages that don't have releasea and which are still packaged normally.

[slel]

comment:5

New version: ore_algebra 0.4 was released on 2019-05-17.

Can someone test whether it installs and works with current Sage?

[slel]

Changed keywords from none to upgrade, ore_algebra

[mezzarobba]

comment:7

Not tested.

---

New commits:

8de39b6

#26332 Remove the ore_algebra package

[mezzarobba]

Changed author from Thierry Monteil to Thierry Monteil, Marc Mezzarobba

[mezzarobba]

Changed commit from ff802e2 to 8de39b6

[mezzarobba]

Changed branch from u/tmonteil/fetch_ore_algebra_directly_from_upstream to u/mmezzarobba/remove_ore_algebra

[jdemeyer]

comment:9

Why remove the package instead of upgrading, as Samuel proposed?

[mezzarobba]

comment:10

Replying to @jdemeyer:

Why remove the package instead of upgrading, as Samuel proposed?

It is an external package with a dependency on Sage. I think people should install it from pip or whatever they like; I don't see the point of having it as a Sage package. And while I try to make sure that the master branch always works with the latest Sage release, the Sage package is broken half of the time. If someone wants to take care of it on a regular basis (and bug Manuel to tag a point release every time something breaks), that's fine with me of course, but that hasn't been the case in the last few years.

[jdemeyer]

comment:11

Replying to @mezzarobba:

I think people should install it from pip or whatever they like; I don't see the point of having it as a Sage package.

There are many packages for which this argument could be made, but we still ship them as Sage packages. The advantages of having a Sage package are user friendliness (sage -i ore_algebra works) and automatic updates with version checking.

I try to make sure that the master branch always works with the latest Sage release

But not everybody uses the latest Sage release. You may break stuff if a newer version of ore_algebra doesn't work with an older version of Sage.

[mezzarobba]

comment:12

Replying to @jdemeyer:

Replying to @mezzarobba:

I think people should install it from pip or whatever they like; I don't see the point of having it as a Sage package.

There are many packages for which this argument could be made, but we still ship them as Sage packages. The advantages of having a Sage package are user friendliness (sage -i ore_algebra works) and automatic updates with version checking.

If it works, yes, except that I personally consider sage -i a strictly less user-friendly option than sage -pip install (< pip install < apt install).

I try to make sure that the master branch always works with the latest Sage release

But not everybody uses the latest Sage release. You may break stuff if a newer version of ore_algebra doesn't work with an older version of Sage.

Yes, of course. I'm alking about the master branch... Fwiw I also try to keep it compatible with earlier versions when that's easy to do.

[mezzarobba]

comment:13

(If anyone really considers it important to have ore_algebra as a Sage package, feel free to revert my changes to this ticket.)

PS regarding Thierry's version: I personally don't like the idea of a “package” that actually installs a version that was not checked by the packager, but I won't argue if people want that.

[mezzarobba]

comment:14

So: If someone prefers the status quo or Thierry's original solution to removing the package, could you please state it clearly? Otherwise, this should be an easy review... Thanks!

[embray]

comment:15

Moving tickets from the Sage 8.8 milestone that have been actively worked on in the last six months to the next release milestone (optimistically).

[sagetrac-git]

Changed commit from 8de39b6 to 1f64d51

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

1f64d51

#26332 Remove the ore_algebra package

[fchapoton]

comment:17

ok, let it go

[fchapoton]

Reviewer: Frédéric Chapoton

[fchapoton]

Changed reviewer from Frédéric Chapoton to none

[fchapoton]

Reviewer: Frédéric Chapoton

[vbraun]

Changed branch from u/mmezzarobba/remove_ore_algebra to 1f64d51