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 filenam

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 t

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 contains the names separated by spaces. >

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 end-of-line character

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 for

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 space