Re: [PATCH v2 0/6] Fast git status via a file system watcher

2017-05-31 Thread Ævar Arnfjörð Bjarmason
On Wed, May 31, 2017 at 3:37 PM, Ben Peart wrote: > On 5/31/2017 3:59 AM, Christian Couder wrote: >> >> On Thu, May 25, 2017 at 3:55 PM, Ben Peart wrote: >>> >>> >>> On 5/24/2017 6:54 AM, Christian Couder wrote: > > > A new git hook

Re: [PATCH v2 0/6] Fast git status via a file system watcher

2017-05-31 Thread Ben Peart
On 5/31/2017 3:59 AM, Christian Couder wrote: On Thu, May 25, 2017 at 3:55 PM, Ben Peart wrote: On 5/24/2017 6:54 AM, Christian Couder wrote: A new git hook (query-fsmonitor) must exist and be enabled (core.fsmonitor=true) that takes a time_t formatted as a string and

Re: [PATCH v2 0/6] Fast git status via a file system watcher

2017-05-31 Thread Christian Couder
On Thu, May 25, 2017 at 3:55 PM, Ben Peart wrote: > > On 5/24/2017 6:54 AM, Christian Couder wrote: >>> >>> A new git hook (query-fsmonitor) must exist and be enabled >>> (core.fsmonitor=true) that takes a time_t formatted as a string and >>> outputs to stdout all files that

Re: [PATCH v2 0/6] Fast git status via a file system watcher

2017-05-31 Thread Christian Couder
Yeah, they could be encoded in the integration script, but it could be better if it was possible to just configure a generic command line. For example if the directory that should be watched for filesystem changes could be passed as well as the time since the last changes,

Re: [PATCH v2 0/6] Fast git status via a file system watcher

2017-05-30 Thread Ben Peart
On 5/30/2017 4:33 PM, Christian Couder wrote: On Tue, May 30, 2017 at 8:05 PM, Ben Peart wrote: On 5/27/2017 2:57 AM, Christian Couder wrote: On Thu, May 25, 2017 at 3:55 PM, Ben Peart wrote: On 5/24/2017 6:54 AM, Christian Couder wrote:

Re: [PATCH v2 0/6] Fast git status via a file system watcher

2017-05-30 Thread Christian Couder
On Tue, May 30, 2017 at 8:05 PM, Ben Peart wrote: > > > On 5/27/2017 2:57 AM, Christian Couder wrote: >> >> On Thu, May 25, 2017 at 3:55 PM, Ben Peart wrote: >>> >>> >>> >>> On 5/24/2017 6:54 AM, Christian Couder wrote: > > > Design >

Re: [PATCH v2 0/6] Fast git status via a file system watcher

2017-05-30 Thread Ben Peart
On 5/27/2017 2:57 AM, Christian Couder wrote: On Thu, May 25, 2017 at 3:55 PM, Ben Peart wrote: On 5/24/2017 6:54 AM, Christian Couder wrote: Design ~~ A new git hook (query-fsmonitor) must exist and be enabled (core.fsmonitor=true) that takes a time_t formatted

Re: [PATCH v2 0/6] Fast git status via a file system watcher

2017-05-27 Thread Christian Couder
On Thu, May 25, 2017 at 3:55 PM, Ben Peart wrote: > > > On 5/24/2017 6:54 AM, Christian Couder wrote: >>> >>> Design >>> ~~ >>> >>> A new git hook (query-fsmonitor) must exist and be enabled >>> (core.fsmonitor=true) that takes a time_t formatted as a string and >>>

Re: [PATCH v2 0/6] Fast git status via a file system watcher

2017-05-25 Thread Ben Peart
On 5/24/2017 6:54 AM, Christian Couder wrote: Design ~~ A new git hook (query-fsmonitor) must exist and be enabled (core.fsmonitor=true) that takes a time_t formatted as a string and outputs to stdout all files that have been modified since the requested time. Is there a reason why

Re: [PATCH v2 0/6] Fast git status via a file system watcher

2017-05-24 Thread Christian Couder
> Design > ~~ > > A new git hook (query-fsmonitor) must exist and be enabled > (core.fsmonitor=true) that takes a time_t formatted as a string and > outputs to stdout all files that have been modified since the requested > time. Is there a reason why there is a new hook, instead of a

[PATCH v2 0/6] Fast git status via a file system watcher

2017-05-18 Thread Ben Peart
Changes from V1 include: - add 64 bit endianness helper get_be64 in compat/bswap.h - switch to using get_be helpers when reading index extension - fix leak of strbuf in refresh_by_fsmonitor - rename update_istate and clean up parameter list - make t/t7519-status-fsmonitor.sh executable -