Overview

Request 853664 accepted

- x11vnc wrapper script: converted to python3 (boo#1179592)

Loading...

Ismail Dönmez's avatar

@sndirsch Your code-changes are no-op, only shebang change is enough.


Stefan Dirsch's avatar
author source maintainer

Sorry, I don't understand. What's wrong with my submitrequest?


Ismail Dönmez's avatar
-    print(', '.join(ignored_arguments))
+    print((', '.join(ignored_arguments)))
[...]
-        print('Port %d is already used'%port)
+        print(('Port %d is already used'%port))

Both parts, before-and-after works the same on python2 and python3, you only need

-#!/usr/bin/env python
+#!/usr/bin/python3

part.


Stefan Dirsch's avatar
author source maintainer

Honestly I didn't understand why 2to3 added the brackets. It looked weird to me, but I'm not a Python expert.

Request History
Stefan Dirsch's avatar

sndirsch created request

- x11vnc wrapper script: converted to python3 (boo#1179592)


Factory Auto's avatar

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

Please review sources


Factory Auto's avatar

factory-auto accepted review

Check script succeeded


Saul Goodman's avatar

licensedigger accepted review

ok


Ismail Dönmez's avatar

namtrac accepted review


Dominique Leuenberger's avatar

dimstar_suse set openSUSE:Factory:Staging:E as a staging project

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


Dominique Leuenberger's avatar

dimstar_suse accepted review

Picked "openSUSE:Factory:Staging:E"


Dominique Leuenberger's avatar

dimstar_suse accepted review

Staging Project openSUSE:Factory:Staging:E got accepted.


Dominique Leuenberger's avatar

dimstar_suse approved review

Staging Project openSUSE:Factory:Staging:E got accepted.


Dominique Leuenberger's avatar

dimstar_suse accepted request

Staging Project openSUSE:Factory:Staging:E got accepted.

openSUSE Build Service is sponsored by