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

Downgrade FreeBSD p5-IO-Tty 1.18->1.17. #92

Merged
merged 1 commit into from
Jan 13, 2024
Merged

Conversation

macdice
Copy link
Collaborator

@macdice macdice commented Jan 13, 2024

There seems to be a problem in 1.18 that breaks PostgreSQL's interactive tests. Until a fixed package rolls out, downgrade to 1.17 if 1.18 is detected.

cpan-authors/IO-Tty#38
https://www.postgresql.org/message-id/flat/757523.1705091568%40sss.pgh.pa.us

@macdice macdice requested a review from anarazel January 13, 2024 09:20
@nbyavuz
Copy link
Collaborator

nbyavuz commented Jan 13, 2024

Hi @macdice,

Thank you for the fix!

According to issue you shared, it looks like that problem won't be fixed at least until version 1.21. Do you think adding 1.19 and 1.20 to grep command is a good idea?

@macdice
Copy link
Collaborator Author

macdice commented Jan 13, 2024

Not sure, but a couple of people (including me just now) have pinged the maintainer, so I don't think they will be another package released that is broken like this. So I think it's OK to just look out for the known broken actually released version, and hard to predict the version that will have the fix.

@macdice macdice requested review from nbyavuz and removed request for anarazel January 13, 2024 09:55
Copy link
Collaborator

@nbyavuz nbyavuz left a comment

Choose a reason for hiding this comment

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

The change looks good and I confirm that FreeBSD passes all tests with that change: https://cirrus-ci.com/task/4835937029455872.

@macdice macdice merged commit 4e1ec44 into main Jan 13, 2024
17 checks passed
@macdice
Copy link
Collaborator Author

macdice commented Jan 13, 2024

Thanks!

@anarazel anarazel deleted the downgrade-p5-io-tty branch January 25, 2024 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants