-
Notifications
You must be signed in to change notification settings - Fork 7
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
Setup error on macOS-latest when adding conda path to PATH (received undefined) #432
Comments
Hi @pauleve thanks for the bug report. setup-conda/src/conda_actions.ts Lines 62 to 63 in 2a87cac
I will have a closer look at the weekend if I manage to find the time. |
@all-contributors please add @pauleve for bug |
I've put up a pull request to add @pauleve! 🎉 |
This action uses the preinstalled `miniconda` and determines its installation base path reading the `CONDA` environ variable. Up until `macOS-12` this worked for all runner images, however `macOS-13` and `macOS-14` (current `macos-latest`) do not install `miniconda` and workflows fail with: ``` Error: The "path" argument must be of type string. Received undefined ``` See #432 With this change, the error will be: ``` Error: Could not determine conda base path, it seams conda is not installed. ``` On any system without the `CONDA` environ variable set. And for macos runner images > 12: ``` Error: Could not determine conda base path, it seams conda is not installed. MacOS images newer than "macos-12" (i.e. "macOS-latest") are known to be incompatible with this action due to a missing miniconda installation. See: #432 ```
I did some digging + debugging and found that As a workaround, I see 2 options:
|
Thank you so much for the investigation, and the workarounds. |
So they decided there will be no Miniconda in future versions of macOS images. What next? Install Miniconda manually? |
- Replace conda install, see s-weigand/setup-conda#432 - Update https://github.com/actions/checkout
* Update python-app.yml - Replace conda install, see s-weigand/setup-conda#432 - Update https://github.com/actions/checkout * Use ts-graphviz/setup-graphviz@v2 See https://github.com/marketplace/actions/setup-graphviz Replaces `conda install -c anaconda graphviz` on MacOS * Use psycopg2-binary Fixes ``` Error: pg_config executable not found. pg_config is required to build psycopg2 from source. Please add the directory containing pg_config to the $PATH or specify the full executable path with the option: python setup.py build_ext --pg-config /path/to/pg_config build ... or with the pg_config option in 'setup.cfg'. If you prefer to avoid building psycopg2 from source, please install the PyPI 'psycopg2-binary' package instead. For further information please check the 'doc/src/install.rst' file (also at <https://www.psycopg.org/docs/install.html>). ``` * Update setup.py
Running
s-weigand/setup-conda@v1
onmacOS-latest
with default options fails at the step "Adding conda path to PATH" withError: The "path" argument must be of type string. Received undefined
Recipe is
(src: https://github.com/colomoto/colomoto-conda/blob/master/.github/workflows/push.yml)
Log is
(src: https://github.com/colomoto/colomoto-conda/actions/runs/8986528695/job/24682859612)
Not sure what I'm doing wrong here, and it was working fine until recently..
The text was updated successfully, but these errors were encountered: