Skip to content

Set minimum bash version to 4.4.0#3873

Merged
aldy505 merged 2 commits intogetsentry:masterfrom
aminvakil:bash4.4
Aug 14, 2025
Merged

Set minimum bash version to 4.4.0#3873
aldy505 merged 2 commits intogetsentry:masterfrom
aminvakil:bash4.4

Conversation

@aminvakil
Copy link
Collaborator

Fixes #3860 by failing sooner :)

There is possibly an easy fix for #3860, but honestly bash 4.2 is pretty ancient, and I think there will be other problems with that aside from #3860 problem.

Here is bash versions on debian distributions:

Package: bash on amd64 -- jessie: 4.3-11+deb8u1; jessie-security: 4.3-11+deb8u2; stretch: 4.4-5; buster: 5.0-4; bullseye: 5.1-2+deb11u1; bookworm: 5.2.15-2+b8; bookworm-proposed-updates: 5.2.15-2+b9; trixie: 5.2.37-2+b5; forky: 5.2.37-2+b5; sid: 5.2.37-2+b5; experimental: 5.3~rc2-1

Debian 9 stretch has been released 2020 and it uses bash >= 4.4.
RHEL 8 has been released 2018 and it uses bash >= 4.4.
Ubuntu 18 Bionic has been released 2018 and it uses bash >= 4.4.

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

@aldy505 aldy505 enabled auto-merge (squash) August 14, 2025 11:31
@aminvakil
Copy link
Collaborator Author

So that's why I couldn't push :) you beat me to merging new commits 😉

@aldy505 aldy505 merged commit 0f606d2 into getsentry:master Aug 14, 2025
9 checks passed
@aminvakil aminvakil deleted the bash4.4 branch August 14, 2025 11:46
@github-actions github-actions bot locked and limited conversation to collaborators Aug 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

options[@]: unbound variable error in install script

2 participants