Issue 30203 - Crashreporter: -crashreportenv:ERRORREPORT_... parameter needs possibility for SPACES
Summary: Crashreporter: -crashreportenv:ERRORREPORT_... parameter needs possibility fo...
Status: CLOSED WONT_FIX
Alias: None
Product: General
Classification: Code
Component: code (show other issues)
Version: 680m42
Hardware: Sun Solaris
: P3 Trivial (vote)
Target Milestone: AOO Later
Assignee: gregor.hartmann
QA Contact: issues@framework
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-06-14 15:42 UTC by jogi
Modified: 2005-08-22 13:51 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description jogi 2004-06-14 15:42:57 UTC 
On Win32 we put the -crashreportenv:ERRORREPORT_*-parameters and the value of it
in "" (ASCIIchar 34). This method does not work on Unix operating systems
(Solaris and Linux).
Bug: We need a possibility to have SPACES like ../dir/dir rest/file.txt in these
-crashreportenv:ERRORREPORT_*-parameters.
Comment 1 hennes.rohling 2004-06-15 15:31:43 UTC 
The code in Testtool removes the " signs from the command line. Does not occur
on windows because Windows process execution code adds " if spaces occur in
parameters. This is neccessary because command line is forwarded as one string
on Windows.
Comment 2 gregor.hartmann 2004-06-17 08:35:28 UTC 
The problem is that String::GetQuotedToken leaves the quotes in instead of
removing them.
So they will have to get removed.
Windows seems to strip them anyways but UNIX doesn`t

 if ( (aToken.GetChar(0) == '"') && (aToken.GetChar(aToken.Len()-1) == '"') )
            aToken = aToken.Copy(1,aToken.Len()-2);
Comment 3 b.osi.ooo 2004-09-09 09:22:32 UTC 
since we switched initialisation of crashreporter to environment varibles, this
isn't needed for OOo 2.0. -> later
Comment 4 gregor.hartmann 2005-08-19 10:37:25 UTC 
.
Comment 5 jogi 2005-08-22 13:50:28 UTC 
QA Automation team meeting: Won't be fixed because command line parameter not
being used by TT environment anymore.
Comment 6 jogi 2005-08-22 13:50:50 UTC 
verified
Comment 7 jogi 2005-08-22 13:51:13 UTC 
closing