Skip to content
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

x64 Windows version #163

Closed
Neustradamus opened this issue Apr 9, 2023 · 7 comments
Closed

x64 Windows version #163

Neustradamus opened this issue Apr 9, 2023 · 7 comments

Comments

@Neustradamus
Copy link

Hi @namazso,

It is possible to create the x64 Windows version?

Currently it is in x86 only.

Thanks in advance.

@namazso
Copy link
Owner

namazso commented Apr 9, 2023

Currently it is in x86 only.

No it isn't. The installer will install the correct version. The fact that it works at all shows it's the right architecture, as only matching architecture can work as a shell extension.

@namazso namazso closed this as completed Apr 9, 2023
@Neustradamus
Copy link
Author

@namazso: I have tested on a Windows x64 machine, it has been installed in "C:\Program Files (x86)\OpenHashTab"

  • AlgorithmsDll-x64-avx.dll
  • AlgorithmsDll-x64-avx2.dll
  • AlgorithmsDll-x64-avx512.dll
  • AlgorithmsDll-x64-sse2.dll
  • OpenHashTab.dll
  • unins000.dat
  • unins000.exe

@namazso
Copy link
Owner

namazso commented Apr 9, 2023

Oh the install location can very well be incorrect, but don't worry about that, just that it installs a 64 bit version correctly, which you can see as it installed the x64 dlls. You can also check the OpenHashTab.dll with any PE viewer to see it's also 64 bit.

@Neustradamus
Copy link
Author

@namazso: Can you solve this installer issue?

  • x64 -> "Program Files"
  • x86 -> "Program Files (x86)"

@namazso
Copy link
Owner

namazso commented Apr 9, 2023

Might look at it but it might just be a limitation of InnoSetup because it sees I have both 32 and 64 bit dlls in the same installer. However, considering it doesn't affect the operation of the program at all, I don't feel like this is too important.

Btw you can simply change the install location in the installer:

image

@Neustradamus
Copy link
Author

@namazso: Have you found the problem?

@namazso
Copy link
Owner

namazso commented Jul 16, 2023

@Neustradamus This was fixed with 3abe557 as part of removing 32-bit support entirely

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants