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

sc-hsm: build without sm/openssl #1103

Closed
wants to merge 1 commit into from
Closed

sc-hsm: build without sm/openssl #1103

wants to merge 1 commit into from

Conversation

alonbl
Copy link
Member

@alonbl alonbl commented Jul 18, 2017

No description provided.

@frankmorgner
Copy link
Member

This file builds fine in the Window's light flavor, i.e. without OpenSSL. What's the problem with the current implementation?

@alonbl
Copy link
Member Author

alonbl commented Jul 18, 2017

I am unsure how... physically impossible...
Please notice ENABLE_OPENPACE requires openssl/ includes.
https://github.com/OpenSC/OpenSC/blob/master/src/libopensc/card-sc-hsm.c#L405

try to build with --disbale-sm and notice additional failures.

@alonbl
Copy link
Member Author

alonbl commented Jul 18, 2017

Maybe you can refine the secure messaging to enable partial functionality in this module. In any case waiting for this to be resolved to be able to bump version in gentoo.

frankmorgner added a commit to frankmorgner/OpenSC that referenced this pull request Jul 18, 2017
@frankmorgner
Copy link
Member

please try #1104

gentoo is using OpenSC without SM?

@alonbl
Copy link
Member Author

alonbl commented Jul 18, 2017

#1104 working.
however, the ENABLE_OPENPACE still requires ENABLE_OPENSSL, maybe in different patch but should be applied.

gentoo provides most of what the configure is capable of. if SM is not to be disabled, please remove the option from configure.

frankmorgner added a commit that referenced this pull request Jul 19, 2017
metsma pushed a commit to metsma/OpenSC that referenced this pull request Dec 6, 2017
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.

2 participants