Issue 66700 - m173-to-be: W32 build fails in soltools/testhxx
Summary: m173-to-be: W32 build fails in soltools/testhxx
Status: CLOSED FIXED
Alias: None
Product: Build Tools
Classification: Code
Component: solenv (show other issues)
Version: current
Hardware: PC Windows, all
: P1 (highest) Trivial (vote)
Target Milestone: OOo 2.0.4
Assignee: Stephan Bergmann
QA Contact: issues@tools
URL:
Keywords:
Depends on:
Blocks: 66757
  Show dependency tree
 
Reported: 2006-06-23 20:07 UTC by quetschke
Modified: 2006-07-10 13:03 UTC (History)
4 users (show)

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


Attachments
testhxx.output (1.44 KB, patch)
2006-06-23 20:26 UTC, quetschke
no flags Details | Diff
Begin of a patch - please fill in the rest (2.00 KB, patch)
2006-06-23 21:02 UTC, quetschke
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this issue.
Description quetschke 2006-06-23 20:07:28 UTC
If this is a transient issue as m173 is not ready yet please ignore and close.

My W32-tcsh build fails with:
-------------
/bin/perl -w create.pl < ../wntmsci10.pro/misc/testhxx.output >
../wntmsci10.pro/bin/testhxx.btm
bad input: no source file at create.pl line 84, <> line 1.
dmake:  Error code 2, while making '../wntmsci10.pro/bin/testhxx.btm'
dmake:  '../wntmsci10.pro/bin/testhxx.btm' removed.

ERROR: Error 65280 occurred while making /cygdrive/d/w1/tin_tcsh/soltools/testhxx
---* tg_merge.mk *---
See also:
<http://go-oo.org/tinderbox/gunzip.cgi?tree=SRC680_m173&brief-log=1151083740.10830#err3>

I'll attach ../wntmsci10.pro/misc/testhxx.output
Comment 1 quetschke 2006-06-23 20:26:53 UTC
Created attachment 37303 [details]
testhxx.output
Comment 2 quetschke 2006-06-23 20:35:25 UTC
testhxx/create.pl mistakes WNT with 4nt. Adding sb as he added that file.
Comment 3 quetschke 2006-06-23 21:02:46 UTC
Created attachment 37304 [details]
Begin of a patch - please fill in the rest
Comment 4 quetschke 2006-06-23 21:10:22 UTC
I forgot a slash '\'
+    $in =~ s!\S+/testhxx.cxx!${my_tmp}!
------------------------------^ here.
Comment 5 jens-heiner.rechtien 2006-06-26 15:22:28 UTC
reassign to sb
Comment 6 Stephan Bergmann 2006-06-26 16:11:21 UTC
.
Comment 7 Stephan Bergmann 2006-06-27 13:41:26 UTC
Made minimal changes (soltools/testhxx/create.pl:1.2.2.1,
soltools/testhxx/makefile.mk:1.2.2.2) so that the build no longer breaks on the
cygwin platform, but the resulting testhxx is not yet functional (it is not
needed further down the build).

@vq:
- I cannot easily test this, so please verify that the fix indeed works.
- If you want to have a working testhxx
(<http://www.openoffice.org/servlets/ReadMsg?list=interface-announce&msgNo=916>)
on the cygwin platform, please do that in an extra issue (and let me know if you
need any help).
Comment 8 quetschke 2006-06-27 17:11:14 UTC
@sb: The fix looks good. I cannot test currently, but this is roughly what I used
as a workaround to keep the build going, so it should be fine.
Comment 9 quetschke 2006-06-27 17:11:43 UTC
I forgot the VERIFIED.
Comment 10 dkeskar 2006-07-07 04:44:58 UTC
On m174 w32/cygwin, I still had the same failure. This fixed it for me. 

--- soltools/testhxx/create.pl   20 Jun 2006 05:09:44 -0000      1.2
+++ soltools/testhxx/create.pl   7 Jul 2006 03:43:23 -0000
@@ -81,7 +81,7 @@
     $in =~ s!\s+-Fo\s*\S+! -Fo%my_tmp%.obj! || die 'bad input: no -Fo';
     $in =~ s!\s+-Zi\s! !;
     $in =~ s!\s+-Fd\s*\S+!!;
-    $in =~ s!\S+\\testhxx.cxx!%my_tmp%!
+    $in =~ s!\S+[\\\/]testhxx.cxx!%my_tmp%!
         || die 'bad input: no source file';
     print STDOUT '@ECHO OFF', "\n";
     print STDOUT 'SET my_tmp=%TMP%\%_PID%_include.cc', "\n";
Comment 11 Stephan Bergmann 2006-07-07 09:17:16 UTC
@dkeskar:  What do you mean with "still"---on a code base where the changes from
soltools/testhxx/create.pl:1.2.2.1 and soltools/testhxx/makefile.mk:1.2.2.2 are
included?  That is, was your comment meant as "here is an alternative fix," or
was it meant as "reopen/not yet fixed?"
Comment 12 Stephan Bergmann 2006-07-07 09:19:56 UTC
[Fix only made it into SRC680m175, not SRC680m174.  Maybe that explains
dkeskar's comment.]
Comment 13 Stephan Bergmann 2006-07-10 13:03:12 UTC
.