Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 635934 (scons-python3)

Summary: [TRACKER] Packages failing with dev-build/scons and Python 3
Product: Gentoo Linux Reporter: Arfrever Frehtes Taifersar Arahesis <arfrever.fta>
Component: Current packagesAssignee: Michał Górny <mgorny>
Status: CONFIRMED ---    
Severity: normal CC: pacho, perfect007gentleman, polynomial-c, python, qa, tsmksubc
Priority: Normal Keywords: Tracker
Version: unspecified   
Hardware: All   
OS: All   
URL: https://dev.gentoo.org/~mgorny/python-guide/buildsys.html#scons
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on: 610970, 635874, 635912, 635914, 635920, 635940, 635960, 635994, 635998, 636048, 636050, 636092, 636172, 636272, 636312, 636356, 636420, 636424, 658052, 661748, 677446, 677622, 685576, 695010, 704856, 708954, 734586, 735374, 735530, 735532, 735534, 735536, 735538, 735540, 735542, 735544, 735546, 735548, 735550, 735552, 735554, 735556, 735558, 735560    
Bug Blocks:    

Description Arfrever Frehtes Taifersar Arahesis 2017-10-30 20:14:23 UTC
Tracker for packages failing with dev-util/scons and Python 3.

Usually the error message is "SyntaxError: invalid syntax".
In affected packages SConstruct file should be ported to Python 3.
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2017-11-04 13:15:13 UTC
Reassigning to Matthew since he committed that skipping maintainer completely, and how cleaning up the resulting mess is his responsibility.
Comment 2 Mike Gilbert gentoo-dev 2017-11-04 15:55:38 UTC
We may want to mask this version of scons until a solution is implemented.

One possible workaround would be to utilize python-any-r1 in affected ebuilds to force python2.
Comment 3 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2017-11-04 16:00:16 UTC
I wouldn't call that a workaround but a proper solution. As long as SConstruct files are totally random freeform Python scripts, we're dealing with running Python code that can be compatible with some specific set of implementations.
Comment 4 Matthew Thode ( prometheanfire ) archtester Gentoo Infrastructure gentoo-dev Security 2017-11-05 04:39:15 UTC
First, yes I messed up by bumping this package (a major version at that).

That being said, I think the proper workaround would be to add a cap on the scons version pulled in.  The actual error seems to be that these consumers can't handle the new scons version.

Now THAT being said, this would have been better added unkeyworded at least, or possibly even masked, given what upstream said about it's readiness for production use.

At the moment I'm traveling so can't (and probably shouldn't) run around adding caps everywhere, but I would like feedback as to the options presented.  The 'quickest', and possibly correct one would be to p.mask it for now.
Comment 5 Larry the Git Cow gentoo-dev 2017-11-05 15:13:58 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=f5861c803b3e4e7da80369c8026c465d49e61416

commit f5861c803b3e4e7da80369c8026c465d49e61416
Author:     Mike Gilbert <floppym@gentoo.org>
AuthorDate: 2017-11-05 15:12:56 +0000
Commit:     Mike Gilbert <floppym@gentoo.org>
CommitDate: 2017-11-05 15:12:56 +0000

    profiles: mask >=dev-util/scons-3.0.0
    
    Bug: https://bugs.gentoo.org/635934

 profiles/package.mask | 5 +++++
 1 file changed, 5 insertions(+)}
Comment 6 Mike Gilbert gentoo-dev 2017-11-05 15:29:39 UTC
> The actual error seems to be that these consumers can't handle the new scons version.

I had assumed that invoking scons with python2.7 would resolve the errors, but after testing a couple of packages that does not seem to be the case.
Comment 7 Pacho Ramos gentoo-dev 2018-07-24 10:07:54 UTC
it seems we have now the latest ebuild using scons+python3, should all the bugs be reassigned to their relevant maintainers then? Currently they are all assigned to prometheanfire, then, I am not sure if they will get any attention from their maintainers (I noticed it just now when seeing new bug 661748). 

If, on the other hand, that bugs are obsolete, I would simply close then (also this tracker) and probably open a new fresh tracker bug for issues with current scons ebuilds

Thanks :)
Comment 8 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2018-07-24 10:28:04 UTC
Feel free to close the bugs.  However, QA still needs to deliver a proper lecture to prometheanfire why you don't skip maintainers and testing, and break stuff in the process.
Comment 9 Matthew Thode ( prometheanfire ) archtester Gentoo Infrastructure gentoo-dev Security 2018-07-24 15:11:24 UTC
pretty sure I had that lecture
Comment 10 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2018-07-26 07:49:46 UTC
(In reply to Matthew Thode ( prometheanfire ) from comment #9)
> pretty sure I had that lecture

I'm sorry, I must have missed the part where you put any effort into fixing the resulting bugs, or bothered to reply or admit that you've made a mistake.
Comment 11 Matthew Thode ( prometheanfire ) archtester Gentoo Infrastructure gentoo-dev Security 2018-07-26 13:56:41 UTC
sorry you missed the show
Comment 12 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2020-08-02 11:53:11 UTC
I suppose it's time to reuse the tracker to start preparing for py2 removal in scons.
Comment 13 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2020-08-02 15:32:38 UTC
Let's link the porting guide:

https://dev.gentoo.org/~mgorny/python-guide/buildsys.html#scons