Page MenuHomeFreeBSD

lang/python33: Update to 3.3.6
ClosedPublic

Authored by brnrd on Dec 23 2015, 1:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 13, 2:22 AM
Unknown Object (File)
Wed, Mar 13, 2:00 AM
Unknown Object (File)
Feb 10 2024, 3:25 PM
Unknown Object (File)
Feb 7 2024, 1:53 PM
Unknown Object (File)
Jan 8 2024, 2:43 PM
Unknown Object (File)
Jan 8 2024, 2:39 PM
Unknown Object (File)
Jan 8 2024, 2:39 PM
Unknown Object (File)
Jan 8 2024, 2:38 PM
Subscribers

Details

Summary

Proposed commit log

lang/python33: Update to 3.3.6

  - Update to 3.3.6 (released 2014-10-11)
  - Backport issue21356 RAND_egd fix [1]
  - Fix build for openssl -nossl3
  - Remove unixcompiler patch, included upstream
  - Switch regression-test to TEST_TARGET/TEST_ARGS
  - Mark DEPRECATED, EXPIRES Sept 2017 [2]
  
[1] http://bugs.python.org/issue21356
[2] https://www.python.org/dev/peps/pep-0398/#lifespan

This is a bug/security fix update. Changelog:

  https://docs.python.org/3.3/whatsnew/changelog.html#python-3-3-6

PR:		195511
Reviewed by:	koobs (mentor, python), feld, miwi, AMDmi3
Approved by:	koobs (python)
Differential_Revision:	https://reviews.freebsd.org/D4695
Test Plan
  • portlint -AC adds
files/patch-PR195511: patch was not generated using ``make makepatch''

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 1817
Build 1824: arc lint + arc unit

Event Timeline

brnrd retitled this revision from to lang/python33: Update to 3.3.6.
brnrd updated this object.
brnrd edited the test plan for this revision. (Show Details)
brnrd added reviewers: koobs, vsevolod.
brnrd edited the test plan for this revision. (Show Details)
brnrd edited the test plan for this revision. (Show Details)

Add patch for lang/python-doc-html

Add patch for lang/python-doc-html

  • And now actually add it!
lang/python-doc-html/distinfo
33

You have accidentally removed all other docs.

lang/python33/Makefile
120–121

Remove build, its no longer needed in this case.

Add the complete python-doc-html/distinfo

  • Remove redundant build for do-test target
brnrd added inline comments.
lang/python-doc-html/distinfo
32

Forgot to move the complete file back to distinfo, fixed!

lang/python33/Makefile
120–121

Done!

miwi added a reviewer: miwi.
miwi edited edge metadata.
miwi removed a subscriber: miwi.
This revision is now accepted and ready to land.Dec 24 2015, 10:25 AM
koobs added a subscriber: Python.
koobs requested changes to this revision.Dec 24 2015, 11:03 AM
koobs edited edge metadata.
  • Add breadcrumbs to the top of patch (ideally that from the top of the raw upstream commit that went into this branch) including link to upstream issue id
  • Missing change entry for reason/rationale for patch (unixcompiler) removal. Add this.
  • Consider marking DEPRECATED, with EXPIRES:

After that, security updates (source only) will be released until 5 years after the release of 3.3 final, which will be September 2017. [1]

[1] https://www.python.org/dev/peps/pep-0398/#lifespan

This revision now requires changes to proceed.Dec 24 2015, 11:03 AM
AMDmi3 requested changes to this revision.Dec 24 2015, 11:13 AM
AMDmi3 added a reviewer: AMDmi3.
AMDmi3 added a subscriber: AMDmi3.
AMDmi3 added inline comments.
lang/python33/Makefile
120–121

This can be converted to new test framework. Remove the target, and add

TEST_TARGET=buildbottest

Note that jobs don't work here anyway, because MAKE_JOBS_UNSAFE sets MAKE_JOBS_NUMBER to 1. I don't know an easy way to fix it without duplicating bsd.port.mk logic, so let's just drop jobs here.

brnrd removed a reviewer: AMDmi3.
brnrd removed a subscriber: AMDmi3.
brnrd added a reviewer: AMDmi3.
brnrd marked 2 inline comments as done.
brnrd updated this object.
brnrd added a subscriber: AMDmi3.

Improve commit message

  • Add breadcrumbs
  • Mention backport of EGD fix
  • Add deprecation date
  • unixcompiler patch removal
lang/python33/Makefile
120–121

Id prefer to keep the MAKE_JOBS_NUMBER here, regardless of its value. Explicit > Implicit and when this port gets fixed for concurrent make jobs, it will just work without requiring changes

lang/python33/Makefile
120–121

Oh, right. Then it should go to TEST_ARGS:

TEST_ARGS= TESTOPTS=-j${MAKE_JOBS_NUMBER}

koobs requested changes to this revision.Dec 24 2015, 11:29 AM
koobs edited edge metadata.
  • Use TEST_TARGET and TEST_ARGS
lang/python33/Makefile
15

No longer receives bug fixes, only security updates. Please update to Python 3.5

This revision now requires changes to proceed.Dec 24 2015, 11:29 AM
brnrd edited edge metadata.

Refactor do-test target to TEST_TARGET/TEST_ARGS

brnrd edited edge metadata.

Rename EGD/SSLv3 patch to match the 3.3 PR

brnrd added inline comments.
lang/python33/Makefile
120–121

Adjusted as suggested to
TEST_TARGET= buildbottest
TEST_ARGS= TESTOPTS=-j${MAKE_JOBS_NUMBER}

AMDmi3 edited edge metadata.

Test-related part LGTM

brnrd edited the test plan for this revision. (Show Details)
koobs updated this object.
brnrd edited edge metadata.
brnrd marked 2 inline comments as done.

Update DEPRECATED description

koobs edited edge metadata.
This revision is now accepted and ready to land.Dec 24 2015, 12:00 PM
koobs edited edge metadata.

Forgot to remove _ from Differential_Revision :'(