Hi Nathan,
On Sun, Mar 01, 2015 at 08:04:34PM -0500, Nathan Scott wrote:
> 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.
Ah righto, yeah it is wanted because multiprocess would take a function
with a single argument. I'll work around that somehow or see if python3
has different magic to do it.
> 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).
Thanks a lot! I will fix this up. I totally did not notice
that we already have a python3-pcp package ;) Good stuff!
Btw. I will be around tonight for the release if needed
regards,
Michele
--
Michele Baldessari Sr. Technical Account Manager
Email: <michele@xxxxxxxxxx>
Office: +39 02 36047647 Mobile: +39 342 7382225
C2A5 9DA3 9961 4FFB E01B D0BC DDD4 DCCB 7515 5C6D
|