Re: [patch] Synchronization of the panels (fwd)

[Miguel de Icaza]

Hello,

 This is an updated version of the dnotify patch from Andrey Siver.

The patch still needs a lot of work, for example, why do we have an
include file at all with the code?  (The core of the code lives in
sync.inc).

Configure patches, fall back patches are missing as well.

Miguel.
___
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel

Re: [patch] Synchronization of the panels

[Miguel de Icaza]

Hello,

  Please consider my patch to synchronize the panels with hard drives
  state.
   
 note that dnotify prevents a watched directory from being unmounted (one
 of the reasons why it is being replaced). that means one has to leave
 the dir in question in both panels. no problem as such, but it might
 lead to surprises ...

It could be argued that if mc is watching a directory, mc is in that
directory so unmounting should not be an issue at this point, but
adopting something like inotify in the long run seems like a better
option.

The other problem with the patch is that it lacks the configure magic to
work on non-dnotify systems, and also lacks support for not aborting if
the code is not available in the current kernel.

The patch is a nice one, but it needs to be productized.


___
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel