Please login to access the resource
Overview

Request 539245 superseded

No description set
Loading...

Dominique Leuenberger's avatar

ffado is also affected by not being compatible to scons/py3


Dominique Leuenberger's avatar

Not every scons-consumer knows about py3 syntax



Martin Pluskal's avatar

scons is python3 compatible - observed error comes from libserf SConstruct (I assume there will be more such cases)


Martin Pluskal's avatar

And libserf issue is resolved here - https://github.com/apache/serf/commit/d4de5a672d8c03b82ba70c1b737926bcf078f761#diff-cc8093822b396591a5530a463c87d048 - will send it asap


Dominique Leuenberger's avatar

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)


Andreas Stieger's avatar

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.


Dominique Leuenberger's avatar

Not exactly - since we no longer execute scons with python 2.7


Martin Pluskal's avatar

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
Tomáš Chvátal's avatar

scarabeus_iv created request


Factory Auto's avatar

factory-auto added opensuse-review-team as a reviewer

Please review sources


Factory Auto's avatar

factory-auto added repo-checker as a reviewer

Please review build success


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


Saul Goodman's avatar

licensedigger accepted review

ok


Staging Bot's avatar

staging-bot set openSUSE:Factory:Staging:H as a staging project

Being evaluated by staging project "openSUSE:Factory:Staging:H"


Staging Bot's avatar

staging-bot accepted review

Picked openSUSE:Factory:Staging:H


Dominique Leuenberger's avatar

dimstar_suse accepted review

Removing from openSUSE:Factory:Staging:H, re-evaluation needed


Dominique Leuenberger's avatar

dimstar_suse added factory-staging as a reviewer

Requesting new staging review


Jan Engelhardt's avatar

jengelh accepted review


openSUSE Build Service is sponsored by