Page MenuHomeFreeBSD

lang/python*: Replace DISABLED_EXTENSIONS with Setup.local
ClosedPublic

Authored by dbaio on Jul 7 2021, 2:07 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 14, 3:12 AM
Unknown Object (File)
Thu, Nov 7, 5:49 AM
Unknown Object (File)
Sat, Oct 19, 12:18 AM
Unknown Object (File)
Oct 17 2024, 2:49 AM
Unknown Object (File)
Oct 16 2024, 7:03 AM
Unknown Object (File)
Oct 15 2024, 5:56 PM
Unknown Object (File)
Oct 14 2024, 4:56 PM
Unknown Object (File)
Oct 14 2024, 9:47 AM
Subscribers

Details

Summary
lang/python*: Replace DISABLED_EXTENSIONS with Setup.local

Currently, lang/python38 and lang/python39 don't honour
DISABLED_EXTENSIONS because patch-issue20210 was removed when
lang/python38 was added to the ports tree. patch-issue20210 is
still present on lang/python36 and lang/python37.

Building with poudriere is not affected because builds are executed
in a clean environment.

Setup.local is the more canonical and recommended method for
customising python builds for shared extensions & third party
libraries.

Support for a *disabled* marker in Setup files was introduced in Python
3.7, so backport this fix to it to keep consistency in the ports tree.

PR:		243358 [1]
PR:		243937 [2]
Reported by:	ngie [1]
Reported by:	jcfyecrayz@liamekaens.com [2]
Reported by:	tuxillo (IRC) DPorts
Reviewed_by:    koobs (python, maintainer)
Approved by:    koobs, danilo (python, maintainer)
MFH:            2020Q3 (build bugfix)
Test Plan
  • testport: OK (poudriere, lang/python??, $modules tested)

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dbaio requested review of this revision.Jul 7 2021, 2:07 AM
koobs edited the summary of this revision. (Show Details)
koobs retitled this revision from lang/python*: Fix DISABLED_EXTENSIONS to lang/python*: Replace DISABLED_EXTENSIONS with Setup.local.Jul 7 2021, 2:37 AM
koobs edited the summary of this revision. (Show Details)
koobs edited the test plan for this revision. (Show Details)
koobs added a subscriber: koobs.

Change looks good. Can we confirm successful QA for the three supported modules, by forcing BUILD_DEPENDS on them so it can be tested in poudriere

Change looks good. Can we confirm successful QA for the three supported modules, by forcing BUILD_DEPENDS on them so it can be tested in poudriere

Thanks koobs... And yes, it's all good on my tests.
I'll push it.

koobs edited the summary of this revision. (Show Details)
koobs edited the test plan for this revision. (Show Details)
koobs added a subscriber: danilo.

@danilo 👍 just update the test plan section to mention those combinations that were tested

This revision is now accepted and ready to land.Jul 8 2021, 1:45 AM