Summary: | devel/scons: Update to 3.1.1 and relax python version | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Terry Kennedy <terry-freebsd> | ||||||
Component: | Individual Port(s) | Assignee: | Po-Chuan Hsieh <sunpoet> | ||||||
Status: | Closed DUPLICATE | ||||||||
Severity: | Affects Only Me | CC: | python, terry-freebsd | ||||||
Priority: | --- | Keywords: | needs-qa | ||||||
Version: | Latest | Flags: | bugzilla:
maintainer-feedback?
(sunpoet) |
||||||
Hardware: | Any | ||||||||
OS: | Any | ||||||||
URL: | https://scons.org/tag/releases.html | ||||||||
See Also: | https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=241463 | ||||||||
Attachments: |
|
Description
Terry Kennedy
2019-08-12 02:05:38 UTC
Thank you for the report and patch Terry. Could you update the attachment to be in the form of a unified diff please (svn diff or diff -u) @Sunpoet I'd also recommend splitting out the relax Python version restriction, as SCons has supported 2 and 3 since 3.0.1 (and earlier), so that that change can be MFH'd "This release supports Python versions 2.7.* as well as 3.5+." This update actually supersedes bug 237122 (update to 3.0.5) I'll leave it up to the maintainer to determine whether this will stay open, and to deal with the update to 3.1.1 after bug 237122 is closed, or whether they will update bug 237122 to 3.1.1 and close this issue as a duplicate of it Created attachment 206460 [details] Patch re-done as unified diff Here's the unified diff version. Also, sorry about mis-remembering bug 237122 as bug 327122. We're a long way off from another 90,000 bug reports, hopefully. 8-} (In reply to Terry Kennedy from comment #0) The scons update itself is not a problem. The real problem is to relax USES=python after the default version changed from 2.7 to 3.6. Most of its consumers are not ready for Python 3. (In reply to Kubilay Kocak from comment #1) I would rather follow up bug #237122 as the patch here is incomplete. (In reply to Sunpoet Po-Chuan Hsieh from comment #3) Yes, I think the change to a Python 3 default was a little premature. What is incomplete about this patch? I have used it (patch < thisfile; make makesum; make) on several systems. *** This bug has been marked as a duplicate of bug 237122 *** (In reply to Terry Kennedy from comment #4) Sorry, I did not say it clearly. The scons patch itself is fine. Unfortunately, the complete patch should contain fixes for more than half of its dependent ports which still stay with python 2 because scons would use python 3.x by default (USES=python). (In reply to Sunpoet Po-Chuan Hsieh from comment #6) Now I understand. I only have a small number of ports on my test system and I had previously checked for "USES= python:2.7" among them, and the only ports that had that did not depend on scons. |