Hello Gunnar!

>> The first one seems better to me.
>
> I use dir_exclude_mtime for now, ok?
Ok.

> When putting the option definition in options.c, options.h and options.dox it
> builds but it don't accept the option.
Maybe the name is too long?

>> Option name 'dir_exclude_mtime=no' unknown.
>
> Code in options.c:
>
>>      [OPT__DIR_EXCLUDE_MTIME] = {
>>              .name="dir_exclude_mtime", .i_val=OPT__NO,
>>              .parse=opt___string2val, .parm=opt___yes_no,
>>      },
>
> What do I miss here?
>
> When I looked over the code it seems to me that I need to add the exclude 
> mtime
> code for every part in fsvs where also the filter option applies.
No, it should be enough to add that either to st__action() (to just avoid 
showing them,
because on commit it might make sense again), or to ac__dispatch() - that 
should be used
everywhere. (?)

> It looks like repeating the filter behaviour again. I only wonder where to 
> start
> because I don't find a function to extend with the exclude check. Can you give
> me a small hint where to start?
Do the above hints help?


Regards,

Phil

-- 
Versioning your /etc, /home or even your whole installation?
             Try fsvs (fsvs.tigris.org)!

------------------------------------------------------
http://fsvs.tigris.org/ds/viewMessage.do?dsForumId=3923&dsMessageId=2446619

To unsubscribe from this discussion, e-mail: [[email protected]].

Reply via email to