Skip to content
This repository was archived by the owner on Mar 4, 2020. It is now read-only.

fix: [deps/uv] fsevents: fix file event reporting (backport: v12.x)#100

Merged
jkleinsc merged 1 commit into
electron-node-v12.xfrom
backport_fsevents_12_x
May 29, 2019
Merged

fix: [deps/uv] fsevents: fix file event reporting (backport: v12.x)#100
jkleinsc merged 1 commit into
electron-node-v12.xfrom
backport_fsevents_12_x

Conversation

@deepak1556

@deepak1556 deepak1556 commented May 27, 2019

Copy link
Copy Markdown
Member

Backports libuv/libuv#2313

Commit 2d2af38 ("fsevents: really watch files with fsevents on macos
10.7+") from last November introduced a regression where events that
were previously reported as UV_RENAME were now reported as UV_CHANGE.
This commit rectifies that.

Fixes: nodejs/node#27869
@deepak1556 deepak1556 requested a review from a team May 27, 2019 21:53

@ryzokuken ryzokuken left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jkleinsc jkleinsc changed the title [deps/uv] fsevents: fix file event reporting (backport: v12.x) fix: [deps/uv] fsevents: fix file event reporting (backport: v12.x) May 29, 2019
@jkleinsc jkleinsc merged commit ec5cc73 into electron-node-v12.x May 29, 2019
@jkleinsc jkleinsc deleted the backport_fsevents_12_x branch May 29, 2019 15:59
deepak1556 added a commit to electron/electron that referenced this pull request Oct 3, 2019
This reverts the patch from electron/node#100
which never got merged due to reasons outlined in libuv/libuv#2313

* Adds new patches that backports libuv/libuv#2459
  and libuv/libuv#2460

Based on nodejs/node#29460
deepak1556 added a commit to electron/electron that referenced this pull request Oct 4, 2019
This reverts the patch from electron/node#100
which never got merged due to reasons outlined in libuv/libuv#2313

* Adds new patches that backports libuv/libuv#2459
  and libuv/libuv#2460

Based on nodejs/node#29460
deepak1556 added a commit to electron/electron that referenced this pull request Oct 4, 2019
This reverts the patch from electron/node#100
which never got merged due to reasons outlined in libuv/libuv#2313

* Adds new patches that backports libuv/libuv#2459
  and libuv/libuv#2460

Based on nodejs/node#29460
deepak1556 added a commit to electron/electron that referenced this pull request Oct 4, 2019
This reverts the patch from electron/node#100
which never got merged due to reasons outlined in libuv/libuv#2313

* Adds new patches that backports libuv/libuv#2459
  and libuv/libuv#2460

Based on nodejs/node#29460
deepak1556 added a commit to electron/electron that referenced this pull request Oct 7, 2019
This reverts the patch from electron/node#100
which never got merged due to reasons outlined in libuv/libuv#2313

* Adds new patches that backports libuv/libuv#2459
  and libuv/libuv#2460

Based on nodejs/node#29460
deepak1556 added a commit to electron/electron that referenced this pull request Oct 7, 2019
This reverts the patch from electron/node#100
which never got merged due to reasons outlined in libuv/libuv#2313

* Adds new patches that backports libuv/libuv#2459
  and libuv/libuv#2460

Based on nodejs/node#29460
deepak1556 added a commit to electron/electron that referenced this pull request Oct 7, 2019
This reverts the patch from electron/node#100
which never got merged due to reasons outlined in libuv/libuv#2313

* Adds new patches that backports libuv/libuv#2459
  and libuv/libuv#2460

Based on nodejs/node#29460
deepak1556 added a commit to electron/electron that referenced this pull request Oct 7, 2019
This reverts the patch from electron/node#100
which never got merged due to reasons outlined in libuv/libuv#2313

* Adds new patches that backports libuv/libuv#2459
  and libuv/libuv#2460

Based on nodejs/node#29460
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants