-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
adds support for pre-compiled binaries #392
Conversation
I tried on 2 different windows machines with node 4.4.3 & 5.1.0 and in both I get build errors: gyp info spawn args '-Goutput_dir=.' ] I tried this on Linux and following the steps able to compile. Also does node-pre-gyp support node 0.10.x? |
gyp: Undefined variable module_name in binding.gyp while trying to load binding.gyp This seem to be error. |
Hi @pvenkatraman, Using my ordered list as a guide, do you get this error at item 1 sub section xvi? Also, are you doing a In regards to, |
I just looked at appveyor.yml with node-pre-gyp and I see testing for nodejs_version: 0.10.36, so it looks like node-pre-gyp does in fact support 0.10.x |
@springmeyer just confirmed that node-pre-gyp supports node v0.10.x and greater via mapbox/node-pre-gyp#203 (comment) |
@pvenkatraman I just updated step 1 sub section ix within the instructions so that it is more clear. I crossed out the old wording. I think this is what was causing you the problem. Let me know. Thanks. |
Is there any chance to get node-pre-gyp and official pre built binarieis? |
Merge upstream changes
update to 1.11
merge 1.12 from upstream
merge 1.12.2 from upstream
For anyone following this PR, we are looking at a lighter weight solution that doesn't have the automatic fallback to compilation if the binary install fails. See #18 (comment) and the dev-2.0 branch package/README.md and the dev-2.0 branch INSTALL.md. If (as I expect) the light weight solution is practical, I'll close out this PR. |
@bchr02 I'll close this now we have the lightweight 2.0 binary installer. Couldn't have done it without you. Many thanks. |
_sorry had to re-add this pull request to address an issue with my fork which was preventing me from submitting additional pull requests_
Using node-pre-gyp and a new bin folder, this pull request adds:
set ORACLE_HOME=C:\oracle\instantclient
for Linux useexport
instead ofset
set PATH=%ORACLE_HOME%;%PATH%
for Linux useexport
instead ofset
set OCI_LIB_DIR=%ORACLE_HOME%\sdk\lib\msvc
for Linux useexport
instead ofset
set OCI_INC_DIR=%ORACLE_HOME%\sdk\include
for Linux useexport
instead ofset
git clone https://github.com/bchr02/node-oracledb
thengit checkout prebuild_support
.git clone https://github.com/oracle/node-oracledb
(or while testing with my repo usegit clone https://github.com/bchr02/node-oracledb
thengit checkout prebuild_support
.)npm install nan
npm install node-gyp -g
npm install node-pre-gyp -g
node-pre-gyp clean
node-pre-gyp configure
node-pre-gyp build
node-pre-gyp package --toolset=oci12.1-msvs2015
xcopy build\stage\*.* bin /C /Y
or for Linux:cp -f build/stage/* bin
git add -A
git commit
git push -u origin master
set ORACLE_HOME=C:\oracle\instantclient
for Linux useexport
instead ofset
set PATH=%ORACLE_HOME%;%PATH%
for Linux useexport
instead ofset
set OCI_LIB_DIR=%ORACLE_HOME%\sdk\lib\msvc
for Linux useexport
instead ofset
set OCI_INC_DIR=%ORACLE_HOME%\sdk\include
for Linux useexport
instead ofset
IMPORTANT TO UNDERSTAND: The bin folder is an online only folder. When npm installing node-oracledb the bin folder will not downloaded. Instead, node-pre-gyp will check the online github folder for a suitable file. _Therefore, while testing and locally compiling be sure to remember this and not try to install from a git cloned copy of node-oracledb that you have compiled for._
By the way, you're going to want to reject any of the files within the bin folder. Also, within package.json change the binary.host path so
bchr02
is changed tooracle
andprebuild_support
is changes tomaster
cc: #18
Signed-off-by: Bill Christo [email protected]