Fix build tests and correct dependency information

meganz / jodid25519

Curve25519-based cryptography in JavaScript

MIT License

34 stars 17 forks source link

Fix build tests and correct dependency information #5

Closed infinity0 closed 10 years ago

infinity0 commented 10 years ago

We don't need to build dependencies on every single invocation; this happened because the phony target "dependencies" hid the real dependency information. Also fixed the build tests.

infinity0 commented 10 years ago

I don't know why travis is failing, one of the builds succeeded https://travis-ci.org/meganz/jodid25519/builds/27706436

It looks like the first build ran "npm prepublish" on asmcrypto for some reason, which did a default-build over our custom version. This then screwed with the second test, but somehow not the first test.

I can't reproduce this however - "npm test" works fine locally, even if I run it twice.