add :recursive option

[avalanche123]

Add ability to prevent recursive directory scan by passing :recursive => false to Listen#to

Let me know what you guys think!

[coveralls]

Coverage remained the same when pulling 6e0a988 on avalanche123:disable-recursive-listen into 478f934 on guard:master.

[thibaudgg]

Hi Bulat, thanks for the PR!

I'm not sure this would be a good idea because recursive option is only used by the polling adapter, what do you want to achieve here? Maybe using ignore option would be more appropriate?

[avalanche123]

Unfortunately even when using :ignore option, listen recursively scans all
directories. This prevents recursive scan and allows to listen to changes
of files in target directory only, ignoring subdirectories. This feature is
listed in readme as planned but not implemented.

Bulat

On Jul 15, 2013, at 4:53, Thibaud Guillaume-Gentil [email protected]
wrote:

Hi Bulat, thanks for the PR!

I'm not sure this would be a good idea because recursive option is only
used by the polling adapter, what do you want to achieve here? Maybe using
ignore option would be more appropriate?

—
Reply to this email directly or view it on
GitHubhttps://github.com//pull/128#issuecomment-20964850
.

[thibaudgg]

Yeah, the "Non-recursive directory scanning" pending feature could be implemented in 2.x, but not with the recursive => false option. I was more thinking in something like scan_deep_level option (#111)... will see.

[avalanche123]

Got it, feel free to close

Bulat

On Jul 16, 2013, at 0:41, Thibaud Guillaume-Gentil [email protected]
wrote:

Yeah, the "Non-recursive directory scanning" pending feature could be
implemented in 2.x, but not with the recursive => false option. I was more
thinking in something like scan_deep_level option
(#111#111)...
will see.

—
Reply to this email directly or view it on
GitHubhttps://github.com//pull/128#issuecomment-21026048
.