Add python path to configuration #103
Conversation
|
Interesting. Do you have any info on what module was requiring python? In theory we should not need this. |
|
Yeah, I was really confused why node was trying to install stuff with python. It looks like the node-gyp module requires python. https://github.com/TooTallNate/node-gyp |
|
Okay I see. So since I have you here... :) Could you try doing this before you run npm install on Windows? it's not realy any different but seems like it'd be a bit cleaner, though im not (part of it is that Im curious) |
|
yeah no dice. I think setting python as a system variable is probably the best shot. appears to work, but then I get caught up on some other fun 64bit node window stuff. I'll play around some more with it and let you know what I dig up. |
|
sorry this sat here so long, I totally missed it. |
Add python path to configuration
|
Hey @bmoregeo any luck with koop on windows? the whole code base changed so the install process might be different. Though the modules are all the same. Would love to see if this will work on windows. |
* Added ability to set field `length` property from provider metadata. Add warning if provider set field-metadata does not match feature properties. Refactored `field/index.js` to improve readability and implement code reuse. Move type detection functions to utils. * Remove detect-types file now that functions have been move to utils.js * Add to changelog. * Add test for metadata-defined field length. * Remove OBJECTID warning (moved to winnow). Added additional ESRI types to those supported in metadata's definition of fields. * Update changelog. * Remove `esrifieldtype` prefix from type mapping. * Adjust type map test. * Remove duplicated shoulds from test.
Add python path to configuration
* update dependencies * add pre-commit hook * update travis config * update changelog
Good Morning,
I tried installing koop but received an error about not having a python system variable. I'm not sure if this is just windows, but you might want to add it to the Readme.