-
-
Notifications
You must be signed in to change notification settings - Fork 480
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: Various CI fixes #29929
Comments
comment:1
I'll repeat a comment I made at #29706: I really needed an environment without homebrew to replicate the failures; just having a minimal homebrew installation, or having one with temporarily moved (So I'm emphasizing that I'm agreeing with the summary: "without homebrew".) |
This comment has been minimized.
This comment has been minimized.
Upstream: Reported upstream. No feedback yet. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Commit: |
New commits:
|
This comment has been minimized.
This comment has been minimized.
Branch pushed to git repo; I updated commit sha1. New commits:
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
This comment has been minimized.
This comment has been minimized.
Branch pushed to git repo; I updated commit sha1. New commits:
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
comment:21
is this still "new"? |
This comment has been minimized.
This comment has been minimized.
Author: Matthias Koeppe |
comment:23
The cygwin CI wishlist has been moved to #30154. Needs review |
Reviewer: Jonathan Kliem |
comment:24
Looks reasonable. Can you provide an example of this ticket in action (you probably have been using it already anyway?). What's |
comment:27
LGTM. |
comment:28
Thanks! |
Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. New commits:
|
Changed branch from u/mkoeppe/tox_ini__add_a_macos_environment_without_homebrew__conda to |
Add a homebrew environment
local-homebrew-macos-usrlocal
that uses the global homebrew in/usr/local
. This will be faster and may be more robust for GitHub Actions builds because "bottles" of more packages can be used. (For a while, for example, allhomebrew-standard
builds fail because the homebrewppl
package cannot be installed from source - "brew install -s ppl" fails - 401 Unauthorized Homebrew/homebrew-core#57499)Disable testsuites that need pip (ssl)
This is on top of:
Also, add "make dist" testing to tox.ini (prompted by breakage in Clean-up for src/bin/sage-env, move src/bin/sage-clone-source, src/bin/sage-sdist to build/bin #29825). This is Fix "make dist", add test run to GitHub Actions #30088
Add a macos environment without homebrew, conda: That's a configuration that was causing trouble in Move Extension options from src/module_list.py to "distutils:" directives in the individual files (part 1 - some packages without OptionalExtensions) #29706 and is so far not tested in the GitHub CI tests. This is added by Fix "make dist", add test run to GitHub Actions #30088 too (
make dist
)Depends on #29851
Depends on #29105
Depends on #30064
Depends on #30048
Depends on #30088
Depends on #30118
Upstream: Reported upstream. No feedback yet.
CC: @kliem
Component: porting
Author: Matthias Koeppe
Branch/Commit:
b62f775
Reviewer: Jonathan Kliem
Issue created by migration from https://trac.sagemath.org/ticket/29929
The text was updated successfully, but these errors were encountered: