Re: [PATCH 1/2] Support for $FILENAMES in tool definitions

2016-06-28 Thread Alex Riesen
Jakub Narębski, Mon, Jun 27, 2016 19:28:07 +0200: > On 2016-06-27, Alex Riesen wrote: > > > This adds a FILENAMES environment variable, which contains the repository > > pathnames of all selected files the list. > > The variable contains the names separated by spaces. > > Why not separate

Re: [PATCH 1/2] Support for $FILENAMES in tool definitions

2016-06-28 Thread Alex Riesen
Johannes Schindelin, Mon, Jun 27, 2016 17:49:30 +0200: > On Mon, 27 Jun 2016, Alex Riesen wrote: > > > This adds a FILENAMES environment variable, which contains the repository > > pathnames of all selected files the list. > > The variable contains the names separated by spaces. > > > > Similar

Re: [PATCH 1/2] Support for $FILENAMES in tool definitions

2016-06-27 Thread Jakub Narębski
On 27 June 2016 at 19:53, Junio C Hamano wrote: > Jakub Narębski writes: > >> On 2016-06-27, Alex Riesen wrote: >> >>> This adds a FILENAMES environment variable, which contains the repository >>> pathnames of all selected files the list. >>> The variable

Re: [PATCH 1/2] Support for $FILENAMES in tool definitions

2016-06-27 Thread Junio C Hamano
Jakub Narębski writes: > On 2016-06-27, Alex Riesen wrote: > >> This adds a FILENAMES environment variable, which contains the repository >> pathnames of all selected files the list. >> The variable contains the names separated by spaces. > > Why not separate filenames with

Re: [PATCH 1/2] Support for $FILENAMES in tool definitions

2016-06-27 Thread Jakub Narębski
On 2016-06-27, Alex Riesen wrote: > This adds a FILENAMES environment variable, which contains the repository > pathnames of all selected files the list. > The variable contains the names separated by spaces. Why not separate filenames with end-of-line character (LF)? It would still be broken

Re: [PATCH 1/2] Support for $FILENAMES in tool definitions

2016-06-27 Thread Johannes Schindelin
Hi Alex, On Mon, 27 Jun 2016, Alex Riesen wrote: > This adds a FILENAMES environment variable, which contains the repository > pathnames of all selected files the list. > The variable contains the names separated by spaces. > > Similar to the FILENAME it is broken yet, if the names contain

[PATCH 1/2] Support for $FILENAMES in tool definitions

2016-06-27 Thread Alex Riesen
This adds a FILENAMES environment variable, which contains the repository pathnames of all selected files the list. The variable contains the names separated by spaces. Similar to the FILENAME it is broken yet, if the names contain spaces. Note that the file marked and diffed immediately after