launchpad-reviewers team mailing list archive

[Mitch Burton <mp+427861@xxxxxxxxxxxxxxxxxx>]

Mitch Burton has proposed merging lp:~mitchburton/python-oops-datedir-repo/python-oops-datedir-repo into lp:python-oops-datedir-repo.

Commit message:
Use python3-style print in bsondump.py

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~mitchburton/python-oops-datedir-repo/python-oops-datedir-repo/+merge/427861

Minor fix prompted by issue during `dpkg --configure` of python3-oops-datedir-repo package:

Setting up python3-oops-datedir-repo (0.0.24-0ubuntu2) ...
  File "/usr/lib/python3/dist-packages/oops_datedir_repo/bsondump.py", line 37
    print __doc__
    ^^^^^^^^^^^^^
SyntaxError: Missing parentheses in call to 'print'. Did you mean print(...)?
dpkg: error processing package python3-oops-datedir-repo (--configure):
 installed python3-oops-datedir-repo package post-installation script subprocess returned error exit status 1
Errors were encountered while processing:
 python3-oops-datedir-repo

-- 
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~mitchburton/python-oops-datedir-repo/python-oops-datedir-repo into lp:python-oops-datedir-repo.

=== modified file 'oops_datedir_repo/bsondump.py'
--- oops_datedir_repo/bsondump.py	2018-03-12 12:06:11 +0000
+++ oops_datedir_repo/bsondump.py	2022-08-04 23:13:02 +0000
@@ -34,7 +34,7 @@
     if argv is None:
         argv = sys.argv
     if len(argv) != 2:
-        print __doc__
+        print(__doc__)
         sys.exit(1)
     # I'd like to use json here, but not everything serializable in bson is
     # easily representable in json - even before getting in to the weird parts,