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

Missing dependency npm-bcrypt when using accounts-password #1

Closed
TPXP opened this issue Nov 2, 2014 · 4 comments
Closed

Missing dependency npm-bcrypt when using accounts-password #1

TPXP opened this issue Nov 2, 2014 · 4 comments

Comments

@TPXP
Copy link

TPXP commented Nov 2, 2014

Hello!

First of all, many thanks for the hard work you put in this fork to support the ARM architecture, it's awesome!

I'm currently trying to create a project using accounts-password, so that users can connect with a password. But there's a little problem when running meteor add accounts-password, because a meteor package used to encrypt passwords : npm-bcrypt does not appear as compatible (you have to build it when you install it with npm) and meteor refuses to continue without this package.

As far as I could see (sorry I am not knowledgeable a lot about meteor internals 😕 ), this package is located at <meteor root>/packages/non-core/npm-bcrypt/. I tried installing the bcrypt package at <meteor root>/packages/non-core/npm-bcrypt/.npm/package/node_modules/bcrypt (note meteor wants version 0.7.7, not 0.8.0!), but still no luck, the package is not recognised as installed, so it tries to install it, and fails when I intent loading my project.

I also saw there was a node_modules in <meteor root>/dev-bundle/lib, but running npm install [email protected], once done, does not solve the problem.

Any idea how to fix this issue ?

Many thanks in advance!

@TomFreudenberg
Copy link
Member

Hi, thanks for pointing that out here. Solution is very easy as you could read in

https://github.com/meteor/meteor/tree/devel/packages/non-core

To use these packages just have two options:

a. general: create a symbolic link into your meteors checkout

cd <your meteor checkout path>/packages
ln -s non-core/npm-bcrypt .

b. project specific

cd <your meteor app path>/packages
ln -s <your meteor checkout path>/non-core/npm-bcrypt .

c. get published versions of the packages from Atmosphere

cd <your meteor app path>
meteor add npm-bcrypt

Personally I would prefer a.) to use always the git delivered versions

Tom

P.S.: There are maybe more non-core npm's. If you need to use it (e.g. npm-node-aes-gcm) go on with same procedere

@TomFreudenberg TomFreudenberg changed the title It is not possible to use accounts-password due to an unatisfied dependency Missing dependency npm-bcrypt when using accounts-password Nov 3, 2014
@TomFreudenberg
Copy link
Member

Issue documented and closed

@TPXP
Copy link
Author

TPXP commented Nov 8, 2014

Many thanks for your help, I can confirm it works now! 👍

@TomFreudenberg
Copy link
Member

ATTENTION !!! As from a comment from MDG you SHOULD NOT link the non-core packages directly inside the packages directoy of a checkout meteor.

The correct advise is:

# setting the env var 
export PACKAGE_DIRS=<your meteor checkout path>/packages/non-core

If this is not suitable for you, you still may link the non-core package to your project

cd <your meteor app path>/packages
ln -s <your meteor checkout path>/non-core/npm-bcrypt .

TomFreudenberg pushed a commit that referenced this issue Jul 8, 2015
The #1 goal is to not say, "Your packages are at
their latest compatible versions" whenever an
update has no effect.  That isn't necessarily
true.  `meteor update` with no arguments never
updates a major/minor of an indirect dependency,
for example.  Also, you may have specified some
packages on the command line (though arguably
"your packages" could be interpreted to refer to
those packages).

In addition, `meteor update` with no arguments now
reports any direct or indirect dependencies that
aren't at their latest versions.

For example:

```
Your top-level dependencies are at their latest compatible versions.

Newer versions of the following indirect dependencies are available:
 * aldeed:collection2 0.1.7 - 2.3.3 is available
To update one or more of these packages, pass their names to `meteor update`.
```

Sort of related to meteor#4170.
TomFreudenberg pushed a commit that referenced this issue Jul 8, 2015
commit 6d34609
Merge: 800ec27 2ce8171
Author: freiit <[email protected]>
Date:   Tue Jun 16 11:56:53 2015 +0200

    Merge pull request #1 from freiit/freiit-patch-1

    Trivial inconsistency in the docs fixed.

commit 2ce8171
Author: freiit <[email protected]>
Date:   Tue Jun 16 11:47:30 2015 +0200

    Trivial inconsistency in the docs fixed.
shutej pushed a commit to shutej/meteor that referenced this issue Jul 18, 2017
…feescript-babel

Fix CoffeeScript exported variables on files processed by Babel
shutej pushed a commit to shutej/meteor that referenced this issue Jul 18, 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

No branches or pull requests

2 participants