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

Fix conda workflow #33828

Closed
tobiasdiez opened this issue May 9, 2022 · 12 comments
Closed

Fix conda workflow #33828

tobiasdiez opened this issue May 9, 2022 · 12 comments

Comments

@tobiasdiez
Copy link
Contributor

Currently is broken because of issues with pari.

https://github.com/sagemath/sage/runs/6236167749?check_suite_focus=true

CC: @isuruf @dimpase @saraedum @mkoeppe @slel

Component: build

Author: Matthias Koeppe

Branch/Commit: 1514218

Reviewer: Dima Pasechnik

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

@tobiasdiez tobiasdiez added this to the sage-9.6 milestone May 9, 2022
@mkoeppe
Copy link
Contributor

mkoeppe commented May 9, 2022

comment:1

@isuruf Where should we point Sage users who encounter problems of this kind with the conda packages?

@saraedum
Copy link
Member

saraedum commented May 9, 2022

comment:2

I think they should open an issue on the feedstock, i.e., at https://github.com/conda-forge/sage-feedstock/issues.

@isuruf
Copy link
Member

isuruf commented May 9, 2022

comment:3

I'm not sure why the single threaded version was picked up. You can change the entry in build/pkgs/pari/distros/conda.txt to pari=*=*_pthread to ensure that we pick up the threaded version.

@mkoeppe
Copy link
Contributor

mkoeppe commented May 10, 2022

Branch: u/mkoeppe/fix_conda_workflow

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 10, 2022

Commit: 1514218

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented May 10, 2022

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

1514218src/doc/en/installation/conda.rst: Where to report failures with conda

@mkoeppe
Copy link
Contributor

mkoeppe commented May 10, 2022

@mkoeppe
Copy link
Contributor

mkoeppe commented May 10, 2022

Author: Matthias Koeppe

@dimpase
Copy link
Member

dimpase commented May 10, 2022

Reviewer: Dima Pasechnik

@dimpase
Copy link
Member

dimpase commented May 10, 2022

comment:8

tests look good (some unrelated doctest errors)

@tobiasdiez
Copy link
Contributor Author

comment:9

Thanks for the quick fix!

@mkoeppe mkoeppe modified the milestones: sage-9.6, sage-9.7 May 15, 2022
@vbraun
Copy link
Member

vbraun commented May 17, 2022

Changed branch from u/mkoeppe/fix_conda_workflow to 1514218

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