[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <d4635a1b-2a50-5858-62f0-c58f5b59b0a2@amd.com>
Date: Tue, 19 Sep 2023 13:06:39 +0530
From: Swapnil Sapkal <Swapnil.Sapkal@....com>
To: Doug Smythies <dsmythies@...us.net>,
'Mario Limonciello' <mario.limonciello@....com>
Cc: rafael.j.wysocki@...el.com, Ray.Huang@....com, li.meng@....com,
shuah@...nel.org, sukrut.bellary@...il.com, gautham.shenoy@....com,
wyes.karny@....com, Perry.Yuan@....com, zwisler@...omium.org,
linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-kselftest@...r.kernel.org,
'Srinivas Pandruvada' <srinivas.pandruvada@...ux.intel.com>
Subject: Re: [PATCH 2/2] tools/power/x86/intel_pstate_tracer: Use pygnuplot
package for Gnuplot
Hello Doug,
Sorry for delay in response.
On 9/18/2023 3:13 AM, Doug Smythies wrote:
> On 2023.09.17 15:17 Doug wrote:
>> On Fri, Sep 15, 2023 at 2:31 PM Mario Limonciello
>>> On 9/15/2023 16:15, Doug Smythies wrote:
>>>> On 2023.09.15 03:41 Swapnil Sapkal wrote:
> ...
>>>> Not really related, but for a few years now I have been meaning to
>>>> change the minimum python version prerequisite to >= 3.0 and
>>>> to change the shebang line.
> ...
>>> Besides the shebang, you should also
>>> use a helper like 2to3 to look for any other changes.
>
> Hi Mario,
>
> I was not aware of the 2to3 helper.
> Thank you mentioning it.
> The 2to3 helper only changed one line,
> which I included in the minimum python version
> patch I just submitted.
>
I tried with installing python3-gnuplot and it worked for me. Initially I
tried with other packages in prerequisites and it didn't worked for me.
I will send v2 removing this patch.
>> I already did the python 3 patch in January, 2020:
>> commit e749e09db30c38f1a275945814b0109e530a07b0
>> tools/power/x86/intel_pstate_tracer: changes for python 3 compatibility
>>
>> I haven't had any issues since, shebang aside.
>
> ... Doug
>
>
--
Thanks and Regards,
Swapnil
Powered by blists - more mailing lists