-
Notifications
You must be signed in to change notification settings - Fork 68
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
npm install libxslt fails with compilation errors #20
Comments
The same on Ubuntu 14.04 |
Temporary fix is to revert to 0.5.1 version. |
Bumping libxmljs-mt to 0.14.3 I changed some paths, because I turned the libxml upstream into a git submodule. I had assumed that the corresponding path settings would get exported from my gyp files, but apparently that is not the case: the paths in question are encoded on in node-libxslt's |
@gagern Hi there. Was trying to follow your instructions but clearly goofed something up. I think I'm able to get past the missing
Any ideas? |
@mlevin2: Sounds like a missing |
This is where libxmljs-mt keeps its xmlversion.h header file, see albanm#20.
By the way, I just worked out how to fix this on the libxmljs-mt level as well. So libxmljs-mt@0.14.5 and later should work even without this modification. But the modification might still useful, in case someone wants do build against a snapshot of libxmljs-mt. I only add the file for compatibility in the So feel free to close this here, and merge #22 if you want to cater for exotic build situations, or reject it if you are happy catering for those who either pull the module from the registry in any case (which should be most users) or at least have the decency to do a |
Ok, I added the include path at the same time that I fixed libxmljs-mt version in order to prevent nan conflict. It is a problem that breaking changes in libxmljs for node-libxslt can be released as simple patches. This is because it relies on stuff that is considered internal by libxmljs. I think I might keep a static dependancy (no ~ or ^). |
0.5.3 just released. |
I agree, this is problematic. I considered whether I should be changing the versioning scheme for libxmljs-mt, but I didn't want to deviate too much from libxmljs.
I believe that's a good idea. If you want me to, I can (try to remember to) file a pull request bumping the version number whenever I do a new release of libxmljs-mt, which in turn would be whenever I find that libxmljs made a release or there is something wrong with the way I deviated from it. By the way, libxmljs supports iojs 2 in its travis tests since libxmljs/libxmljs#318, and iojs 3 since the nan 2 updates. If you want you can try that out for node-libxslt as well. Just be sure to not manually upgrade npm on those. |
Ok, thanks. |
Hmm, at one point I had this working, but I had to delete my node_modules folder and I guess I hadn't saved the config correctly. I'm now unable to install libxslt on Yosemite. Can anyone point me in the right direction?
|
@mlevin2 Try using libxmljs-mt 0.14.5. If it does work, changing the |
Sorry, no, 0.14.5 is nan2, requires #23 addressed. Manually performing the prepublish step from gagern/libxmljs@0fe2b62 should work: cp vendor/libxml.conf/include/libxml/xmlversion.h vendor/libxml/include/libxml/ If that doesn't help, you might be seeing some different issue which only looks the same at first glance. Since your pasted code doesn't mention |
@gagern node 4.2.0, npm 2.14.7, node-gyp 3.0.3, OSX Yosemite. I'm getting farther. Here's the status:
|
Node 4 support requires #23 being addressed. |
Thanks, I'll give your update a try. |
@gagern success! I'm a dummy and had another problem but I was able to merge in your nan2 branch and get everything working. Thanks so much for your help. |
I use WSL 1.x and Docker Desktop 2.0.x on Windows 10 1903, with linked volumes that are on my NTFS filesystem. These types of errors are impossible for me to get around with any of the suggested hacks. What DOES work is to create a temp folder inside my container volume (not linked), |
npm install libxslt
fails on Yosemite withI've tried with the latest version of node-gyp. I also tried installing libxmljs out of curiosity and it worked fine. Any ideas what I've missed? Thanks.
The text was updated successfully, but these errors were encountered: