Example needs autogypi to build and run

[charto]

Example needs autogypi to build and run

[jjrv]

autogypi is listed in devDependencies in nbind's package.json, and dependencies in the example. So it should be installed with the steps:

npm install
npm test

There, npm install when run inside the git cloned repo or the example, should install it. Maybe something silently failed when you ran it with older npm installed, or you didn't notice an error message and continued? It sounds like the correct fix is to set the minimum required npm version in package.json so it would fail more spectacularly before even getting close to installing autogypi. I'll need to do the same to autogypi's package.json too. Thanks for reporting.

[jjrv]

Another possible fix is to rename autolink and publish it in the global npm namespace instead of the @lib scope. Problem is that the "autolink" name is taken and I don't want to reserve any other name for it either, since it's a utility package only ever meant to be used by autodts and autogypi. Since it has no wider utility, I think it belongs in a scope, but older npm versions lacking scope support will then fail to install nbind correctly for such a minor reason... What do you think is the best option here?

[brodybits]

I had already upgraded npm before following the Example section, still had to install autogypi globally and run it manually before I could build it using npm install. But if I change "prepublish" to "install" in the sample package.json, uninstall my autogypi, and remove the artifacts (auto.gypi, build, and node_modules), then I am able to build it using npm install.

If you want to work with older npm versions, you could start all of your utility packages with a prefix that you do not expect anyone will (or should) use. But I we suspect we should be able to expect the users to update npm (or start using newer npm on new system installations) over time.

[jjrv]

It seems prepublish was wrong in this case, I replaced it with install and added notes about updating npm. Thanks for the help.