[l10n-issues] [Issue 109378] transex3: set path to gsi check in localize.pl

2010-05-11 Thread pmladek
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=109378


User pmladek changed the following:

What|Old value |New value

  Status|RESOLVED  |VERIFIED





--- Additional comments from pmla...@openoffice.org Tue May 11 12:36:59 
+ 2010 ---
Verified in CWS l10ntooling18.

-
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: issues-unsubscr...@l10n.openoffice.org
For additional commands, e-mail: issues-h...@l10n.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[l10n-issues] [Issue 109378] transex3: set path to gsi check in localize.pl

2010-05-10 Thread ihi
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=109378


User ihi changed the following:

What|Old value |New value

  Status|REOPENED  |RESOLVED

  Resolution|  |FIXED





--- Additional comments from i...@openoffice.org Mon May 10 17:28:31 + 
2010 ---
ok what I did now is:
diff -r d4e30bd655ac l10ntools/scripts/localize.pl
--- a/l10ntools/scripts/localize.pl Tue Apr 13 18:26:22 2010 +0200
+++ b/l10ntools/scripts/localize.pl Mon May 10 19:21:48 2010 +0200
@@ -284,9 +284,11 @@
 foreach my $lang( keys( %{ $string_hash } ) )
 {
 my @sdf_file;
+next , if( $lang eq en-US ); 
 
-# mkdir
-my $current_l10n_file = $WIN eq TRUE ?
$l10n_file.\\$lang\\localize.sdf : $l10n_file./$lang/localize.sdf;
+mkdir $l10n_file./$lang;
+my $current_l10n_file = $l10n_file./$lang/localize.sdf;
+
 print Writing '$current_l10n_file'\n;
 if( open DESTFILE ,  $current_l10n_file ){
 
@@ -393,7 +395,7 @@
 close ( $TMPHANDLE );
 
 $command = $ENV{WRAPCMD}  if( $ENV{WRAPCMD} );
-$command .= fix_cygwin_path($ENV{SOLARVER}/$ENV{INPATH}/bin/gsicheck);
+$command .= $ENV{SOLARVER}/$ENV{INPATH}/bin/gsicheck;
 
 my $errfile = $sdffile..err;
 $command .=  -k -c -wcf $tmpfile -wef $errfile -l \\ $sdffile;

there is still something strange when there are more langs in the sdf file but I
will fix this on the master then.

I will do the OOO_VENDOR substitution as masterfix as well, as I integrated 2
l10n cws's now that would conflict

-
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: issues-unsubscr...@l10n.openoffice.org
For additional commands, e-mail: issues-h...@l10n.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[l10n-issues] [Issue 109378] transex3: set path to gsi check in localize.pl

2010-05-07 Thread pmladek
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=109378


User pmladek changed the following:

What|Old value |New value

  Status|RESOLVED  |REOPENED

  Resolution|FIXED |





--- Additional comments from pmla...@openoffice.org Fri May  7 16:04:32 
+ 2010 ---
Hmm, it does not work in CWS-l10ntooling18:

--- cut ---
$ localize -m -x -o -l cs -f ooo-build-cs.sdf 
Undefined subroutine main::fix_cygwin_path called at
/usr/src/packages/cws-l10ntooling18/l10ntooling18-d4e30bd655ac/solver/300/unxlngx6.pro/bin/localize.pl
line 396.
--- cut ---

The problem is that someone removed fix_cygwin_path definition from localize.pl.
It is not longer used there on other locations. I think that we could remove it
from the newly added line as well. We could use just:

--- cut ---
$command .= $ENV{SOLARVER}/$ENV{INPATH}/bin/gsicheck;
--- cut ---


BTW: when I do this, I see one new problem:

--- cut ---
$ localize -m -x -o -l cs -f ooo-build-cs.sdf 
Writing
'/usr/src/packages/cws-l10ntooling18/l10ntooling18-d4e30bd655ac/l10n/source/cs/localize.sdf'
Writing
'/usr/src/packages/cws-l10ntooling18/l10ntooling18-d4e30bd655ac/l10n/source/en-US/localize.sdf'
Can't open/create
'/usr/src/packages/cws-l10ntooling18/l10ntooling18-d4e30bd655ac/l10n/source'
--- cut ---

It evidently ignores the -l option and tries to write also the en-US
localization that is included in the ooo-build-cs.sdf file. Should I open
separate issue for this?

Note that the Can't open/create error is cause because the l10n/source/en-US
direcotry does not exist.

-
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: issues-unsubscr...@l10n.openoffice.org
For additional commands, e-mail: issues-h...@l10n.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[l10n-issues] [Issue 109378] transex3: set path to gsi check in localize.pl

2010-05-07 Thread pmladek
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=109378





--- Additional comments from pmla...@openoffice.org Fri May  7 16:06:46 
+ 2010 ---
Created an attachment (id=69364)
ooo-build-cs.sdf that was used for testing


-
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: issues-unsubscr...@l10n.openoffice.org
For additional commands, e-mail: issues-h...@l10n.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[l10n-issues] [Issue 109378] transex3: set path to gsi check in localize.pl

2010-05-07 Thread ihi
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=109378





--- Additional comments from i...@openoffice.org Fri May  7 16:27:10 + 
2010 ---
strange ... I will fix that on monday

-
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: issues-unsubscr...@l10n.openoffice.org
For additional commands, e-mail: issues-h...@l10n.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[l10n-issues] [Issue 109378] transex3: set path to gsi check in localize.pl

2010-04-13 Thread ihi
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=109378


User ihi changed the following:

What|Old value |New value

  Status|NEW   |RESOLVED

  Resolution|  |FIXED

Target milestone|---   |OOo 3.3





--- Additional comments from i...@openoffice.org Tue Apr 13 16:33:28 + 
2010 ---
patch applied in cws l10ntooling18

-
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: issues-unsubscr...@l10n.openoffice.org
For additional commands, e-mail: issues-h...@l10n.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[l10n-issues] [Issue 109378] transex3: set path to gsi check in localize.pl

2010-04-13 Thread ihi
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=109378





--- Additional comments from i...@openoffice.org Tue Apr 13 16:50:03 + 
2010 ---
I also added a gsicheck wrapper script for easy calling gsicheck from command 
line

-
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: issues-unsubscr...@l10n.openoffice.org
For additional commands, e-mail: issues-h...@l10n.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[l10n-issues] [Issue 109378] transex3: set path to gsi check in localize.pl

2010-02-17 Thread pmladek
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=109378
 Issue #|109378
 Summary|transex3: set path to gsicheck in localize.pl
   Component|l10n
 Version|OOO320m12
Platform|All
 URL|
  OS/Version|Linux
  Status|NEW
   Status whiteboard|
Keywords|
  Resolution|
  Issue type|PATCH
Priority|P3
Subcomponent|code
 Assigned to|pjanik
 Reported by|pmladek





--- Additional comments from pmla...@openoffice.org Wed Feb 17 17:47:41 
+ 2010 ---
This is continuation of the issue #107386.

localize.pl is not able to call gsicheck because solver/*.bin is not longer in
$PATH.

We solved this problem by setting the WRAPCMD variable in
ooo-build/bin/localize-ooo. It did not work well because the PATH was not in
quotation marks.
Also it did not solve the situation when someone wanted to call localize from
the commandline. Finally, I feel that WRAPCMD has another purpose.

I will attach a diff that call gsicheck with absolute path. I think that it
follows the current approach used in other locations. For example, the absolute
path is used even in rsc/source/prj/start.cxx.

Thanks Kálmán „KAMI” Szalai for proposing this solution.

-
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: issues-unsubscr...@l10n.openoffice.org
For additional commands, e-mail: issues-h...@l10n.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[l10n-issues] [Issue 109378] transex3: set path to gsi check in localize.pl

2010-02-17 Thread pmladek
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=109378


User pmladek changed the following:

What|Old value |New value

 Attachment is patch|  |Created an attachment (id=
|  |67884)
Proposed change.






--- Additional comments from pmla...@openoffice.org Wed Feb 17 17:49:07 
+ 2010 ---
Created an attachment (id=67884)
Proposed change.


-
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: issues-unsubscr...@l10n.openoffice.org
For additional commands, e-mail: issues-h...@l10n.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[l10n-issues] [Issue 109378] transex3: set path to gsi check in localize.pl

2010-02-17 Thread pmladek
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=109378


User pmladek changed the following:

What|Old value |New value

  CC|''|'kami_'





--- Additional comments from pmla...@openoffice.org Wed Feb 17 17:52:10 
+ 2010 ---
ihi, could you please look at it?

-
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: issues-unsubscr...@l10n.openoffice.org
For additional commands, e-mail: issues-h...@l10n.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org



[l10n-issues] [Issue 109378] transex3: set path to gsi check in localize.pl

2010-02-17 Thread pmladek
To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=109378


User pmladek changed the following:

What|Old value |New value

 Assigned to|pjanik|ihi





--- Additional comments from pmla...@openoffice.org Wed Feb 17 18:29:07 
+ 2010 ---
.

-
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: issues-unsubscr...@l10n.openoffice.org
For additional commands, e-mail: issues-h...@l10n.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org