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

Python bump to 3.8.8 #19273

Closed
wants to merge 3 commits into from
Closed

Python bump to 3.8.8 #19273

wants to merge 3 commits into from

Conversation

fuzzard
Copy link
Contributor

@fuzzard fuzzard commented Feb 24, 2021

Description

Bump python and patches to 3.8.8 for tools/depends platforms (Android/Apple).
I guess this may be considered a partial backport of #19246 , but only the package bumps for python, no other changes are backported.

Motivation and Context

There are a couple of security fixes. Release notes with CVE numbers can be seen via the below link

https://www.python.org/downloads/release/python-388/

How Has This Been Tested?

Runtime tested ios/osx

Screenshots (if appropriate):

Types of change

  • Bug fix (non-breaking change which fixes an issue)
  • Clean up (non-breaking change which removes non-working, unmaintained functionality)
  • Improvement (non-breaking change which improves existing functionality)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that will cause existing functionality to change)
  • Cosmetic change (non-breaking change that doesn't touch code)
  • None of the above (please explain below)

Checklist:

  • My code follows the Code Guidelines of this project
  • My change requires a change to the documentation, either Doxygen or wiki
  • I have updated the documentation accordingly
  • [] I have read the Contributing document
  • I have added tests to cover my change
  • All new and existing tests passed

@lrusak
Copy link
Contributor

lrusak commented Feb 24, 2021

I don't know if we really have a policy about bumping packages in a release. This is likely fine but worth a discussion about the merits and risks involved. Likely it will be @DaveTBlake's final word. I say this because I would also like to bump libdav1d but am wondering if it's allowed as it may not directly be a "bug fix" which would be the same as this PR.

@fuzzard
Copy link
Contributor Author

fuzzard commented Feb 24, 2021

Definitely understand where you are coming from. The one thing i think thats different in the 2 scenario's, this update is specifically to resolve 2 known security CVE's. yes theres other stuff in the 3.8.5-8 changes, but the intent is to fix the security issues.

An option could be to backport the actual fixes to 3.8.5, so if thats the way we want to go i could do that, but i think the more maintainable approach for us is to do the bringup to the official release with the fixes.

Either way, im not fussed. If its not desired, can be closed.

@lrusak
Copy link
Contributor

lrusak commented Feb 24, 2021

Either way, im not fussed. If its not desired, can be closed.

Apologies if my previous comment's tone wasn't great. I think this PR is a good thing and I would like to see it included in v19.

I'm not sure if any long term team members can chime in about previous release cycles and what may have been bumped (or not).

@DaveTBlake
Copy link
Member

I'm not sure if any long term team members can chime in about previous release cycles and what may have been bumped (or not).

Yes that is what I would like to know. My feeling is we generally don't do it, but possibly just because we are busy moving on, and no technical reason.

This resolves 2 known security CVEs which is reason to do it in v19.1, but are there any downsides to such a bump?

@phunkyfish
Copy link
Contributor

I think bumping the micro version of a dep is ok if fixing security issues.

@wsnipex
Copy link
Member

wsnipex commented Feb 24, 2021

Security fixes must be done. We can't stay old libraries with know security issues forever.
I'd still recommend this goes into master first to get some testing.

@fuzzard
Copy link
Contributor Author

fuzzard commented Feb 24, 2021

@Paxxi If you find time to bundle up some 3.8.8 binaries for windows, happy to add to this. No rush though.

@DaveTBlake
Copy link
Member

I'd still recommend this goes into master first to get some testing.

Yes, master first and get tested in good time for 19.1

@fuzzard fuzzard changed the title [tools/depends] bump Python to 3.8.8 Python bump to 3.8.8 Mar 2, 2021
@phunkyfish
Copy link
Contributor

jenkins build this please

@DaveTBlake
Copy link
Member

Consensus is we want security updates backported, but not looking like this will make v19.1, since equivalent change has not been merged into master yet. @fuzzard what are your thoughts?

@DaveTBlake DaveTBlake modified the milestones: Matrix 19.1, Matrix 19.2 May 6, 2021
@DaveTBlake
Copy link
Member

Bumped to 19.2 as equivalent change has not been merged into master yet.

@thezoggy
Copy link

@fuzzard
Copy link
Contributor Author

fuzzard commented Jul 11, 2021

Because 3.8.8 was the most current at the time of this PR

@ksooo ksooo modified the milestones: Matrix 19.2, Matrix 19.3 Sep 26, 2021
@fuzzard fuzzard removed this from the Matrix 19.3 milestone Oct 19, 2021
@fuzzard fuzzard added PR Cleanup: Abandoned PR closed as part of cleanup. Appears to be abandoned by author, may be obsolete. and removed v19 Matrix labels Oct 19, 2021
@fuzzard fuzzard closed this Oct 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants