Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tox.ini: Add opensuse, add/update opensuse.txt across build/pkgs/ #30619

Closed
jengelh mannequin opened this issue Sep 20, 2020 · 81 comments
Closed

tox.ini: Add opensuse, add/update opensuse.txt across build/pkgs/ #30619

jengelh mannequin opened this issue Sep 20, 2020 · 81 comments

Comments

@jengelh
Copy link
Mannequin

jengelh mannequin commented Sep 20, 2020

To test openSUSE Leap:

tox -e docker-opensuse-standard

Depends on #30663
Depends on #30900
Depends on #30899

CC: @kliem @dimpase @jhpalmieri @seblabbe

Component: packages: standard

Author: Jan Engelhardt, Matthias Koeppe

Branch/Commit: e5d114b

Reviewer: Matthias Koeppe, Jonathan Kliem, Dima Pasechnik

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

@jengelh jengelh mannequin added this to the sage-9.2 milestone Sep 20, 2020
@jengelh
Copy link
Mannequin Author

jengelh mannequin commented Sep 20, 2020

comment:1

(singleton repository with write permissions for everyone‽ No bot seems to add a note to the ticket when a push occurs. Very strange, this Trac system.)

    e4a917d10ec495da90818eeaa60b7711c2e3cf25        refs/heads/u/gh-jengelh/30619

A number of third-party packages are available, or will be Real-Soon-Now™, in openSUSE. Add their package names or virtual names according to the txt file scheme of the other distros.

@fchapoton
Copy link
Contributor

New commits:

e4a917dbuild/pkgs/*: add/update distros/opensuse.txt

@fchapoton
Copy link
Contributor

Commit: e4a917d

@fchapoton
Copy link
Contributor

Branch: u/gh-jengelh/30619

@fchapoton
Copy link
Contributor

comment:3

you need to write your real name in the Author field above here, and to set the ticket to needs review

@mkoeppe
Copy link
Contributor

mkoeppe commented Sep 20, 2020

comment:4

Thanks for this contribution.

We would also like to add CI infrastructure for testing the build on opensuse using Docker. This would follow the same steps as #30044 (for voidlinux).

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 20, 2020

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

0ea1e48build/pkgs/*: add/update distros/opensuse.txt

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 20, 2020

Changed commit from e4a917d to 0ea1e48

@jengelh
Copy link
Mannequin Author

jengelh mannequin commented Sep 20, 2020

Author: Jan Engelhardt

@mkoeppe

This comment has been minimized.

@mkoeppe
Copy link
Contributor

mkoeppe commented Sep 21, 2020

Changed branch from u/gh-jengelh/30619 to u/mkoeppe/30619

@mkoeppe
Copy link
Contributor

mkoeppe commented Sep 21, 2020

comment:9

Next, need the following two files

sed: ./build/pkgs/opensuse.txt: No such file or directory
sed: ./build/pkgs/opensuse-bootstrap.txt: No such file or directory

New commits:

3e08239tox.ini: Add opensuse

@mkoeppe
Copy link
Contributor

mkoeppe commented Sep 21, 2020

Changed commit from 0ea1e48 to 3e08239

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 22, 2020

Changed commit from 3e08239 to d55d81a

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 22, 2020

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

d11f55bbuild/pkgs/opensuse.txt, opensuse-bootstrap.txt: New
0c4134abuild/bin/sage-print-system-package-command, build/bin/write-dockerfile.sh: Handle opensuse
f8e4821tox.ini: docker now needs HOME too
d55d81aRemove opensuse.txt for deleted packages

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 22, 2020

Changed commit from d55d81a to 58a4d56

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 22, 2020

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

f436ba1build/pkgs/*: add/update distros/opensuse.txt
3cad935tox.ini: Add opensuse
5d0d9a5build/pkgs/opensuse.txt, opensuse-bootstrap.txt: New
fde82bbbuild/bin/sage-print-system-package-command, build/bin/write-dockerfile.sh: Handle opensuse
be2b22etox.ini: docker now needs HOME too
58a4d56Remove opensuse.txt for deleted packages

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 22, 2020

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

3282aebbuild/pkgs/*/distros/opensuse.txt: One package per line, add shell quotes
f1625f7build/pkgs/opensuse.txt, opensuse-bootstrap.txt: Populate
8426571build/bin/write-dockerfile.sh: Set options for 'zypper install'

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 22, 2020

Changed commit from 58a4d56 to 8426571

@mkoeppe
Copy link
Contributor

mkoeppe commented Sep 22, 2020

