Skip to content

Conversation

imlucas
Copy link
Contributor

@imlucas imlucas commented Dec 14, 2016

Compass has a defined an electron-prebuilt version of 1.2.8.
hadron-build previously required ^1.4.12 and this would cause
unexpected behavior only when developing Compass. In this case, we were compiling native add-on’s during npm install targeting the abi for
1.2.8 but they would suddenly start to fail as hadron-build develop
would use 1.4.12 which is abi incompatible.

Developers

If you notice incompatible abi version messages in the devtools console
or suddenly you can’t connect to mongodb instances using auth or via
ssh tunnel, please run the below to dedupe the electron-prebuilt
versions currently installed in your environment:

npm install;
npm dedupe;

Upstream

See mongodb-js/hadron-build#49

Compass has a defined an electron-prebuilt version of `1.2.8`.
`hadron-build` previously required `^1.4.12` and this would cause
unexpected behavior only when developing Compass. In this case, we were
compiling native add-on’s  during `npm install` targeting the abi for
`1.2.8` but they would suddenly start to fail as `hadron-build develop`
would use `1.4.12` which is abi incompatible.

## Developers

If you notice incompatible abi version messages in the devtools console
or suddenly you can’t connect to mongodb instances using auth or via
ssh tunnel, please run the below to dedupe the electron-prebuilt
versions currently installed in your environment:

```bash
npm install;
npm dedupe;
```
Copy link
Contributor

@pzrq pzrq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for my SSH Tunnel / COMPASS-503 which is good, so I guess just waiting on Travis, thanks @imlucas

@pzrq pzrq merged commit a4746ae into master Dec 14, 2016
@pzrq pzrq deleted the dedupe-electron-prebuilt branch December 14, 2016 02:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants