New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Everything package don't work #205

Closed
Ublimjo opened this Issue Jun 14, 2017 · 9 comments

Comments

Projects
None yet
4 participants
@Ublimjo

Ublimjo commented Jun 14, 2017

ashampoo_snap_mercredi 14 juin 2017_10h22m11s_001_console - keypirinha

10:02:16.513 Python: Traceback (most recent call last):
10:02:16.513 Python: File "C:\Program Files\Keypirinha\default\Packages\Everything.keypirinha-package\everything.py", line 101, in on_suggest
10:02:16.513 Python: File "C:\Program Files\Keypirinha\default\Packages\Everything.keypirinha-package\everything.py", line 149, in search
10:02:16.513 Python: File "C:\Program Files\Keypirinha\default\Packages\Everything.keypirinha-package\lib\everything_ipc.py", line 182, in query
10:02:16.513 Python: File "C:\Program Files\Keypirinha\default\Packages\Everything.keypirinha-package\lib\everything_ipc.py", line 302, in create_window
10:02:16.514 Python: File "lib\ctypes_init
.py", line 357, in getattr
10:02:16.514 Python: File "lib\ctypes_init
.py", line 362, in getitem
10:02:16.514 Python: AttributeError: function 'SetWindowLongPtrW' not found
10:02:24.980 ** ERROR: Everything.Everything: Something bad happened while requesting Everything to perform your search.
10:02:24.982 Python: Traceback (most recent call last):
10:02:24.982 Python: File "C:\Program Files\Keypirinha\default\Packages\Everything.keypirinha-package\everything.py", line 101, in on_suggest
10:02:24.983 Python: File "C:\Program Files\Keypirinha\default\Packages\Everything.keypirinha-package\everything.py", line 149, in search
10:02:24.983 Python: File "C:\Program Files\Keypirinha\default\Packages\Everything.keypirinha-package\lib\everything_ipc.py", line 182, in query
10:02:24.983 Python: File "C:\Program Files\Keypirinha\default\Packages\Everything.keypirinha-package\lib\everything_ipc.py", line 302, in create_window
10:02:24.983 Python: File "lib\ctypes_init
.py", line 357, in getattr
10:02:24.983 Python: File "lib\ctypes_init
.py", line 362, in getitem
10:02:24.983 Python: AttributeError: function 'SetWindowLongPtrW' not found
10:02:25.921 ** ERROR: Everything.Everything: Something bad happened while requesting Everything to perform your search.
10:02:25.929 Python: Traceback (most recent call last):
10:02:25.929 Python: File "C:\Program Files\Keypirinha\default\Packages\Everything.keypirinha-package\everything.py", line 101, in on_suggest
10:02:25.929 Python: File "C:\Program Files\Keypirinha\default\Packages\Everything.keypirinha-package\everything.py", line 149, in search
10:02:25.929 Python: File "C:\Program Files\Keypirinha\default\Packages\Everything.keypirinha-package\lib\everything_ipc.py", line 182, in query
10:02:25.930 Python: File "C:\Program Files\Keypirinha\default\Packages\Everything.keypirinha-package\lib\everything_ipc.py", line 302, in create_window
10:02:25.930 Python: File "lib\ctypes_init
.py", line 357, in getattr
10:02:25.930 Python: File "lib\ctypes_init
.py", line 362, in getitem
10:02:25.930 Python: AttributeError: function 'SetWindowLongPtrW' not found

@ueffel

This comment has been minimized.

Show comment
Hide comment
@ueffel

ueffel Jun 14, 2017

Are you running 32 Bit Windows?

ueffel commented Jun 14, 2017

Are you running 32 Bit Windows?

@Ublimjo

This comment has been minimized.

Show comment
Hide comment
@Ublimjo

Ublimjo commented Jun 16, 2017

Yes

@ueffel

This comment has been minimized.

Show comment
Hide comment
@ueffel

ueffel Jun 16, 2017

There seems to be an issue with the SetWindowLongPtrW on 32 Bit Windows.
https://msdn.microsoft.com/de-de/library/windows/desktop/ms644898(v=vs.85).aspx states, to be compatible with both 32bit and 64bit windows, one should use SetWindowLongPtr

