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.

Sign up for GitHub

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

Jump to bottom

transcrypt 2.2.1 #123026

Closed
wants to merge 2 commits into from
Closed

transcrypt 2.2.1 #123026

wants to merge 2 commits into from

Conversation

p-linnane
Copy link
Member

Created with brew bump-formula-pr.

@BrewTestBot BrewTestBot added the bump-formula-pr PR was created using `brew bump-formula-pr` label Feb 11, 2023
@Moisan
Copy link
Member

Moisan commented Feb 11, 2023

Seems like we also need vim on linux. I took the liberty to make the change directly.

@chenrui333
Copy link
Member

xxd is only needed when using openssl@3, but I did not see in the list?

@p-linnane
Copy link
Member Author

xxd is only needed when using openssl@3, but I did not see in the list?

From the release notes:

This release fixes a bug introduced in version 2.2.0 that made transcrypt incompatible with the LibreSSL version 3 and later, which particularly affected users on MacOS 13 "Ventura" which shipped with the LibreSSL project's binary version 3.3.6.

So it looks like it's trying to use the system LibreSSL?

@p-linnane p-linnane added the ready to merge PR can be merged once CI is green label Feb 13, 2023
@BrewTestBot
Copy link
Member

🤖 A scheduled task has triggered a merge.

@p-linnane p-linnane deleted the bump-transcrypt-2.2.1 branch February 13, 2023 20:46
@github-actions github-actions bot added the outdated PR was locked due to age label Mar 16, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@branchvincent branchvincent branchvincent left review comments

@cho-m cho-m cho-m left review comments

@iMichka iMichka iMichka approved these changes

Assignees
No one assigned
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` outdated PR was locked due to age ready to merge PR can be merged once CI is green
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@p-linnane @Moisan @chenrui333 @BrewTestBot @iMichka @branchvincent @cho-m