comment:13
No provider of 'arb-devel' found.
No provider of 'brial-devel' found.
No provider of 'cddlib-tools' found.
No provider of 'cliquer' found.
No provider of 'cliquer-devel' found.
No provider of 'edge-addition-planarity-suite' found.
No provider of 'giac-devel' found.
No provider of 'libbraiding-devel' found.
No provider of 'pkgconfig(cddlib)' found.
No provider of 'pkgconfig(gd)' found.
No provider of 'pkgconfig(gf2x)' found.
No provider of 'pkgconfig(m4rie)' found.
No provider of 'pkgconfig(readline)' found.
No provider of 'ppl-devel' found.
No provider of 'sympow' found.

...

configure: notice: the following SPKGs did not find equivalent system packages: arb brial bzip2 cbc cddlib cliquer cmake coxeter3 eclib ecm fflas_ffpack flint flintqs fplll freetype gc gf2x giac givaro gp2c isl lcalc libatomic_ops libbraiding libgd libnauty libpng libsemigroups lrcalc m4ri m4rie ninja_build openssl palp pandoc pari pari_elldata pari_galdata pari_galpol pari_nftables pari_seadata pari_seadata_small pcre perl_cpan_polymake_prereq perl_term_readline_gnu planarity ppl python3 r readline rw symmetrica sympow zeromq zn_poly
checking for the package system in use... opensuse
configure: hint: installing the following system packages is recommended and may avoid building some of the above SPKGs from source:
configure:   $ sudo zypper install arb-devel brial-devel "pkgconfig(bzip2)" cddlib-tools "pkgconfig(cddlib)" cliquer cliquer-devel cmake coxeter "pkgconfig(fflas-ffpack)" flint-devel "pkgconfig(fplll)" "pkgconfig(freetype2)" "pkgconfig(bdw-gc)" "pkgconfig(gf2x)" giac-devel "pkgconfig(givaro)" gp2c "pkgconfig(isl)" "pkgconfig(atomic_ops)" libbraiding-devel gd "pkgconfig(gd)" "pkgconfig(libpng16)" "pkgconfig(libsemigroups)" "pkgconfig(m4ri)" "pkgconfig(m4rie)" ninja "pkgconfig(ssl)" pandoc pari-devel pari-gp pari-elldata pari-galdata pari-galpol pari-nftables pari-seadata "pkgconfig(libpcre)" "pkgconfig(libpcreposix)" "pkgconfig(libpcrecpp)" "perl(Term::ReadLine::Gnu)" edge-addition-planarity-suite ppl-devel python3-devel R-base "pkgconfig(readline)" sympow "pkgconfig(libzmq)" zn_poly-devel

@mkoeppe mkoeppe changed the title update opensuse.txt across build/pkgs/ tox.ini: Add opensuse, add/update opensuse.txt across build/pkgs/ Sep 22, 2020
@mkoeppe
Copy link
Contributor

mkoeppe commented Sep 22, 2020

Changed author from Jan Engelhardt to Jan Engelhardt, Matthias Koeppe

@mkoeppe
Copy link
Contributor

mkoeppe commented Sep 23, 2020

Reviewer: Matthias Koeppe, ...

@mkoeppe
Copy link
Contributor

mkoeppe commented Sep 25, 2020

Dependencies: #30663

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 25, 2020

Changed commit from 8426571 to 1a7d52f

@mkoeppe
Copy link
Contributor

mkoeppe commented Nov 30, 2020

comment:47

Replying to @dimpase:

And edge-addition-planarity-suite-devel is missing, and edge-addition-planarity-suite does not install headers.

The latter should be fixed.

Please push a fix for this one to the ticket.

@mkoeppe
Copy link
Contributor

mkoeppe commented Nov 30, 2020

comment:48

The cddlib header issues are best taken care of in #29413

@jengelh
Copy link
Mannequin Author

jengelh mannequin commented Nov 30, 2020

comment:49

cddlib is not picked up as cdd.h is expected in /usr/include, but is installed in /usr/include/cddlib.

This is how cddlib 0.94l installs itself. Since cddlib.pc does not specify any -I other than the default -I${includedir}, the developers mean that cdd.h is now to be included through a #include <cddlib/cdd.h> line, rather than { #include <cdd.h> plus some forced -I injection in the project making use of cdd}.

coxeter has as upstream an old ​https://github.com/tscrim/coxeter - and does not install headers, oops...

Surely this is an upstream problem, is it not?

21:12 a4:../coxeter/coxeter-3.1+git7 » make install DESTDIR=$PWD/rt
make: *** No rule to make target 'install'.  Stop.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 30, 2020

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

df606a4build/pkgs/planarity/distros/opensuse.txt: Use -devel

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Nov 30, 2020

