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

Test with libyaml 0.2.4 #404

Closed
wants to merge 1 commit into from
Closed

Test with libyaml 0.2.4 #404

wants to merge 1 commit into from

Conversation

perlpunk
Copy link
Member

@perlpunk perlpunk commented May 20, 2020

libyaml 0.2.2 tests are currently failing:
https://travis-ci.org/github/yaml/pyyaml/jobs/689453153#L1145

Maybe because the wrong testsuite commit gets checked out.

Edit: .appveyor.yml should also be updated

Edit: I fixed the libyaml tests with this: yaml/libyaml@a718a89

If you plan a 5.3.2 bugfix release, this PR should be merged after that.

@bsolomon1124
Copy link
Contributor

bsolomon1124 commented May 23, 2020

@perlpunk for 5.3.1, was 0.2.2 the libyaml version tested against prior to that release or was it an earlier libyaml version?

Secondly, I'm curious if PyYAML has any sort of policy for when/under what circumstances/how often it bumps the libyaml version against which it tests? Asking partly for #407.

@perlpunk
Copy link
Member Author

or 5.3.1, was 0.2.2 the libyaml version tested against prior to that release or was it an earlier libyaml version?

5.3.1 was tested against libyaml 0.2.2.
But there was a bug in the libyaml testsuite code, which is very complicated, which resulted in checking out the wrong version of the test data, that's why we got the errors in travis.
It's fixed now, but I hope we can get rid of that fragile code at some point: yaml/libyaml#171

Secondly, I'm curious if PyYAML has any sort of policy for when/under what circumstances/how often it bumps the libyaml version against which it tests?

In general, I would say, bugfix releases (like 5.3.2) should always be built with the same libyaml version.
Otherwise, the libyaml version should be updated whenever possible, I would say, but @ingydotnet is the one who decides what gets in a release.
The goal is also to try and keep libyaml & PyYAML behaviour in sync, so if a bugfix happens in libyaml that allows parsing something which was not accepted before, that bug should also be fixed in PyYAML.

@bsolomon1124
Copy link
Contributor

If you plan a 5.3.2 bugfix release, this PR should be merged after that.

Okay, so if I'm reading this correctly, the plan would be for 5.3.2 to keep things at 0.2.2 libyaml upstream, but possibly to upgrade (for both tests and potentially wheels) in some release after that.

For #407, I have it at 0.2.4 for now, but that can easily be changed to 0.2.2.

And the last thing I'd note is, if the libyaml version is going to be referenced multiple places here, it might make sense to keep it in a single place (maybe a LIBYAML_VERSION file or something). I'll resist the the urge to suggest adding libyaml as a Git submodule 😉

@perlpunk
Copy link
Member Author

libyaml 0.2.5 is in the 5.4 release, closing

@perlpunk perlpunk closed this Jan 29, 2021
@perlpunk perlpunk deleted the perlpunk/libyaml-travis branch September 23, 2021 23:44
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