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

Windows support in JsThemis #475

Merged
merged 1 commit into from
May 22, 2019
Merged

Conversation

ilammy
Copy link
Collaborator

@ilammy ilammy commented May 22, 2019

Update binding.gyp file for Windows. This allows users to install JsThemis on Windows provided that Themis Core is installed under C:\Program Files\Themis. We expect MSYS2-based build of Themis Core to be installed there and use its naming convention and directory layout. Unfortunately, Windows does not have a notion of standard directories for development libraries and headers so we have to tell MSBuild exactly where to look for the library.

Update binding.gyp file for Windows. This allows users to install
JsThemis on Windows provided that Themis Core is installed under
"C:\Program Files\Themis". We expect MSYS2-based build of Themis
Core to be installed there and use its naming convention and
directory layout. Unfortunately, Windows does not have a notion
of standard directories for development libraries and headers
so we have to tell MSBuild exactly where to look for the library.
@ilammy ilammy added jsthemis O-Windows 🖥️ Operating system: Windows labels May 22, 2019
@ilammy ilammy requested review from vixentael and Lagovas May 22, 2019 10:01
Copy link
Collaborator

@Lagovas Lagovas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ilammy ilammy merged commit 1b01bc5 into cossacklabs:master May 22, 2019
@ilammy ilammy deleted the jsthemis-on-windows branch May 23, 2019 15:33
@ilammy ilammy added W-JsThemis 🍭 Wrapper: JsThemis for Node.js, JavaScript API, npm packages and removed E-Node.js labels Sep 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-Windows 🖥️ Operating system: Windows W-JsThemis 🍭 Wrapper: JsThemis for Node.js, JavaScript API, npm packages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants