Skip to content

Commit acb1953

Browse files
author
Release Manager
committed
gh-36468: Fix mamba install in conda ci workflow <!-- ^^^^^ Please provide a concise, informative and self-explanatory title. Don't put issue numbers in there, do this in the PR body below. For example, instead of "Fixes #1234" use "Introduce new method to calculate 1+1" --> <!-- Describe your changes here in detail --> <!-- Why is this change required? What problem does it solve? --> <!-- If this PR resolves an open issue, please link to it here. For example "Fixes #12345". --> <!-- If your change requires a documentation PR, please link it appropriately. --> Mamba currently has problems with dependencies installed from other channels (libarchive/libarchive#1857, conda-incubator/setup-miniconda#292, conda-forge/libarchive-feedstock#69), see eg h ttps://github.com/sagemath/sage/actions/runs/6525067702/job/17717363515. As workaround we use miniforge now (which comes with a preinstalled mamba from conda-forge) ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> <!-- If your change requires a documentation PR, please link it appropriately --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> <!-- Feel free to remove irrelevant items. --> - [ ] The title is concise, informative, and self-explanatory. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on - #12345: short description why this is a dependency - #34567: ... --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> URL: #36468 Reported by: Tobias Diez Reviewer(s): Isuru Fernando, Matthias Köppe
2 parents eb03f27 + edbb8a7 commit acb1953

File tree

1 file changed

+4
-15
lines changed

1 file changed

+4
-15
lines changed

.github/workflows/ci-conda.yml

Lines changed: 4 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -39,18 +39,6 @@ jobs:
3939
env:
4040
GH_TOKEN: ${{ github.token }}
4141

42-
- name: Check for Miniconda
43-
id: check_conda
44-
run: echo ::set-output name=installed::$CONDA
45-
46-
# Miniconda is installed by default in the ubuntu-latest, however not in the act-image to run it locally
47-
- name: Install Miniconda
48-
if: steps.check_conda.outputs.installed == ''
49-
run: |
50-
wget https://repo.anaconda.com/miniconda/Miniconda3-py39_4.10.3-Linux-x86_64.sh -O ~/miniconda.sh
51-
bash ~/miniconda.sh -b -p $HOME/miniconda
52-
echo "CONDA=$HOME/miniconda" >> $GITHUB_ENV
53-
5442
- name: Create conda environment files
5543
run: ./bootstrap-conda
5644

@@ -61,12 +49,13 @@ jobs:
6149
key:
6250
${{ runner.os }}-conda-${{ hashFiles('src/environment-3.11.yml') }}
6351

64-
- name: Setup Conda
52+
- name: Setup Conda environment
6553
uses: conda-incubator/setup-miniconda@v2
6654
with:
6755
python-version: ${{ matrix.python }}
68-
mamba-version: "*"
69-
channels: conda-forge,defaults
56+
miniforge-version: latest
57+
use-mamba: true
58+
channels: conda-forge
7059
channel-priority: true
7160
activate-environment: sage
7261
environment-file: src/${{ matrix.conda-env }}-${{ matrix.python }}.yml

0 commit comments

Comments
 (0)