Overview
Request 539245 superseded
- Created by scarabeus_iv
- In state superseded
- Supersedes 538658
- Superseded by 542080
-
Open review for
repo-checker
-
Open review for
factory-staging
scons is python3 compatible - observed error comes from libserf SConstruct (I assume there will be more such cases)
And libserf issue is resolved here - https://github.com/apache/serf/commit/d4de5a672d8c03b82ba70c1b737926bcf078f761#diff-cc8093822b396591a5530a463c87d048 - will send it asap
Thanks for chasing this down - please submit the fix, then reopen this submission so we can try to restage it (and let's hope we won't see many more of those)
This problem in scons was fixed here: https://github.com/SConsProject/scons/commit/2e0de3c55f22b3eaa7767b69740b898f3d2f46bf
This was included in the scons 3.0.0 submission as scons-3.0.0-support-python-2-prints.patch, and I have previously verified that for the Python 3 case this is effective.
Not exactly - since we no longer execute scons with python 2.7
I am testing this in home:pluskalm:python3 - changes for serf and scons submitted but there is still something missing - so lets wait with submissions
Request History
scarabeus_iv created request
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto added repo-checker as a reviewer
Please review build success
factory-auto accepted review
Check script succeeded
licensedigger accepted review
ok
staging-bot set openSUSE:Factory:Staging:H as a staging project
Being evaluated by staging project "openSUSE:Factory:Staging:H"
staging-bot accepted review
Picked openSUSE:Factory:Staging:H
dimstar_suse accepted review
Removing from openSUSE:Factory:Staging:H, re-evaluation needed
dimstar_suse added factory-staging as a reviewer
Requesting new staging review
jengelh accepted review
ffado is also affected by not being compatible to scons/py3
Not every scons-consumer knows about py3 syntax
libserf: http://svn.apache.org/viewvc?view=revision&revision=1814604
SR#539994