You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by "C. Michael Pilato" <cm...@collab.net> on 2005/01/04 13:55:40 UTC

Re: svn commit: r12545 - trunk/tools/hook-scripts/mailer

maxb@tigris.org writes:

> +# A revision will be reported on if any changed-path matches the for_paths
> +# option. If only some of the changed-paths of a revision match, this
> +# controls the behaviour for the non-matching paths. Possible values are:
> +#   show-all: (Default) Show in both summary and diffs.
> +#   show-summary: Show the changed paths in the summary, but omit the diffs.
> +#   hide: Show nothing more than a note saying "and changes in other areas"
> +excluded_paths = show-all

Minor nit.  If this new option is about showing excluded paths, why
not use:

   show_excluded_paths: all | summary | none

?

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: svn commit: r12545 - trunk/tools/hook-scripts/mailer

Posted by "C. Michael Pilato" <cm...@collab.net>.
"Max Bowsher" <ma...@ukf.net> writes:

> > Minor nit.  If this new option is about showing excluded paths, why
> > not use:
> >
> >   show_excluded_paths: all | summary | none
> 
> I revised the option to show_nonmatching_paths.

Me saw, and me likes.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: svn commit: r12545 - trunk/tools/hook-scripts/mailer

Posted by Max Bowsher <ma...@ukf.net>.
C. Michael Pilato wrote:
> maxb@tigris.org writes:
>
>> +# A revision will be reported on if any changed-path matches the 
>> for_paths
>> +# option. If only some of the changed-paths of a revision match, this
>> +# controls the behaviour for the non-matching paths. Possible values 
>> are:
>> +#   show-all: (Default) Show in both summary and diffs.
>> +#   show-summary: Show the changed paths in the summary, but omit the 
>> diffs.
>> +#   hide: Show nothing more than a note saying "and changes in other 
>> areas"
>> +excluded_paths = show-all
>
> Minor nit.  If this new option is about showing excluded paths, why
> not use:
>
>   show_excluded_paths: all | summary | none

I revised the option to show_nonmatching_paths.

Max.




---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org