[PR #583] [MERGED] disable buidling/deployment of python wheels #1228

New Issue

claunia · 2026-01-29T20:53:05Z

claunia commented

2026-01-29 20:53:05 +00:00

📋 Pull Request Information

Original PR: https://github.com/google/brotli/pull/583
Author: @anthrotype
Created: 8/8/2017
Status: ✅ Merged
Merged: 8/23/2017
Merged by: @eustas

Base: master ← Head: no-python-deploy

📝 Commits (10+)

75aa483 [appveyor] remove 'deploy' stage; only test python 2.7 and 3.6
f54b514 remove terrify submodule as not needed any more
2475c01 [travis] just test py2.7 and 3.6 on linux; remove extra osx python builds
86d4b19 [setup.py] only rebuild if dependency are newer; fix typo in list of 'depends'
384f8b6 [ci] only run 'python setup.py test'
44b5e66 [Makefile] run 'build_ext --inplace' instead of 'develop' as default target
4e370f5 [Makefile] add 'develop' target; remove unnecessary 'tests' target
0a3c14c [Makefile] setup.py test requires setuptools; run python -m unittest
47b4aab [python/README.md] add ref to 'develop' target; remove 'tests', just 'make test'
44f6d75 [setup.py] import modules as per nicksay's comment

📊 Changes

10 files changed (+60 additions, -361 deletions)

View changed files

📝 .gitmodules (+0 -3)
📝 .travis.yml (+16 -17)
📝 python/Makefile (+8 -5)
📝 python/README.md (+9 -8)
📝 scripts/.travis.sh (+1 -19)
📝 scripts/appveyor.yml (+8 -50)
➖ scripts/install_py.ps1 (+0 -177)
➖ scripts/run_with_compiler.cmd (+0 -80)
➖ scripts/terryfy (+0 -1)
📝 setup.py (+18 -1)

📄 Description

Since this is now done in a separate https://github.com/google/brotli-wheels, it makes sense remove that from the upstream brotli's Travis and Appveyor setup.
We still run python setup.py test on both 2.7 and 3.6 on linux and windows, as well as on the system's python on macOS.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/google/brotli/pull/583 **Author:** [@anthrotype](https://github.com/anthrotype) **Created:** 8/8/2017 **Status:** ✅ Merged **Merged:** 8/23/2017 **Merged by:** [@eustas](https://github.com/eustas) **Base:** `master` ← **Head:** `no-python-deploy` --- ### 📝 Commits (10+) - [`75aa483`](https://github.com/google/brotli/commit/75aa48324429bab7fcc79955ea4b3445dd845664) [appveyor] remove 'deploy' stage; only test python 2.7 and 3.6 - [`f54b514`](https://github.com/google/brotli/commit/f54b514dbb96e6f84640cc50287f4af464ac10e2) remove terrify submodule as not needed any more - [`2475c01`](https://github.com/google/brotli/commit/2475c0149ba723c7d6ede1fd2c008b3b6b1bba65) [travis] just test py2.7 and 3.6 on linux; remove extra osx python builds - [`86d4b19`](https://github.com/google/brotli/commit/86d4b19339223fd319cc234b90f88d0e7c394a3e) [setup.py] only rebuild if dependency are newer; fix typo in list of 'depends' - [`384f8b6`](https://github.com/google/brotli/commit/384f8b6443f42ae7b1206802eadcdf312dc0558e) [ci] only run 'python setup.py test' - [`44b5e66`](https://github.com/google/brotli/commit/44b5e66785d9e656e1fab9ddccf0d1045d246e5c) [Makefile] run 'build_ext --inplace' instead of 'develop' as default target - [`4e370f5`](https://github.com/google/brotli/commit/4e370f536bc5669d20bd1f3357c9f8ea46cccd19) [Makefile] add 'develop' target; remove unnecessary 'tests' target - [`0a3c14c`](https://github.com/google/brotli/commit/0a3c14c7c03320ffece4829c383c9915b22fa739) [Makefile] `setup.py test` requires setuptools; run `python -m unittest` - [`47b4aab`](https://github.com/google/brotli/commit/47b4aabac800a0736d55609551aa50adf866ccf7) [python/README.md] add ref to 'develop' target; remove 'tests', just 'make test' - [`44f6d75`](https://github.com/google/brotli/commit/44f6d75004600acb068d7777ffdf97f5692a25b4) [setup.py] import modules as per nicksay's comment ### 📊 Changes **10 files changed** (+60 additions, -361 deletions) <details> <summary>View changed files</summary> 📝 `.gitmodules` (+0 -3) 📝 `.travis.yml` (+16 -17) 📝 `python/Makefile` (+8 -5) 📝 `python/README.md` (+9 -8) 📝 `scripts/.travis.sh` (+1 -19) 📝 `scripts/appveyor.yml` (+8 -50) ➖ `scripts/install_py.ps1` (+0 -177) ➖ `scripts/run_with_compiler.cmd` (+0 -80) ➖ `scripts/terryfy` (+0 -1) 📝 `setup.py` (+18 -1) </details> ### 📄 Description Since this is now done in a separate https://github.com/google/brotli-wheels, it makes sense remove that from the upstream brotli's Travis and Appveyor setup. We still run `python setup.py test` on both 2.7 and 3.6 on linux and windows, as well as on the system's python on macOS. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

claunia added the pull-request label 2026-01-29 20:53:05 +00:00

claunia referenced this issue

2026-01-29 20:55:48 +00:00

[PR #1228] [CLOSED] Bump actions/cache from 4.1.2 to 4.2.0 #1788

claunia referenced this issue

2026-01-29 20:55:48 +00:00

[PR #1228] Bump actions/cache from 4.1.2 to 4.2.0 #1791

Sign in to join this conversation.