[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Issue 298: Remove parameter /notempfile

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: 2007-03-21 19:43:44 CET

Tobias Schäfer wrote:

> this patch finally implements issue 298. I've tested it, but please review.
>
> All details can be found here:
> http://issues.tortoisesvn.net/?do=details&task_id=298

> Index: doc/source/en/TortoiseSVN/tsvn_app_automation.xml
> ===================================================================
> --- doc/source/en/TortoiseSVN/tsvn_app_automation.xml (revision 9012)
> +++ doc/source/en/TortoiseSVN/tsvn_app_automation.xml (working copy)
> @@ -43,11 +43,10 @@
> several paths, separated by a <literal>*</literal> character.
> </para>
> <para>
> - Since TortoiseSVN uses temporary files to pass multiple arguments between
> - the shell extension and the main program, you <emphasis>must</emphasis>
> - add the <option>/notempfile</option> parameter! If you don't, the command
> - won't work and the file you pass with the <option>/path</option> parameter
> - will be deleted!
> + TortoiseSVN uses temporary files to pass multiple arguments between
> + the shell extension and the main program. Since TortoiseSVN 1.5.0 the

Instead of "since TSVN 1.5.0", I think it's better to write:
"From TortoiseSVN 1.5.0 on and later, ..."

> + IDS_ERR_INVALIDPATH "The parameters /path and /pathfile are mutally exclusive."

Maybe better this way:
"The parameters '/path' and '/pathfile' are mutally exclusive.\nYou must
only specify one of them."

(also note: 'mutually', not 'mutally')

> Index: src/TortoiseShell/ContextMenu.cpp
> ===================================================================
> --- src/TortoiseShell/ContextMenu.cpp (revision 9012)
> +++ src/TortoiseShell/ContextMenu.cpp (working copy)
> @@ -1134,9 +1134,12 @@
> CRegStdString tortoiseMergePath(_T("Software\\TortoiseSVN\\TMergePath"), _T("TortoiseMerge.exe"), false, HKEY_LOCAL_MACHINE);
>
> //TortoiseProc expects a command line of the form:
> + //"/command:<commandname> /pathfile:<path> /revstart:<revisionstart> /revend:<revisionend> /deletefile

/deletefile?
Isn't it /deletefilepath?

> + // or
> //"/command:<commandname> /path:<path> /revstart:<revisionstart> /revend:<revisionend>
> - //path is either a path to a single file/directory for commands which only act on single files (log, checkout, ...)
> - //or a path to a temporary file which contains a list of filepaths
> + //
> + //* path is a path to a single file/directory for commands which only act on single files (log, checkout, ...)

single files -> single items
(it acts on files and folders, not just files)

Other than those little nitpicks, the patch is fine!

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org
Received on Wed Mar 21 19:44:04 2007

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.