Changed commit from e7709d4 to df606a4

@mkoeppe
Copy link
Contributor

mkoeppe commented Nov 30, 2020

Changed reviewer from Matthias Koeppe, Jonathan Kliem to Matthias Koeppe, Jonathan Kliem, Dima Pasechnik

@mkoeppe
Copy link
Contributor

mkoeppe commented Nov 30, 2020

comment:51

I would think other changes are beyond the scope of this ticket.

@dimpase
Copy link
Member

dimpase commented Nov 30, 2020

comment:52

on tumbleweed (that is, with gcc 10)
the sagelib interface to lcalc does not build

[sagelib-9.3.beta2] In file included from /home/dima/tracmirror/local/include/Lfunction/L.h:537,
[sagelib-9.3.beta2]                  from build/cythonized/sage/libs/lcalc/lcalc_sage.h:1,
[sagelib-9.3.beta2]                  from build/cythonized/sage/libs/lcalc/lcalc_Lfunction.cpp:674:
[sagelib-9.3.beta2] /home/dima/tracmirror/local/include/Lfunction/Lvalue.h: In member function �~@~XComplex L_function<ttype>::value_via_Riemann_sum(Complex, const char*) [with ttype = std::complex<double>]�~@~Y:
[sagelib-9.3.beta2] /home/dima/tracmirror/local/include/Lfunction/Lvalue.h:489:5: error: control reaches end of non-void function [-Werror=return-type]
[sagelib-9.3.beta2]   489 |     }
[sagelib-9.3.beta2]       |     ^
[sagelib-9.3.beta2] /home/dima/tracmirror/local/include/Lfunction/Lvalue.h: In member function �~@~XComplex L_function<ttype>::value_via_Riemann_sum(Complex, const char*) [with ttype = int]�~@~Y:

@dimpase
Copy link
Member

dimpase commented Nov 30, 2020

comment:53

let's open another ticket to fix lcalc on gcc-10.
Basically, the problem is that the function
template <class ttype> Complex L_function <ttype>::value_via_gamma_sum(..)
does a bunch of conditional returns, but there is no "final" else, so compiler
is right to be unhappy.

@dimpase
Copy link
Member

dimpase commented Nov 30, 2020

comment:54

please see #30987

@mkoeppe
Copy link
Contributor

mkoeppe commented Nov 30, 2020

comment:55

For lcalc, see also #28015 (Drop lcalc)

@kiwifb
Copy link
Member

kiwifb commented Dec 13, 2020

comment:56

We removed python_openid earlier in the 9.3 cycle :) do not re-add bits of it with this ticket.

Warning: build/pkgs/python_openid/type is missing.
Warning: build/pkgs/python_openid/type is missing.
Warning: build/pkgs/python_openid/type is missing.
Warning: build/pkgs/python_openid/type is missing.
Warning: build/pkgs/python_openid/type is missing.
Warning: build/pkgs/python_openid/type is missing.

that's when bootstrapping the documentation. And rightly so

fbissey@moonloop /dev/shm/portage/sci-mathematics/sage-9999/work/sage-9999 $ ll build/pkgs/python_openid/
total 0
drwxr-xr-x   3 fbissey fbissey   60 Dec 13 15:30 .
drwxr-xr-x 303 fbissey fbissey 6.4K Dec 13 15:30 ..
drwxr-xr-x   2 fbissey fbissey   60 Dec 13 15:30 distros
fbissey@moonloop /dev/shm/portage/sci-mathematics/sage-9999/work/sage-9999 $ ll build/pkgs/python_openid/distros/
total 4.0K
drwxr-xr-x 2 fbissey fbissey 60 Dec 13 15:30 .
drwxr-xr-x 3 fbissey fbissey 60 Dec 13 15:30 ..
-rw-r--r-- 1 fbissey fbissey 23 Dec 13 15:30 opensuse.txt

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 13, 2020

Changed commit from df606a4 to e5d114b

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Dec 13, 2020

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

e5d114bbuild/pkgs/python_openid/distros/opensuse.txt: Remove

@mkoeppe
Copy link
Contributor

mkoeppe commented Dec 13, 2020

comment:58

The ticket is already merged in Volker's branch, but here we go...

@kiwifb
Copy link
Member

kiwifb commented Dec 13, 2020

comment:59

Replying to @mkoeppe:

The ticket is already merged in Volker's branch, but here we go...

He probably won't see it then. We should be ready to have a follow up to clean.

@vbraun
Copy link
Member

vbraun commented Dec 14, 2020

Changed branch from u/mkoeppe/30619 to e5d114b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants