Skip to content

snapcraft: switch to openssl@3#134749

Merged
BrewTestBot merged 3 commits into
Homebrew:masterfrom
carlocab:snapcraft-openssl3
Jul 2, 2023
Merged

snapcraft: switch to openssl@3#134749
BrewTestBot merged 3 commits into
Homebrew:masterfrom
carlocab:snapcraft-openssl3

Conversation

@carlocab

Copy link
Copy Markdown
Member

See #134251. This has linkage with openssl@1.1.

@carlocab carlocab added openssl-3-migration Related to switching to an OpenSSL 3 dependency staging-branch-pr This pull request targets a *-staging branch. labels Jun 23, 2023
@github-actions github-actions Bot added python Python use is a significant feature of the PR or issue rust Rust use is a significant feature of the PR or issue labels Jun 23, 2023
@carlocab carlocab force-pushed the snapcraft-openssl3 branch 2 times, most recently from d578a26 to 95a483b Compare June 23, 2023 16:18
Comment thread Formula/snapcraft.rb Outdated
Comment on lines 95 to 99

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't work. 😅

@cho-m cho-m Jun 23, 2023

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like --ignore-installed prevents pip from removing the newer version, so it isn't possible to downgrade setuptools with the DSL.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Worked for pip at #134447 though.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then might be specific to setuptools or some undefined behavior.

In case of pip, I locally tried installing older pip in venv and the output version numbers are all over the place (pip install shows wheel is for 22.3.1 but then claims to have installed 23.0.1. Afterward, pip list still shows 23.0.1 but importing pip shows 22.3.1):

pip list
Package    Version
---------- -------
pip        23.0.1
setuptools 67.6.1pip install -vvv --no-deps --no-binary :all:  --ignore-installed ./pip-22.3.1/
Using pip 23.0.1 from /private/tmp/test-venv/venv/lib/python3.11/site-packages/pip (python 3.11)
...
  Created wheel for pip: filename=pip-22.3.1-py3-none-any.whl size=2051532 sha256=fdc1a1a2e7a8d5285b5d60bc1a1e6344845e395df4d70049eb69dcf21f9fe7e6
  Stored in directory: /Users/cho-m/Library/Caches/pip/wheels/c0/bd/3d/b1f9df2824bf1974a39b5c47d6b437cb4fd7628bcb5f486efa
Successfully built pip
Installing collected packages: pip

  changing mode of /private/tmp/test-venv/venv/bin/pip to 755
  changing mode of /private/tmp/test-venv/venv/bin/pip3 to 755
  changing mode of /private/tmp/test-venv/venv/bin/pip3.11 to 755
Successfully installed pip-23.0.1
Remote version of pip: 23.1.2
Local version of pip:  23.0.1
Was pip installed by pip? Truepip list
Package    Version
---------- -------
pip        23.0.1
setuptools 67.6.1python3 -c 'import pip; print(pip.__version__)'
22.3.1

Only difference for setuptools is imported version is also unchanged:

pip install -vvv --no-deps --no-binary :all:  --ignore-installed ./setuptools-65.7.0/
Using pip 22.3.1 from /private/tmp/test-venv/venv/lib/python3.11/site-packages/pip (python 3.11)
...
  Created wheel for setuptools: filename=setuptools-65.7.0-py3-none-any.whl size=1234046 sha256=0f45194ed9132662327a88c9484237cfddbf7e852866a9d0479f99cc41cb79b2
  Stored in directory: /Users/cho-m/Library/Caches/pip/wheels/04/97/31/286624957c941070bd12d9bdf8c69f98b7306bd4aadbdf9067
Successfully built setuptools
Installing collected packages: setuptools

Successfully installed setuptools-67.6.1pip list
Package    Version
---------- -------
pip        23.0.1
setuptools 67.6.1python3 -c 'import setuptools; print(setuptools.__version__)'
67.6.1

@carlocab carlocab added help wanted Task(s) needing PRs from the community or maintainers test failure CI fails while running the test-do block CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. labels Jun 23, 2023
@p-linnane p-linnane force-pushed the snapcraft-openssl3 branch from 95a483b to b097e4e Compare June 26, 2023 23:51
@p-linnane p-linnane changed the base branch from openssl-migration-staging to master June 26, 2023 23:51
@p-linnane p-linnane removed the staging-branch-pr This pull request targets a *-staging branch. label Jun 26, 2023
@cho-m cho-m force-pushed the snapcraft-openssl3 branch from b097e4e to 369a8a6 Compare July 2, 2023 03:21
@github-actions github-actions Bot added the autosquash Automatically squash pull request commits according to Homebrew style. label Jul 2, 2023
@carlocab carlocab added ready to merge PR can be merged once CI is green and removed help wanted Task(s) needing PRs from the community or maintainers test failure CI fails while running the test-do block autosquash Automatically squash pull request commits according to Homebrew style. labels Jul 2, 2023
@github-actions

github-actions Bot commented Jul 2, 2023

Copy link
Copy Markdown
Contributor

🤖 An automated task has requested bottles to be published to this PR.

@github-actions github-actions Bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label Jul 2, 2023
@BrewTestBot BrewTestBot enabled auto-merge July 2, 2023 15:06
@BrewTestBot BrewTestBot added this pull request to the merge queue Jul 2, 2023
Merged via the queue into Homebrew:master with commit 79232a7 Jul 2, 2023
@carlocab carlocab deleted the snapcraft-openssl3 branch July 2, 2023 15:24
@github-actions github-actions Bot added the outdated PR was locked due to age label Aug 2, 2023
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. openssl-3-migration Related to switching to an OpenSSL 3 dependency outdated PR was locked due to age python Python use is a significant feature of the PR or issue ready to merge PR can be merged once CI is green rust Rust use is a significant feature of the PR or issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants