Skip to content

Fix build on GNU/Linux#4

Open
LudovicRousseau wants to merge 6 commits into
cardid:mainfrom
LudovicRousseau:main
Open

Fix build on GNU/Linux#4
LudovicRousseau wants to merge 6 commits into
cardid:mainfrom
LudovicRousseau:main

Conversation

@LudovicRousseau

Copy link
Copy Markdown

No description provided.

Use "-framework PCSC" on macOS only.
On GNU/Linux we use pkg-config(1) to get the correct arguments
webcard.cpp:135:3: error: use of undeclared identifier 'mbstowcs_s'
                mbstowcs_s<256>(&ccNumChar, buffer, "\\\\?PnP?\\Notification", 128);
                ^
webcard.cpp:138:41: error: assigning to 'const char *' from incompatible type 'wchar_t [256]'
                _rgReaderStates[_cReaders].szReader = buffer;
webcard.cpp:145:2: error: cannot jump from switch statement to this case label
        default:
        ^
webcard.cpp:133:8: note: jump bypasses variable initialization
                char buffer[] = "\\\\?PnP?\\Notification";
                     ^
The error was:
./install.sh: 9: [: Linux: unexpected operator
The error was:
./install.sh: 38: Bad substitution
@Sciumo

Sciumo commented Jun 6, 2023

Copy link
Copy Markdown

Is this project live? I don't see any issue with this PR.

@ROBERT-MCDOWELL

Copy link
Copy Markdown

@Sciumo maybe it's time to fork

@ROBERT-MCDOWELL

Copy link
Copy Markdown

@LudovicRousseau could you fork this project and apply your PR? then I think I will follow your fork thanks!

@LudovicRousseau

Copy link
Copy Markdown
Author

I do not use this project.
I think the idea was to write about it for my list at https://blog.apdu.fr/posts/2010/04/pcsc-sample-in-different-languages/
But the project is not maintained.

@ROBERT-MCDOWELL

Copy link
Copy Markdown

@LudovicRousseau ok. I wonder if it's not better to use wasm today rather than an extension since it won't work on android/IOS anyhow...

@Sciumo

Sciumo commented Feb 29, 2024

Copy link
Copy Markdown

It's probably better to rewrite. WASM is a build chain question.
There is a Rust version already.
https://docs.rs/pcsc/latest/pcsc/

@Sciumo

Sciumo commented Feb 29, 2024

Copy link
Copy Markdown

It's probably better to rewrite. WASM is a build chain question. There is a Rust version already. https://docs.rs/pcsc/latest/pcsc/

Which cites @LudovicRousseau blog! 🥇
https://ludovicrousseau.blogspot.com/

@ROBERT-MCDOWELL

Copy link
Copy Markdown

@Sciumo thanks for the link!

@LudovicRousseau

Copy link
Copy Markdown
Author

It's probably better to rewrite. WASM is a build chain question. There is a Rust version already. https://docs.rs/pcsc/latest/pcsc/

Which cites @LudovicRousseau blog! 🥇 https://ludovicrousseau.blogspot.com/

See also https://blog.apdu.fr/posts/2017/03/pcsc-sample-in-rust/

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

Successfully merging this pull request may close these issues.

3 participants