Mounir IDRASSI posted a comment on merge request #2

@yan12125: Thank you for the update. It is good to hear that Wine has finally included a winscard.dll implementation. I will look at it.

yan12125 updated merge request #2

Fix building with wine 6.23

yan12125 posted a comment on merge request #2

Now that upstream Wine includes an implementation of winscard.dll, I will drop my fork.

yan12125 modified a comment on merge request #2

There are some other errors from newer wine. 7.13 removed wine/unicode.h and 7.22 dropped support for non-msvcrt modules: x86_64-w64-mingw32-gcc -c -o dlls/winscard/x86_64-windows/winscard.o ../wine-winscard/dlls/winscard/winscard.c -Idlls/winscard \ -I../wine-winscard/dlls/winscard -Iinclude -I../wine-winscard/include \ -I../wine-winscard/include/msvcrt -D_UCRT -D__WINESRC__ -D__WINE_PE_BUILD -Wall -g -O2 In file included from ../wine-winscard/dlls/winscard/winscard.c:20: ../wine-winscard/dlls/winscard/wine-compat.h:3:10:...

yan12125 modified a comment on merge request #2

There are some other errors from newer wine. 7.21 removed wine/unicode.h and 7.22 dropped support for non-msvcrt modules: x86_64-w64-mingw32-gcc -c -o dlls/winscard/x86_64-windows/winscard.o ../wine-winscard/dlls/winscard/winscard.c -Idlls/winscard \ -I../wine-winscard/dlls/winscard -Iinclude -I../wine-winscard/include \ -I../wine-winscard/include/msvcrt -D_UCRT -D__WINESRC__ -D__WINE_PE_BUILD -Wall -g -O2 In file included from ../wine-winscard/dlls/winscard/winscard.c:20: ../wine-winscard/dlls/winscard/wine-compat.h:3:10:...

yan12125 posted a comment on merge request #2

There are some other errors from newer wine. 7.21 removed wine/unicode.h and 7.22 dropped support for non-msvcrt modules: x86_64-w64-mingw32-gcc -c -o dlls/winscard/x86_64-windows/winscard.o ../wine-winscard/dlls/winscard/winscard.c -Idlls/winscard \ -I../wine-winscard/dlls/winscard -Iinclude -I../wine-winscard/include \ -I../wine-winscard/include/msvcrt -D_UCRT -D__WINESRC__ -D__WINE_PE_BUILD -Wall -g -O2 In file included from ../wine-winscard/dlls/winscard/winscard.c:20: ../wine-winscard/dlls/winscard/wine-compat.h:3:10:...

yan12125 created merge request #2

Fix building with wine 6.23

yan12125 posted a comment on merge request #1

Glad to help :D

Mounir IDRASSI posted a comment on merge request #1

Thank you @yan12125 for your contribution. I have merged your merge request.

Mounir IDRASSI updated merge request #1

Fix building with wine >= 5.6

yan12125 posted a comment on merge request #1

I updated build instructions for Wine >= 6.18. I didn't test it with older Wine, but it should still work.

yan12125 posted a comment on merge request #1

Done! Thanks for the fast reponse! I didn't find a way to build only winscard.dll. Also, I tested my steps on Wine 6.6~6.8 only.

Mounir IDRASSI posted a comment on merge request #1

Thank you for your efforts on this, I appreciate. I'm ok with this approach. Can you add it to your pull-request?

yan12125 posted a comment on merge request #1

Thanks a lot for the fast and detailed explanations. Sorry for the delay - I got some issues before I found a way when I tried to make sources compatible with both wine 5.0 and 6.x. Here are my attempts: Find a macro for indicating the wine version. There is only a string macro PACKAGE_VERSION, which seems not useful for version comparisons. Use a user-defined macro to choose between wine_dl* and dl* functions. A work-in-place https://sourceforge.net/u/yan12125/scard4wine/ci/19fa5f748b7d3022c0b9e69e1401d4a123ad246d/...

Mounir IDRASSI posted a comment on merge request #1

Thank you for this contribution. Since version 5.0.x is still maintained, I prefer to have this change implement using macro defines that are defined correctly depending on the version of WINE used for the build (if version < 5.6, defines will use wine_XXX functions and if version >= 5.6, defines will use dlfcn functions). Is it possible to update the pull request to use this approach? Concerning submission to upstream or staging, I have nothing against but the last time I tried, I gave up because...

yan12125 created merge request #1

Fix building with wine >= 5.6

SCard4Wine released /1.2.0/README.txt

SCard4Wine released /1.2.0/winscard-linux.tar.gz

Mounir IDRASSI committed [51b614]

Fix various memory related issues that causes crashes or non visibility of readers.

Mounir IDRASSI committed [467521]

Update installation script to also copy winscard.dll.fake

Peter Szili posted a comment on ticket #3

Hi thanks for your reply. I remake wine with your winscard.c with Suse 13.2 x86,...

Mounir IDRASSI posted a comment on ticket #3

Hi, Thank you for the report. I was able to reproduce the "Invalid floating point...

Peter Szili created ticket #3

Deplhi can not load the new wincard.dll.so

William Gnann posted a comment on ticket #2

Yes, I think I installed the card reader on Linux properly. The pcsc_scan returns...

Mounir IDRASSI posted a comment on ticket #2

Thank you for posting all the logs files. From what I see, there is nothing that...

William Gnann created ticket #2

Safesign crashes with wine 1.[67]

Mounir IDRASSI committed [c14c02]

fix SCardStatusA() and SCardTransmit() to be co...

Mounir IDRASSI modified ticket #1

SafeSign crash

Mounir IDRASSI posted a comment on ticket #1

Hi, The debug dump you posted seem to confirm that Winscard is not responsible and...

Anderson posted a comment on ticket #1

Hi! Sorry for the delay. It seems that one of Internet Banking "Security Module"...

Mounir IDRASSI posted a comment on ticket #1

Hi, Thank you for your update. Indeed, your investigation of the issue will be helpful...

Anderson posted a comment on ticket #1

Hi, I've managed to make it work directly on linux (without wine, with a native safesign...

Mounir IDRASSI posted a comment on ticket #1

Hi, SCard4Wine is known to work with many smart card middlewareon the market like...

Anderson created ticket #1

SafeSign crash