Skip to content

Commit c9bb582

Browse files
author
Stephen Parente
committed
Leverage Makefile install to move files in the correct directory
1 parent 844dee1 commit c9bb582

File tree

4 files changed

+9
-40
lines changed

4 files changed

+9
-40
lines changed

binding.gyp

+3-3
Original file line numberDiff line numberDiff line change
@@ -64,16 +64,16 @@
6464
'OS=="linux"',
6565
{
6666
"libraries": [
67-
"<(module_root_dir)/build/Release/librdkafka++.so",
68-
"-Wl,-rpath=<(module_root_dir)/build/Release",
67+
"<(module_root_dir)/build/deps/librdkafka++.so",
68+
"-Wl,-rpath=<(module_root_dir)/build/deps",
6969
],
7070
}
7171
],
7272
[
7373
'OS=="mac"',
7474
{
7575
"libraries": [
76-
"<(module_root_dir)/build/Release/librdkafka++.dylib",
76+
"<(module_root_dir)/build/deps/librdkafka++.dylib",
7777
],
7878
}
7979
]

configure

+4-1
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,11 @@
55

66
# This does not get run on windows which uses the build in solutions file
77

8+
# Get script directory
9+
scriptdir=$(cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd)
10+
811
pushd ./deps/librdkafka &> /dev/null
912

10-
./configure $*
13+
./configure --prefix="${scriptdir}/build/deps" --libdir="${scriptdir}/build/deps" $*
1114

1215
popd &> /dev/null

deps/librdkafka.gyp

+1-34
Original file line numberDiff line numberDiff line change
@@ -60,47 +60,14 @@
6060
},
6161
},
6262
{
63-
"conditions": [
64-
[
65-
'OS=="mac"',
66-
{
67-
'copies': [
68-
{
69-
'files': [
70-
'<(module_root_dir)/deps/librdkafka/src-cpp/librdkafka++.dylib',
71-
'<(module_root_dir)/deps/librdkafka/src-cpp/librdkafka++.1.dylib',
72-
'<(module_root_dir)/deps/librdkafka/src/librdkafka.dylib',
73-
'<(module_root_dir)/deps/librdkafka/src/librdkafka.1.dylib'
74-
],
75-
'destination': '<(module_root_dir)/build/Release'
76-
}
77-
],
78-
},
79-
{
80-
'copies': [
81-
{
82-
'files': [
83-
'<(module_root_dir)/deps/librdkafka/src-cpp/librdkafka++.so',
84-
'<(module_root_dir)/deps/librdkafka/src-cpp/librdkafka++.so.1',
85-
'<(module_root_dir)/deps/librdkafka/src/librdkafka.so',
86-
'<(module_root_dir)/deps/librdkafka/src/librdkafka.so.1',
87-
'<(module_root_dir)/deps/librdkafka/src-cpp/librdkafka++.a',
88-
'<(module_root_dir)/deps/librdkafka/src/librdkafka.a',
89-
],
90-
'destination': '<(module_root_dir)/build/Release'
91-
}
92-
],
93-
}
94-
]
95-
],
9663
"actions": [
9764
{
9865
"action_name": "build_dependencies",
9966
"inputs": [
10067
"<(module_root_dir)/deps/librdkafka/config.h",
10168
],
10269
"action": [
103-
"make", "-C", "<(module_root_dir)/deps/librdkafka", "libs"
70+
"make", "-C", "<(module_root_dir)/deps/librdkafka", "libs", "install"
10471
],
10572
"conditions": [
10673
[

util/configure.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,7 @@ var fs = require('fs');
66
var path = require('path');
77

88
var baseDir = path.resolve(__dirname, '../');
9-
var releaseDir = path.join(baseDir, 'build', 'Release');
10-
var debugDir = path.join(baseDir, 'build', 'Debug');
9+
var releaseDir = path.join(baseDir, 'build', 'deps');
1110

1211
var isWin = /^win/.test(process.platform);
1312

0 commit comments

Comments
 (0)