Issue 89: Adding new sync_method - fsevents

[vasu-dasari]

Issue has been detailed at: #89

Currently sync polls directories and files for any changes and recompile the changed files and loads them. When sync starts monitoring more projects under one umbrella CPU can get overwhelmed with this polling. it will be good if sync looks for file system changes notifications and act upon them.

There is a project synrc/fs which provides filesystem change events. This supports Mac, Windows and Linux platforms.

This commit integrates sync with fs. Here is what it does:

1. Added a new app evnironment variable, sync_method. User can specifiy this to scanner or fsevents. By default it will be scanner, which would retain current behavior. If user specifies fsevents, new logic kicks in.
2. discover_src_dirs, will discover all source directories as requested by user. and starts fs process for monited directories.
3. When a source file or a header file changes, sync gets {fs,file_event} event. And it will handle the changed files a second later so that it can suppress/collect all events on a file.
4. Make sure no timers are active in stable state.

Signed-off-by: Vasu Dasari [email protected]

[choptastic]

This looks great! Thanks. I'll be giving this a more thorough review this weekend or next week, but on first glance, it looks solid.

…

-Jesse

On Mon, Jul 13, 2020 at 8:36 AM Vasu Dasari ***@***.***> wrote: Issue has been detailed at: #89 <#89> Currently sync polls directories and files for any changes and recompile the changed files and loads them. When sync starts monitoring more projects under one umbrella CPU can get overwhelmed with this polling. it will be good if sync looks for file system changes notifications and act upon them. There is a project synrc/fs which provides filesystem change events. This supports Mac, Windows and Linux platforms. This commit integrates sync with fs. Here is what it does: 1. Added a new app evnironment variable, sync_method. User can specifiy this to scanner or fsevents. By default it will be scanner, which would retain current behavior. If user specifies fsevents, new logic kicks in. 2. discover_src_dirs, will discover all source directories as requested by user. and starts fs process for monited directories. 3. When a source file or a header file changes, sync gets {fs,file_event} event. And it will handle the changed files a second later so that it can suppress/collect all events on a file. 4. Make sure no timers are active in stable state. Signed-off-by: Vasu Dasari ***@***.*** ------------------------------ You can view, comment on, or merge this pull request online at: #90 Commit Summary - Issue 89: Adding new sync_method - fsevents File Changes - *M* rebar.config <https://github.com/rustyio/sync/pull/90/files#diff-31d7a50c99c265ca2793c20961b60979> (4) - *M* src/sync.app.src <https://github.com/rustyio/sync/pull/90/files#diff-086d23ad68d8de69e4a661ecaacbfe5a> (5) - *M* src/sync_scanner.erl <https://github.com/rustyio/sync/pull/90/files#diff-f73bf96f96dc8e6256bb56c464860a46> (167) Patch Links: - https://github.com/rustyio/sync/pull/90.patch - https://github.com/rustyio/sync/pull/90.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#90>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AABNRHCKXVNL2P7EILRMHZDR3MEXDANCNFSM4OYPXI3Q> .

-- Jesse Gumm Owner, Sigma Star Systems 414.940.4866 || sigma-star.com || @jessegumm

[vasu-dasari]

Thanks