pcp
[Top] [All Lists]

pcp2pdf under python3

To: Michele Baldessari <mbaldess@xxxxxxxxxx>
Subject: pcp2pdf under python3
From: Nathan Scott <nathans@xxxxxxxxxx>
Date: Sun, 1 Mar 2015 20:04:34 -0500 (EST)
Cc: pcp <pcp@xxxxxxxxxxx>
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <414630668.17841453.1425257577629.JavaMail.zimbra@xxxxxxxxxx>
Reply-to: Nathan Scott <nathans@xxxxxxxxxx>
Thread-index: nIgvCZGnh6/jKf1APvfrHTD4mY0YbQ==
Thread-topic: pcp2pdf under python3
Hi Michele,

I had a quick look at python3 support in pcp2pdf, since that
is freshly in my mind from last week, and came across a few
issues so far.  The biggest being this one, affecting code in
__main__.py:

http://stackoverflow.com/questions/14087598/python-3-3-importerror-no-module-named-configparser

There's also a syntax error at stats.py graph_wrapper function
due to the extra parenthesis, not sure if that was intentional
to ensure a tuple is passed perhaps? ... (if that is what that
does?)  Anyway, it seems in python3 you can't do that anymore.

Attached patch resolves a few, but I wasn't sure how you want
to go about solving that configparser issue so will leave it
here for now.  Patch attached includes a few tweaks to github
URLs as well (plus some little things I may have sent earlier
but aren't merged - or maybe I forgot to send 'em through? --
they're from awhile back, anyway).

cheers.

--
Nathan

Attachment: pcp2pdf.patch
Description: Text Data

<Prev in Thread] Current Thread [Next in Thread>