wxPython-users

cmp() in many places???

Classic

List

Threaded

3 messages Options

johnf

cmp() in many places???

I have discovered that python2 cmp() are still in several files like:

line 10378 in ultimatelistctrl.py. This fact has me concerned that the linux wheel I used was not best? I used

wxPython-4.0.1-cp36-cp36m-linux_x86_64.whl

Anybody have some info on this???? Could it be that we are still using old code within the wheel???

Johnf

--
You received this message because you are subscribed to the Google Groups "wxPython-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
For more options, visit https://groups.google.com/d/optout.

Scott Talbert

Re: cmp() in many places???

On Sat, 21 Apr 2018, John Fabiani wrote:

> I have discovered that python2 cmp() are still in several files like:line
> 10378 in ultimatelistctrl.py. This fact has me concerned that the linux
> wheel I used was not best? I used
>
> wxPython-4.0.1-cp36-cp36m-linux_x86_64.whl
>
>
> Anybody have some info on this???? Could it be that we are still using old
> code within the wheel???

Looks like no:
https://github.com/wxWidgets/Phoenix/blob/master/wx/lib/agw/ultimatelistctrl.py#L10378

--
You received this message because you are subscribed to the Google Groups "wxPython-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
For more options, visit https://groups.google.com/d/optout.

Robin Dunn

Re: cmp() in many places???

In reply to this post by johnf

On Saturday, April 21, 2018 at 7:55:18 AM UTC-7, johnf wrote:

I have discovered that python2 cmp() are still in several files like:
line 10378 in ultimatelistctrl.py. This fact has me concerned that the linux wheel I used was not best? I used

wxPython-4.0.1-cp36-cp36m-linux_x86_64.whl

Anybody have some info on this???? Could it be that we are still using old code within the wheel???

There are a few Python2-specific things scattered around the codebase. When you run into them please create a fix that works with both 2 and 3 and submit a PR to the GitHub project. Or at least create an issue if you're not able to do a PR.

Robin