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.

Sign up for GitHub

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

Jump to bottom

Add more cygwin system packages #29397

Closed
mkoeppe opened this issue Mar 23, 2020 · 31 comments
Closed

Add more cygwin system packages #29397

mkoeppe opened this issue Mar 23, 2020 · 31 comments
Labels
c: porting p: major / 3 t: enhancement
Milestone
sage-9.1

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Mar 23, 2020

cygwin-standard builds take too long because several system packages are not installed or recognized.

https://github.com/mkoeppe/sage/runs/525632833?check_suite_focus=true

configure: notice: the following SPKGs did not find equivalent system packages: 
arb cbc cliquer eclib ecm fflas_ffpack flint fplll gf2x gfan givaro glpk gp2c gsl lcalc 
libatomic_ops libsemigroups lrcalc m4ri m4rie mpfi nauty ncurses ntl openblas 
pari pari_elldata pari_galdata pari_galpol pari_nftables pari_seadata pari_seadata_small 
pcre planarity python3 r readline rw symmetrica tachyon zeromq
checking for the package system in use... cygwin
configure: hint: installing the following system packages is recommended and may avoid building some of the above SPKGs from source:
configure:   $ # first install apt-cyg from https://github.com/transcode-open/apt-cyg
apt-cyg install libflint-devel libgsl-devel ncurses-devel libreadline-devel libzmq-devel

With this ticket (https://github.com/mkoeppe/sage/runs/528920665):

configure: notice: the following SPKGs did not find equivalent system packages: 
arb cbc cliquer eclib ecm fflas_ffpack fplll gf2x gfan givaro gp2c gsl lcalc 
libsemigroups lrcalc m4ri m4rie mpfi nauty openblas 
pari pari_elldata pari_galdata pari_galpol pari_nftables pari_seadata pari_seadata_small 
planarity r rw symmetrica tachyon zeromq
checking for the package system in use... cygwin
configure: hint: installing the following system packages is recommended and may avoid building some of the above SPKGs from source:
configure:   $ # first install apt-cyg from https://github.com/transcode-open/apt-cyg
apt-cyg install libgsl-devel liblapack-devel libopenblas R libzmq-devel

Follow-up for BLAS at #29398.

CC: @dimpase @embray @tscrim

Component: porting

Author: Matthias Koeppe

Branch/Commit: f29a168

Reviewer: Dima Pasechnik

Issue created by migration from https://trac.sagemath.org/ticket/29397
@mkoeppe mkoeppe added this to the sage-9.1 milestone Mar 23, 2020
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 23, 2020

Branch: u/mkoeppe/add_more_cygwin_system_packages

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 23, 2020

New commits:

56a4f4cbuild/pkgs/glpk/distros/cygwin.txt: New

@mkoeppe

This comment has been minimized.

Sign in to view
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 23, 2020

Commit: 56a4f4c

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 23, 2020

Changed commit from 56a4f4c to ba09bba

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 23, 2020

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

ba09bbabuild/pkgs/ncurses/distros/cygwin.txt: Fixup

@mkoeppe

This comment has been minimized.

Sign in to view
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 23, 2020

comment:6

The mpfr test is from #27258. cygwin has libmpfr-devel-4.0.2 - is that too old?

@dimpase
Copy link
Member

dimpase commented Mar 23, 2020

comment:7

no, 4.0.1 should be ok

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 23, 2020

Changed commit from ba09bba to 391d224

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 23, 2020

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

391d224build/pkgs/openblas/distros/cygwin.txt: New

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 23, 2020

comment:9

Replying to @dimpase:

no, 4.0.1 should be ok

Right, sorry, I misread the log

@mkoeppe

This comment has been minimized.

Sign in to view
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 23, 2020

Changed commit from 391d224 to e9f831a

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 23, 2020

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

e9f831abuild/pkgs/ntl/distros/cygwin.txt: New

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 23, 2020

Changed commit from e9f831a to 905148b

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 23, 2020

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

905148bbuild/pkgs/pcre/distros/cygwin.txt: New

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 23, 2020

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

12540acbuild/pkgs/r/distros/cygwin.txt: New
3fd30e3build/pkgs/libatomic_ops/distros/cygwin.txt: New
f29a168build/pkgs/cygwin.txt: python -> python37

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Mar 23, 2020

Changed commit from 905148b to f29a168

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 23, 2020

Author: Matthias Koeppe

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 24, 2020

comment:15

Openblas is not recognized

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 24, 2020

comment:16

Zmq is 4.2.3; our minimum is 4.2.5

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 24, 2020

comment:17

Cygwin only has package config files blas.pc, cblas.pc, lapack.pc, not openblas.pc.
libopenblas overrides the dll /usr/bin/cygblas-0.dll

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 24, 2020

comment:18

I've opened #29398 for openblas.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 24, 2020

comment:19

Replying to @mkoeppe:

Zmq is 4.2.3; our minimum is 4.2.5

Zmq changelog https://github.com/zeromq/libzmq/releases

Our last update in #25906 was from 4.0.5.p2 and motivated with dropping some build patches that were upstreamed.

@mkoeppe

This comment has been minimized.

Sign in to view
@dimpase
Copy link
Member

dimpase commented Mar 24, 2020

comment:21

We can lower the minimum zmq version requirement, or at least try to.

@mkoeppe
Copy link
Contributor Author

mkoeppe commented Mar 24, 2020

comment:22

I don't think it adds much to the build time, and eventually cygwin will catch up

@dimpase
Copy link
Member

dimpase commented Mar 24, 2020

comment:23

ok

@dimpase
Copy link
Member

dimpase commented Mar 24, 2020

Reviewer: Dima Pasechnik

@vbraun
Copy link
Member

vbraun commented Mar 29, 2020

Changed branch from u/mkoeppe/add_more_cygwin_system_packages to f29a168

@vbraun vbraun closed this as completed in 23bd1c7 Mar 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: porting p: major / 3 t: enhancement
Projects
None yet
Milestone
sage-9.1
Development

No branches or pull requests
3 participants
@vbraun @dimpase @mkoeppe