"noise" from python

[afgcons]

txt2pdf --title DocList_Inventory --page-numbers --font-size 12 -l -o /tmp/ScratchPad/doclist-inventory.pdf /dev/stdin

/usr/lib/python3/dist-packages/reportlab/pdfbase/pdfdoc.py:596: SyntaxWarning: "is" with a literal. Did you mean "=="?
if enc is 'auto':
/usr/lib/python3/dist-packages/reportlab/pdfbase/pdfdoc.py:615: SyntaxWarning: "is" with a literal. Did you mean "=="?
if enc is 'auto':
Python 3.8.0 (default, Dec 9 2021, 17:53:27)
[GCC 8.4.0] on linux
Writing '/dev/stdin' with 101 characters per line and 33 lines per page...
PDF document: 11 pages
doclist-inventory.pdf

• '/dev/stdin' is read, not written
• --title doesn't work

Please investigate, I like your txt2pdf. Thanks !

[clach04]

Thanks for spotting and reporting this @afgcons - the path shows this is coming from Reportlab, not txt2pdf directly.

Are you in a position to report this to the Reportlab project? https://pypi.org/project/reportlab/

[afgcons]

You report it, please! Your informed wording gets action.

Is --title not working related?

Thanks !

[clach04]

@afgcons I appreciate the kind words and confidence you have given, it's misplaced but appreciated 😝 I'll add it to my TODO list but I can't commit a time line due to some other commitments, and of course I'm not sure on time line once reported upstream.

The other items probably are not related to Python 3.11 (or Reportlab).

1. Based on the output you shared the "stdin" text does look like a bug in txt2pdf (not looked but hopefully a one-liner fix). This needs a new ticket
2. I've not used the title parameter, hilariously I have a TODO note in my tool that uses txt2pdf to use it 🤣 so this looks like another separate bug that needs a ticket.

If you can create those tickets that would be really helpful as I'm rather slammed at the moment. None of these seem difficult to fix (other than lack of time).

[clach04]

For Reportlab bug, I just checked out the latest code and problem appears to be resolved:

python -m pip install pillow chardet
#...Successfully installed chardet-5.2.0 pillow-10.2.0

hg clone https://hg.reportlab.com/hg-public/reportlab/
cd reportlab
cd src
python -m reportlab.pdfbase.pdfdoc

Looks like this was addressed in changeset: 4506:147dce84e51e

changeset:   4506:147dce84e51e
user:        robin <robin@reportlab.com>
date:        Tue May 21 16:34:07 2019 +0100
summary:     pull request #58 from Marius Gedminas

diff -r aa53c94c0a55 -r 147dce84e51e src/reportlab/pdfbase/pdfdoc.py
--- a/src/reportlab/pdfbase/pdfdoc.py   Tue May 21 16:32:31 2019 +0100
+++ b/src/reportlab/pdfbase/pdfdoc.py   Tue May 21 16:34:07 2019 +0100
@@ -593,7 +593,7 @@
		 s = self.s
		 enc = getattr(self,'enc','auto')
		 if isBytes(s):
-            if enc is 'auto':
+            if enc == 'auto':
				 try:
					 if s.startswith(codecs.BOM_UTF16_BE):
						 u = s.decode('utf_16_be')
@@ -612,7 +612,7 @@
						 stderr.write('Error in %s' % (repr(s),))
						 raise
		 elif isUnicode(s):
-            if enc is 'auto':
+            if enc == 'auto':
				 if _checkPdfdoc(s):
					 s = s.encode('pdfdoc')
				 else:

However as this tool supports both Python 2 and 3, it deliberately limits the version to an older version.

@afgcons if you upgrade your reportlab version I think this problem will go away. Is this something you can test?

I'm not sure what the long term solution is here, whilst supporting both py2 and py3. Maybe separate requirements files? I know simple version restrictions can be added, for example python_version . Untested thing to test (same package, with two different restrictions):

requirements.txt

reportlab; python_version > "2.7"
reportlab<=3.6.1; python_version <= "2.7"

[afgcons]

I appreciate your help but feel guilty for the disruption. I ran your upgrade instructions. No go, Pillow needs dependencies and I am python newbie.

Look, this can wait. I'll survive without txt2pdf :) until you find a couple of hours to prepare an upgrade.

Thanks again !