As a temporary fix (until its fixed in the official release, which might be a while because @polyvertex is on a long vacation), you could replace the call to SetWindowLongPtrW with SetWindowLongPtr (without W) within the <KEYPIRINHA_HOME>/default/Packages/Everything.keypirinha-package/lib/everything_ipc.py File

If you're not a coder, you can use the attached file, rename it to Everything.keypirinha-package and overwrite the existing file in <KEYPIRINHA_HOME>/default/Packages/

Everything.gz

ueffel commented Jun 16, 2017

There seems to be an issue with the SetWindowLongPtrW on 32 Bit Windows.
https://msdn.microsoft.com/de-de/library/windows/desktop/ms644898(v=vs.85).aspx states, to be compatible with both 32bit and 64bit windows, one should use SetWindowLongPtr

As a temporary fix (until its fixed in the official release, which might be a while because @polyvertex is on a long vacation), you could replace the call to SetWindowLongPtrW with SetWindowLongPtr (without W) within the <KEYPIRINHA_HOME>/default/Packages/Everything.keypirinha-package/lib/everything_ipc.py File

If you're not a coder, you can use the attached file, rename it to Everything.keypirinha-package and overwrite the existing file in <KEYPIRINHA_HOME>/default/Packages/

Everything.gz

@Ublimjo

This comment has been minimized.

Show comment
Hide comment
@Ublimjo

Ublimjo Jun 18, 2017

It does not work yet
It generates the same error

Ublimjo commented Jun 18, 2017

It does not work yet
It generates the same error

@polyvertex

This comment has been minimized.

Show comment
Hide comment
@polyvertex

polyvertex Jun 19, 2017

Member

Sorry for the delay, I'm a bit away from the numeric world right now. Thanks @ueffel for attempting to provide a patch. I will investigate asap.

Member

polyvertex commented Jun 19, 2017

Sorry for the delay, I'm a bit away from the numeric world right now. Thanks @ueffel for attempting to provide a patch. I will investigate asap.

@ueffel

This comment has been minimized.

Show comment
Hide comment
@ueffel

ueffel Jun 19, 2017

Sorry it didnt work. I couldnt test it because i dont have a 32bit system :(

ueffel commented Jun 19, 2017

Sorry it didnt work. I couldnt test it because i dont have a 32bit system :(

@zwkin

This comment has been minimized.

Show comment
Hide comment
@zwkin

zwkin Jul 13, 2017

Hi, @Ublimjo , I have the same problem with you. I solved it by replacing the call kpwt.user32.SetWindowLongPtrW with kpwt.SetWindowLongPtr, and it work for me.

zwkin commented Jul 13, 2017

Hi, @Ublimjo , I have the same problem with you. I solved it by replacing the call kpwt.user32.SetWindowLongPtrW with kpwt.SetWindowLongPtr, and it work for me.

@polyvertex

This comment has been minimized.

Show comment
Hide comment
@polyvertex

polyvertex Jul 15, 2017

Member

Nice catch @zwkin! This 32-bit use case is actually already taken into account by the keypirinha_wintypes module but I did not fully propagate changes to everything_ipc.

I'm currently on a 64-bit machine so I haven't tested the fix but you should be fine with Here is the fixed version: Everything.keypirinha-package.zip (rename to Everything.keypirinha-package and overwrite the original /default/Packages/Everything.keypirinha-package archive). Please let us know how it goes @Ublimjo.

Member

polyvertex commented Jul 15, 2017

Nice catch @zwkin! This 32-bit use case is actually already taken into account by the keypirinha_wintypes module but I did not fully propagate changes to everything_ipc.

I'm currently on a 64-bit machine so I haven't tested the fix but you should be fine with Here is the fixed version: Everything.keypirinha-package.zip (rename to Everything.keypirinha-package and overwrite the original /default/Packages/Everything.keypirinha-package archive). Please let us know how it goes @Ublimjo.

@polyvertex

This comment has been minimized.

Show comment
Hide comment
@polyvertex

polyvertex Jul 19, 2017

Member

Fixed in 2.15.2.
Do not hesitate to comment if issue persists.

Member

polyvertex commented Jul 19, 2017

Fixed in 2.15.2.
Do not hesitate to comment if issue persists.

@polyvertex polyvertex closed this Jul 19, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment