node-deb icon indicating copy to clipboard operation
node-deb copied to clipboard

No more node_modules/.bin directory in the debian package

Open Skywalker13 opened this issue 7 years ago • 7 comments

Hi

I don't know exactly why and when but it seems that with an update a buster, the node_modules/.bin directory is no longer packaged. Any ideas?

I use the auto strategy (default) and node_modules is present

Thanks

Skywalker13 avatar Jan 31 '19 21:01 Skywalker13

I think that's is related to #71 that breaks too much things... no more symlinks in .bin then no more .bin dir... and with the copy strategy the binaries in .bin/ are juste not usable anymore because it's no longer symlinks..

Skywalker13 avatar Jan 31 '19 22:01 Skywalker13

This sounds like it might be related to #71, yeah. I'm a little swamped with work things but I'll see if I can get around to doing a copy-with-dereference.

heartsucker avatar Feb 01 '19 09:02 heartsucker

Ah #71 was already closed, so those should be referenced. Is the error you're experiencing in this ticket the same error as your mentioned in #71?

heartsucker avatar Feb 01 '19 09:02 heartsucker

Yes, after more investigate it's the same. I'm looking for forking and fixing the regression ASAP because I must provide a working package today.

Skywalker13 avatar Feb 01 '19 12:02 Skywalker13

With this "revert" https://github.com/Xcraft-Inc/node-deb/commit/a317f931d10ff7359d04b51e5965e7c3347c350e it works as expected with the copy strategy. But with the auto strategy, I continue to lose the .bin directory :-(

Skywalker13 avatar Feb 01 '19 12:02 Skywalker13

Ok so would the solution to fix this for everyone be to:

  • do not dereference symlinks in node_modules/.bin
  • dereference all other symlinks

heartsucker avatar Mar 01 '19 15:03 heartsucker

It should be fine most of times, even if IMO dereferencing symlinks looks wrong to me.

Skywalker13 avatar Mar 08 '19 07:03 Skywalker13