Re: /notempfile

On Monday 25 September 2006 09:04, Gavin Lambert wrote:

Let me summarize this tread:

1. The automation interface which is currently
/command:update /path="d:\temp\wc\" /notempfile
changes to
/command:update /path="d:\temp\wc\"
Since the switch /notempfile is not evaluated there won't be a change in
behavior if it is passed. Users using the automation interface cannot
forget /tempfile anymore.

2. The interface for the shell extension is currently
/command:update /path="%temp%\svn1234.tmp"

You propose to change it to:
/command:update /tempfile="%temp%\svn1234.tmp"

If we change the following I agree:
- rename /tempfile to /pathfile to make it clear that /path and /pathfile
are mutally exclusive.
- add a switch to delete the file specified by /pathfile by tortoiseproc
instead of always deleting it. This allows the calling application to
delete the file after having called tortoiseproc.

This is my proposal based on your ideas:
/command:update /pathfile="%temp%\svn1234.tmp" /deletepathfile

This change will break all calls which currently don't specify /notempfile.
All calls from the shell extension will be changed to reflect this new
interface.

What should tortoiseproc do if both /path and /pathfile are given? I would
add both path lists to a single path list internally.

Pathfile is currently a file in unicode encoding. To make it easier to pass
a list of files we should also support simple ascii encoding. This is a
different issue though. This is to support what you proposed with the
parameter /listfile

Tobias

-- 
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.net
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: users-help@tortoisesvn.tigris.org