Skip to content
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

Some dep updates #832

Merged
merged 1 commit into from
Nov 19, 2015
Merged

Some dep updates #832

merged 1 commit into from
Nov 19, 2015

Conversation

Martii
Copy link
Member

@Martii Martii commented Nov 19, 2015

  • fakes3 update
  • Retested mongoose delete op
  • Mostly bug fixes with a few new features

NOTES

  • One NPM (preexisting) warning on dev with:
npm WARN EPEERINVALID mongodb-core@1.2.21 requires a peer of kerberos@~0.0 but none was installed.

... assuming previous maintainer response of Automattic/mongoose#3362 (comment) since this appears to be working (plus we don't use kerberos other than to prevent error in compiling bins under Linux)

See those packages changelogs for details

* *fakes3* update
* Retested mongoose delete op
* Mostly bug fixes with a few new features

**NOTES**
* One NPM warning on dev with:

``` sh-session
npm WARN EPEERINVALID mongodb-core@1.2.21 requires a peer of kerberos@~0.0 but none was installed.

```

... assuming previous maintainer response of Automattic/mongoose#3362 (comment) since this appears to be working


See those packages changelogs for details
@Martii Martii added migration Use this to indicate that it may apply to an existing or announced migration. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. labels Nov 19, 2015
Martii added a commit that referenced this pull request Nov 19, 2015
@Martii Martii merged commit f0614af into OpenUserJS:master Nov 19, 2015
@Martii Martii deleted the someDepUpdates branch November 19, 2015 05:18
@Martii
Copy link
Member Author

Martii commented Nov 19, 2015

Pro log:

+-- connect-mongo@0.8.2 
| +-- lodash@3.10.1 
| `-- mongodb@2.0.48 
|   +-- es6-promise@2.1.1 
|   +-- UNMET PEER DEPENDENCY kerberos@~0.0
|   +-- mongodb-core@1.2.21 
|   `-- readable-stream@1.0.31 
|     +-- core-util-is@1.0.1 
|     +-- isarray@0.0.1 
|     `-- string_decoder@0.10.31

... appears to be connect-mongo... a little unusual that this just showed up and the (production) backend hasn't changed much.


Current npm:

$ npm -v
3.3.6

Trying npm@3.4.0 ... same warning.


Refs:

@Martii Martii added the needs mitigation Needs additional followup. label Nov 19, 2015
Martii pushed a commit to Martii/OpenUserJS.org that referenced this pull request Nov 19, 2015
* Added `try...catch` just in case it's a fatal error

**NOTES**

``` sh-session
problems": [

    "peer dep missing: kerberos@~0.0, required by mongodb-core@1.2.21"

],
```

... this shows up in the stdout as being the "error" along with the full compliment of packages installed ... see OpenUserJS#832 and parent OpenUserJS#318

*(never would have guessed this on nodejitsu... glad we aren't there anymore)*
Martii pushed a commit to Martii/OpenUserJS.org that referenced this pull request Nov 19, 2015
* This removes the warning/error on `npm install`

Ref:
* https://github.com/mongodb/node-mongodb-native/blob/2db9984f5972ff1916f46f1aab025373444aa39d/README.md#diagnosing-on-unix ... not quite what I was expecting but will still need followup when we get to *node*@6.x LTS

Applies to OpenUserJS#832 and may need further mitigation on *node* LTS upgrade next year.

Closes jdesboeufs/connect-mongo#203 *(not sure if this will close automatically but need the reference anyhow)*
@Martii Martii removed the needs mitigation Needs additional followup. label Nov 19, 2015
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. migration Use this to indicate that it may apply to an existing or announced migration.
Development

Successfully merging this pull request may close these issues.

None yet

1 participant