fedora-business-cards

#2 Fixups for the Python 3 port
Merged by bex. Opened by pviktori.
pviktori/fedora-business-cards fixups  into  master

Download 2.patch

Change shebang of the convenience script to Python 3

The code is not compatible with Python 2. (Should it have been?)

Simplify status prints

The Python 2 original with trailing comma and explicit flush:

print "Generating front...",
sys.stdout.flush()

would be best translated as:

print("Generating front...", end=" ", flush=True)

(file=sys.stdout is the default)
but I see no reason to print all statuses on the same line, possibly making other messages more unreadable.
So, I changed them to simple print().

Thank you! I have a new PR coming in that adds new generators. If you don't mind looking at it as well that would be awesome. I've had to move from optparse to argparser ...

Pull-Request has been merged by bex

Mention me as @pviktori :)

block blast
Thanks for sharing this simple recipe.
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.