Skip to content

Commit

Permalink
fix(NODE-6253): use runtime linking against system kerberos libraries…
Browse files Browse the repository at this point in the history
… by default (#188)
  • Loading branch information
nbbeeken authored Jul 10, 2024
1 parent ec3ab7a commit 04044d2
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .evergreen/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ tasks:
commands:
- func: run tests ubuntu
vars:
GYP_DEFINES: kerberos_use_rtld=true
GYP_DEFINES: kerberos_use_rtld=false
NPM_OPTIONS: --build-from-source
- name: run-prebuild
commands:
Expand Down
6 changes: 3 additions & 3 deletions .github/scripts/build.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ async function parseArguments() {
const pkg = JSON.parse(await fs.readFile(resolveRoot('package.json'), 'utf8'));

const options = {
'kerberos_use_rtld': { type: 'boolean', default: false },
'kerberos_use_rtld': { type: 'boolean', default: true },
help: { short: 'h', type: 'boolean', default: false }
};

Expand Down Expand Up @@ -66,8 +66,8 @@ async function buildBindings(args, pkg) {
// it will also produce `./prebuilds/kerberos-vVERSION-napi-vNAPI_VERSION-OS-ARCH.tar.gz`.

let gypDefines = process.env.GYP_DEFINES ?? '';
if (args.kerberos_use_rtld) {
gypDefines += ' kerberos_use_rtld=true';
if (!args.kerberos_use_rtld) {
gypDefines += ' kerberos_use_rtld=false';
}

gypDefines = gypDefines.trim();
Expand Down
2 changes: 1 addition & 1 deletion binding.gyp
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
],
'variables': {
'ARCH': '<(host_arch)',
'kerberos_use_rtld%': 'false'
'kerberos_use_rtld%': 'true'
},
'xcode_settings': {
'GCC_ENABLE_CPP_EXCEPTIONS': 'YES',
Expand Down

0 comments on commit 04044d2

Please sign in to comment.