[tools-issues] [Issue 66155] ~/.cwsrc parsing broken .. .
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=66155 User hr changed the following: What|Old value |New value Status|VERIFIED |CLOSED --- Additional comments from [EMAIL PROTECTED] Tue Sep 5 06:05:20 -0700 2006 --- Close issue. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[tools-issues] [Issue 66155] ~/.cwsrc parsing broken . ..
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=66155 --- Additional comments from [EMAIL PROTECTED] Wed Jun 7 08:19:44 -0700 2006 --- 1) You're correct. That was a last minute change that I did not review properly sorry about that. 2) blah. This brings back unpleasant memories of why I haven't used perl in a while. Thanks for the review. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[tools-issues] [Issue 66155] ~/.cwsrc parsing broken . ..
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=66155 User hjs changed the following: What|Old value |New value Status|RESOLVED |VERIFIED --- Additional comments from [EMAIL PROTECTED] Wed Jun 7 07:37:09 -0700 2006 --- looks fine for me now ->mmeeks: you might want to have a look anyhow... - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[tools-issues] [Issue 66155] ~/.cwsrc parsing broken . ..
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=66155 User hr changed the following: What|Old value |New value Status|NEW |RESOLVED Resolution| |FIXED Target milestone|--- |OOo 2.0.4 --- Additional comments from [EMAIL PROTECTED] Wed Jun 7 05:55:43 -0700 2006 --- Fixed in ause056 - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[tools-issues] [Issue 66155] ~/.cwsrc parsing broken . ..
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=66155 --- Additional comments from [EMAIL PROTECTED] Wed Jun 7 02:04:42 -0700 2006 --- > Ause - the old nitpicker - found a small glitch in one of the REs (we spend 20 > min to understand it :-) ... and I found another one :-) Perl - a lovely write-only language :-) > I told you we are nitpicking ... :-) Nah these are good nits; any chance you can fix as you commit ? :-) - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[tools-issues] [Issue 66155] ~/.cwsrc parsing broken . ..
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=66155
--- Additional comments from [EMAIL PROTECTED] Tue Jun 6 10:30:51 -0700
2006 ---
Ause - the old nitpicker - found a small glitch in one of the REs (we spend 20
min to understand it :-) ... and I found another one :-)
First (the line with the first + sign in the patch): The square backets in the
RE are exchanged for round ones. This isn't right, is it? This either doesn't
match (most of the cases) or match in funny ways. See attached snippet.
Secondly and here we are nitpicking :-):
Please exchange the mentioned round brackets to square brackets in the "if" line
of the snippet . Now, if there is a whitespace before the quote sign on both
ends of the value string ie,
foo= "bar "
then the RE which kills the surrounding quotes of the value will also kill the
last space inside the quote which still belongs to the value.
my $str = 'BAR1= "foo "';
print "$str\n";
if ( $str =~ m/(\w(\w\d)+)=(.*)$/ ) {
my $var = $1;
my $val = $2;
print "$var\n";
print "$val\n";
$val =~ s/(\s*['"])?(.*)\1\s*$/\2/;
print "x${val}x\n";
}
I told you we are nitpicking ... :-)
-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
[tools-issues] [Issue 66155] ~/.cwsrc parsing broken . ..
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=66155 --- Additional comments from [EMAIL PROTECTED] Tue Jun 6 08:28:54 -0700 2006 --- Thank you. I'd also suggest a warning that failure to connect to server is likely related to an invalid password. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[tools-issues] [Issue 66155] ~/.cwsrc parsing broken . ..
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=66155 --- Additional comments from [EMAIL PROTECTED] Tue Jun 6 06:35:39 -0700 2006 --- Hi Michael, no problem, I'll attach it to the next upcoming CWS. - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[tools-issues] [Issue 66155] ~/.cwsrc parsing broken . ..
To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=66155 User mmeeks changed the following: What|Old value |New value Attachment is patch| |Created an attachment | |(id=36943) patch --- Additional comments from [EMAIL PROTECTED] Tue Jun 6 03:13:41 -0700 2006 --- Created an attachment (id=36943) patch - Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
