To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=53572





------- Additional comments from [EMAIL PROTECTED] Tue Mar 20 01:00:06 +0000 
2007 -------
@maho: It also breaks the "OOo-style" W32-.NET2003-**sh build.

I looked at the diff for config_office, this is *NOT* OK to be integrated.

I'll comment on a few differences, but this hunk from set_soenv.in is the one
that breaks the W32-build (not tested, but I'm pretty sure):

@@ -1374,8 +1398,7 @@
 elsif ($platform =~ m/cygwin/)
 {  $SOLARINC         .= $STLPORT_stlport.
                         $I.'$JAVA_HOME'.$ds."include".$ds."win32".
-                        $I.'$JAVA_HOME'.$ds."include".
-                        $I.'$PSDK_HOME'.$INCLUDE;
+                        $I.'$JAVA_HOME'.$ds."include";
    # Mingw is different
    if ( $COM eq "MSC" )
    {  $SOLARINC       .= $I.'$COMPATH'.$ds."include";


Some comments on the rest starting from the beginning:

This
-   CC=`echo $CC | $SED -n "s/^guw.exe //p"`
-   CXX=`echo $CXX | $SED -n "s/^guw.exe //p"`
+   CC=`echo $CC | $SED "s/^guw.pl //"`
+   CXX=`echo $CXX | $SED "s/^guw.pl //"`
is wrong.

Please don't use the name "LIB_INCLUDE_PATH" if it is for MinGW only, use
something like: MINGW_LIB_INCLUDE_PATH

In set_soenv.in this:
# zip.exe / unzip.exe path for Windows users.
comment doesn't reflect the change.

Overall please watch your indentation in set_soenv.in. Your code looks
significantly different from the code that is already there, please blend in ;)

Also, you forgot to regenerate configure.

---------------------------------------------------------------------
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]

Reply via email to