Skip to content

web: remove checks for SSL verification support#34307

Merged
alalazo merged 1 commit intodevelopfrom
remove-no-ssl-verification-check
Dec 5, 2022
Merged

web: remove checks for SSL verification support#34307
alalazo merged 1 commit intodevelopfrom
remove-no-ssl-verification-check

Conversation

@tgamblin
Copy link
Copy Markdown
Member

@tgamblin tgamblin commented Dec 5, 2022

We no longer support Python <3.6, so we don't need to check whether Python supports SSL verification in spack.util.web.

  • Remove a bunch of logic we needed to appease Python 2

@spackbot-app spackbot-app bot added core PR affects Spack core functionality fetching utilities labels Dec 5, 2022
@tgamblin tgamblin requested review from alalazo and scheibelp December 5, 2022 00:17
@tgamblin tgamblin changed the title web: remove SSL verification checks web: remove checks for SSL verification support Dec 5, 2022
@tgamblin tgamblin force-pushed the remove-no-ssl-verification-check branch from 1acde76 to 99bf887 Compare December 5, 2022 00:30
We no longer support Python <3.6, so we don't need to check whether Python supports SSL
verification in `spack.util.web`.

- [x] Remove a bunch of logic we needed to appease Python 2
@tgamblin tgamblin force-pushed the remove-no-ssl-verification-check branch from 99bf887 to cae32c1 Compare December 5, 2022 00:31
@tgamblin tgamblin requested a review from adamjstewart December 5, 2022 05:59
@alalazo alalazo merged commit 94dc86e into develop Dec 5, 2022
@alalazo alalazo deleted the remove-no-ssl-verification-check branch December 5, 2022 07:46
luke-dt pushed a commit to dantaslab/spack that referenced this pull request Dec 5, 2022
We no longer support Python <3.6, so we don't need to check whether Python supports SSL
verification in `spack.util.web`.

- [x] Remove a bunch of logic we needed to appease Python 2
amd-toolchain-support pushed a commit to amd-toolchain-support/spack that referenced this pull request Feb 16, 2023
We no longer support Python <3.6, so we don't need to check whether Python supports SSL
verification in `spack.util.web`.

- [x] Remove a bunch of logic we needed to appease Python 2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core PR affects Spack core functionality fetching utilities

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants