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

Remove all remaining bottle :unneeded #79448

Closed
wants to merge 304 commits into from
Closed

Remove all remaining bottle :unneeded #79448

wants to merge 304 commits into from

Conversation

MikeMcQuaid
Copy link
Member

Fixes #75943

@nandahkrishna nandahkrishna added bottle-up-everything Make every formula bottled CI-force-arm [DEPRECATED] Don't pass --skip-unbottled-arm to brew test-bot. and removed CI-force-arm [DEPRECATED] Don't pass --skip-unbottled-arm to brew test-bot. labels Jun 16, 2021
@nandahkrishna
Copy link
Member

nandahkrishna commented Jun 16, 2021

@MikeMcQuaid Just to confirm, do you want these to be bottled separately for ARM (and not in this PR)? Some of them may not work on ARM because they depend on, e.g., openjdk@8. (And none will be bottled for ARM without the CI-force-arm label.)

@MikeMcQuaid MikeMcQuaid added the CI-force-arm [DEPRECATED] Don't pass --skip-unbottled-arm to brew test-bot. label Jun 16, 2021
@MikeMcQuaid
Copy link
Member Author

@nandahkrishna I want these all bottled for ARM when possible. I plan on removing the PRs from here that refuse to build.

@nandahkrishna
Copy link
Member

Alright, I'll have to rerun CI since the label was added just after it was supposed to be detected 😅.

@cho-m
Copy link
Member

cho-m commented Jun 19, 2021

Top 20 Catalina bottles by size. May be better to have separate PR for some:

du -sh *catalina* | sort -hr | head -20
1.7G	kettle--9.1.0.0-324_1.catalina.bottle.tar.gz
1.5G	nifi--1.13.2.catalina.bottle.tar.gz
1.1G	prestosql--344.catalina.bottle.tar.gz
624M	apache-ctakes--4.0.0_1.catalina.bottle.tar.gz
474M	hadoop--3.3.0.catalina.bottle.tar.gz
466M	hapi-fhir-cli--5.4.0.catalina.bottle.tar.gz
364M	apache-drill--1.18.0.catalina.bottle.tar.gz
263M	sonarqube--8.9.0.43852.catalina.bottle.tar.gz
218M	apache-spark--3.1.2.catalina.bottle.tar.gz
185M	pig--0.17.0_1.catalina.bottle.tar.gz
182M	solr--8.8.2.catalina.bottle.tar.gz
174M	stanford-parser--4.2.0.catalina.bottle.tar.gz
172M	stanford-ner--4.2.0.catalina.bottle.tar.gz
158M	[email protected]_1.catalina.bottle.tar.gz
153M	fits--1.5.0.catalina.bottle.tar.gz
151M	redpen--1.10.4_1.catalina.bottle.tar.gz
140M	mahout--0.13.0_1.catalina.bottle.tar.gz
139M	titan-server--1.catalina.bottle.tar.gz
133M	tika--1.26.catalina.bottle.tar.gz
126M	grails--4.0.10.catalina.bottle.tar.gz

@MikeMcQuaid
Copy link
Member Author

Thanks for the work here all! Will try to split this out at some point (but feel free to beat me to it).

@MikeMcQuaid
Copy link
Member Author

Closing in favour of separate PRs.

@chenrui333 chenrui333 removed the in progress Stale bot should stay away label Jun 26, 2021
@github-actions github-actions bot added the outdated PR was locked due to age label Jul 27, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bottle-up-everything Make every formula bottled CI-force-arm [DEPRECATED] Don't pass --skip-unbottled-arm to brew test-bot. outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bottle all bottle :unneeded
6 participants