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

Backports to 6x #22359

Merged
merged 4 commits into from Oct 13, 2022
Merged

Backports to 6x #22359

merged 4 commits into from Oct 13, 2022

Conversation

ljacomet
Copy link
Member

@ljacomet ljacomet added this to the 6.9.3 milestone Oct 11, 2022
@ljacomet ljacomet requested a review from a team October 11, 2022 12:46
@ljacomet ljacomet self-assigned this Oct 11, 2022
This was linked to issues Oct 11, 2022
Fix buffer overflow error in KryoBackedDecoder [Backport 6.x] #19523
Closed
Support constraints without version in GMM [Backport 6.9.x] #20189
Closed
jbartok and others added 4 commits October 13, 2022 12:03
@jbartok @ljacomet
Fix buffer overflow error in KryoBackedDecoder
3e38392 
Fixes #19523
@ljacomet
Fix nullability hints
69fa659
@ljacomet
Fix typo in exception
834fbce
@ljacomet
Add support for dependency constraint without version
b95575c 
Prior to this change, a published Gradle Module Metadata that contained
a constraint without a version definition would result in a null
VersionConstraint instead of an empty one.
This caused issue during metadata serialization and potentially
resolution.
The field is not nullable and an empty version constraint is now used as
the default value.

Fixes #20189
@ljacomet ljacomet force-pushed the ljacomet/release6x/backports-6x branch from 0706242 to b95575c Compare October 13, 2022 10:03
@gradle gradle deleted a comment from ljacomet Oct 13, 2022
@bot-gradle
Copy link
Collaborator

OK, I've already triggered a build for you.

@bot-gradle
Copy link
Collaborator

Pre-tested commit build failed.

@ljacomet
Copy link
Member Author

Cannot reproduce test error locally. Let's try once more.

@ljacomet
Copy link
Member Author

@bot-gradle test and merge

@gradle gradle deleted a comment from ljacomet Oct 13, 2022
@gradle gradle deleted a comment from ljacomet Oct 13, 2022
@bot-gradle
Copy link
Collaborator

OK, I've already triggered a build for you.

@bot-gradle bot-gradle merged commit c18e0b2 into release6x Oct 13, 2022
@ljacomet ljacomet deleted the ljacomet/release6x/backports-6x branch October 13, 2022 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eskatos eskatos eskatos approved these changes

Assignees

@ljacomet ljacomet

Labels
None yet
Projects
None yet
Milestone
6.9.3
4 participants
@ljacomet @bot-gradle @eskatos @jbartok