-
Notifications
You must be signed in to change notification settings - Fork 9
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
Error in node v6: TypeError: haystack.get is not a function #13
Comments
Can confirm +1 |
I only get this error on Windows. @binarykitchen Are you on Windows? |
@alexbooker looks like he's on Linux |
The "slug" module has a transitive dependency called "bufferjs". "bufferjs" stopped working when Node 6 removed the Buffer.get method [1]. This broke all modules that depend on "bufferjs" in environments running Node 6. I sent a pull request [2] to fix "bufferjs" but I'm not hopeful it'll get merged any time soon. What's more, even my pull request *does* get merged, the "slug" module hasn't been updated in many months. Closes #105. [1] coolaj86/node-bufferjs#13 [2] coolaj86/node-bufferjs#14
me on windoze? no way lol ... yup, linux it is |
It's weird that I only experience this on Windows |
@alexbooker perhaps you need to try from a clean installation? |
I'm getting the same error in a travis-ci test, just for Node 6.x. https://travis-ci.org/liferay/generator-liferay-theme/jobs/126265971 |
+1 Also getting this error on Windows:
My system:
Any workaround for this? |
@pazoozooCH, there are two open PR's that can fix this, but it seems like @coolaj86 doesn't maintain this package anymore so it's unlikely that it will be fixed. |
@nakamin , thanks, we found a solution with another dependency, so it's not that urgent for me anymore 😄 |
Sorry, I don't mean to be a bum, just busy with other work these days. I just have repo access to @alexbooker because he hit me up on twitter, so feel free to ask him for access if you need to fix anything once he pulls in and publishes his patch. |
@coolaj86 Sorry if came off rude or mean... wasn't the intention 😄. Giving repo access to @alexbooker is definitley a great idea! I'm sure he'll take care of all the issues. |
Also on linux, FWIW. Actually installed fine on Windows... |
@anyong Yeah, I see no reason why this problem would be limited to a platform -- I suspect you aren't running Node 6 on Windows. I just bumped the version number in 72c7763 and have updated the If someone can confirm that the problem is solved, that'd be great, thank you! |
I can confirm that I'm no longer getting the error in my travis-ci tests. Thanks @alexbooker |
That is terrific! Thanks, @Robert-Frampton. |
This says all. Happens only on node v6:
The text was updated successfully, but these errors were encountered: