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.

Sign up for GitHub

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

Jump to bottom

Separate npm for sqlcipher? #417

Closed
coolaj86 opened this issue Feb 22, 2015 · 4 comments
Closed

Separate npm for sqlcipher? #417

coolaj86 opened this issue Feb 22, 2015 · 4 comments
Labels
sqlcipher

Comments

@coolaj86
Copy link
Contributor

Should we have a separate npm package for sqlcipher?

To me it seems different enough to deserve it's own package and its own pre-built bins. What do you think?

Related Note: #418
@nisargjhaveri
Copy link

I agree with you.

I think, someone tried to do something on these lines, but it is not maintained anymore. https://github.com/delaballe/node-sqlcipher

If you or someone can create and maintain a separate package for sqlcipher, it would be of great help.
Thanks :)

@fritx
Copy link

fritx commented Oct 8, 2016

Hello there
Ive just created a package https://github.com/fritx/cross-sqlcipher
which was a built on top of node-sqlite3.

Current it works with node and cross-platform.
And I will make it work with nwjs and electron soon.

@fritx
Copy link

fritx commented Oct 8, 2016

cross-sqlcipher contains just scripts that help you land things, without any sqlite logic.
So there is no duplication or over-design.

fritx pushed a commit to fritx/unix-sqlcipher that referenced this issue Oct 9, 2016
0.0.3 - pass npm_config_argv for target build, such as nwjs
f55b357 
https://github.com/Gaubee/blog/issues/91
TryGhost/node-sqlite3#417
TryGhost/node-sqlite3#497
TryGhost/node-sqlite3#587
fritx added a commit to fritx/win-sqlcipher that referenced this issue Oct 9, 2016
@fritx
0.0.3 - pass npm_config_argv for target build, such as nwjs
f0a9aa1 
fritx/unix-sqlcipher@f55b357
TryGhost/node-sqlite3#417
TryGhost/node-sqlite3#497
fritx added a commit to fritx/unix-sqlcipher that referenced this issue Nov 9, 2017
@fritx
0.0.3 - pass npm_config_argv for target build, such as nwjs
2bb825b 
https://github.com/Gaubee/blog/issues/91
TryGhost/node-sqlite3#417
TryGhost/node-sqlite3#497
TryGhost/node-sqlite3#587
@daniellockyer
Copy link
Member

It's possibly a good idea but out of scope for this project and the issue is stale, so I'm going to close it 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
sqlcipher
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@coolaj86 @daniellockyer @nisargjhaveri @fritx