Hmn ... I tested 42X on 32-bit Windows 7 and it worked for me.

Looks like this is a 64-bit build, I didn't think that was working.

On  2 Oct, Matthias Seidel wrote:
> Windows build (42X) breaks:
> 
> =============
> Building module expat
> =============
> 
> Entering /cygdrive/c/Source/openoffice/main/expat
> 
> mkdir: cannot create directory
> './wntmsci12.pro/misc/build/expat-2.2.9/lib': File exists
> Making: ../../../../../wntmsci12.pro/slo/x64/xmltok.obj
> mkdir: cannot create directory '../../../../../wntmsci12.pro/slo/x64/':
> File exists
> xmltok.c
> dmake:  Error: -- `../../../../../wntmsci12.pro/slo/x64/loadlibrary.obj'
> not found, and can't be made
> dmake:  Error code 255, while making
> './wntmsci12.pro/misc/build/so_built_so_expat'
> 
> 1 module(s):
>         expat
> need(s) to be rebuilt
> 
> Reason(s):
> 
> ERROR: error 65280 occurred while making
> /cygdrive/c/Source/openoffice/main/expat
> 
> When you have fixed the errors in that module you can resume the build
> by running:
> 
>         build --from expat
> 
> Am 02.10.19 um 19:51 schrieb Matthias Seidel:
>> Hi Don,
>>
>> Great, I just started a new build for Windows...
>>
>> Would that be something we could cherry-pick for 4.1.8?
>>
>> Regards,
>>
>>    Matthias
>>
>> Am 02.10.19 um 19:08 schrieb truck...@apache.org:
>>> This is an automated email from the ASF dual-hosted git repository.
>>>
>>> truckman pushed a commit to branch trunk
>>> in repository https://gitbox.apache.org/repos/asf/openoffice.git
>>>
>>>
>>> The following commit(s) were added to refs/heads/trunk by this push:
>>>      new 190cb14  Upgrade bundled expat to 2.2.9.
>>> 190cb14 is described below
>>>
>>> commit 190cb14849237ec7207449ab78b1e9f4cddb3d32
>>> Author: Don Lewis <truck...@apache.org>
>>> AuthorDate: Wed Oct 2 10:07:37 2019 -0700
>>>
>>>     Upgrade bundled expat to 2.2.9.
>>>     
>>>     Delete expat-winapi.patch which was needed to get linkage on Windows
>>>     to work with the expat static libraries.  Instead add -DXML_STATIC
>>>     to the compiler flags when building the modules that link to the
>>>     expat static libraries to accomplish the same thing, but in a
>>>     supported way.
>>>     
>>>     Don't build the expat man pages since we don't have the necessary
>>>     build dependencies and we don't need them anyway.
>>> ---
>>>  main/expat/{expat-2.2.3.patch => expat-2.2.9.patch} | 14 +++++++-------
>>>  main/expat/expat-winapi.patch                       | 14 --------------
>>>  main/expat/makefile.mk                              |  9 ++++-----
>>>  main/expat/prj/d.lst                                |  4 ++--
>>>  main/external_deps.lst                              |  6 +++---
>>>  main/solenv/gbuild/gbuild.mk                        |  5 +++++
>>>  main/solenv/inc/settings.mk                         |  5 +++++
>>>  7 files changed, 26 insertions(+), 31 deletions(-)
>>>
>>> diff --git a/main/expat/expat-2.2.3.patch b/main/expat/expat-2.2.9.patch
>>> similarity index 92%
>>> rename from main/expat/expat-2.2.3.patch
>>> rename to main/expat/expat-2.2.9.patch
>>> index 2453518..96e23a5 100644
>>> --- a/main/expat/expat-2.2.3.patch
>>> +++ b/main/expat/expat-2.2.9.patch
>>> @@ -1,5 +1,5 @@
>>> ---- misc/expat-2.2.3/lib/makefile.mk       Wed Aug 20 14:33:55 2008
>>> -+++ misc/build/expat-2.2.3/lib/makefile.mk Wed Aug 20 14:26:42 2008
>>> +--- misc/expat-2.2.9/lib/makefile.mk       Wed Aug 20 14:33:55 2008
>>> ++++ misc/build/expat-2.2.9/lib/makefile.mk Wed Aug 20 14:26:42 2008
>>>  @@ -1 +1,91 @@
>>>  -dummy
>>>  +#**************************************************************
>>> @@ -49,11 +49,11 @@
>>>  +UNICODE_SLOFILES=$(SLO)$/xmlparse.obj
>>>  +UNICODECDEFS+=-DXML_UNICODE
>>>  +
>>> -+.IF "$(OS)"=="WNT"
>>> -+ LOADLIBOBJFILE=$(SLO)/loadlibrary.obj
>>> -+.ELSE
>>> -+ LOADLIBOBJFILE=
>>> -+.ENDIF
>>> ++#.IF "$(OS)"=="WNT"
>>> ++# LOADLIBOBJFILE=$(SLO)/loadlibrary.obj
>>> ++#.ELSE
>>> ++# LOADLIBOBJFILE=
>>> ++#.ENDIF
>>>  +
>>>  +LIB1ARCHIV=$(LB)$/libascii_$(TARGET)_xmlparse.a
>>>  +LIB1TARGET=$(SLB)$/ascii_$(TARGET)_xmlparse.lib
>>> diff --git a/main/expat/expat-winapi.patch b/main/expat/expat-winapi.patch
>>> deleted file mode 100644
>>> index 17667ac..0000000
>>> --- a/main/expat/expat-winapi.patch
>>> +++ /dev/null
>>> @@ -1,14 +0,0 @@
>>> -diff -ur misc/expat-2.2.3/lib/expat_external.h 
>>> misc/build/expat-2.2.3/lib/expat_external.h
>>> ---- misc/expat-2.2.3/lib/expat_external.h  2016-06-21 05:58:38.000000000 
>>> -0700
>>> -+++ misc/build/expat-2.2.3/lib/expat_external.h    2016-07-31 
>>> 17:10:53.551556000 -0700
>>> -@@ -7,10 +7,6 @@
>>> - 
>>> - /* External API definitions */
>>> - 
>>> --#if defined(_MSC_EXTENSIONS) && !defined(__BEOS__) && !defined(__CYGWIN__)
>>> --#define XML_USE_MSC_EXTENSIONS 1
>>> --#endif
>>> --
>>> - /* Expat tries very hard to make the API boundary very specifically
>>> -    defined.  There are two macros defined to control this boundary;
>>> -    each of these can be defined before including this header to
>>> diff --git a/main/expat/makefile.mk b/main/expat/makefile.mk
>>> index 6a96d92..4d19604 100644
>>> --- a/main/expat/makefile.mk
>>> +++ b/main/expat/makefile.mk
>>> @@ -38,17 +38,16 @@ all:
>>>  
>>>  # --- Files --------------------------------------------------------
>>>  
>>> -TARFILE_NAME=expat-2.2.3
>>> -TARFILE_MD5=f053af63ef5f39bd9b78d01fbc203334
>>> +TARFILE_NAME=expat-2.2.9
>>> +TARFILE_MD5=875a2c2ff3e8eb9e5a5cd62db2033ab5
>>>  ADDITIONAL_FILES=lib$/makefile.mk
>>> -PATCH_FILES=$(TARFILE_NAME).patch \
>>> -            expat-winapi.patch
>>> +PATCH_FILES=$(TARFILE_NAME).patch
>>>  
>>>  CONFIGURE_DIR=
>>>  .IF "$(OS)"=="WNT"
>>>  CONFIGURE_ACTION=
>>>  .ELSE
>>> -CONFIGURE_ACTION=.$/configure
>>> +CONFIGURE_ACTION=.$/configure --without-docbook
>>>  .ENDIF
>>>  
>>>  BUILD_DIR=lib
>>> diff --git a/main/expat/prj/d.lst b/main/expat/prj/d.lst
>>> index 951ac27..fe7184e 100644
>>> --- a/main/expat/prj/d.lst
>>> +++ b/main/expat/prj/d.lst
>>> @@ -1,7 +1,7 @@
>>>  mkdir: %_DEST%\inc%_EXT%\external\expat
>>>  mkdir: %_DEST%\lib%_EXT%\x64
>>> -..\%__SRC%\misc\build\expat-2.2.3\lib\expat.h 
>>> %_DEST%\inc%_EXT%\external\expat.h
>>> -..\%__SRC%\misc\build\expat-2.2.3\lib\expat_external.h 
>>> %_DEST%\inc%_EXT%\external\expat_external.h
>>> +..\%__SRC%\misc\build\expat-2.2.9\lib\expat.h 
>>> %_DEST%\inc%_EXT%\external\expat.h
>>> +..\%__SRC%\misc\build\expat-2.2.9\lib\expat_external.h 
>>> %_DEST%\inc%_EXT%\external\expat_external.h
>>>  ..\%__SRC%\slb\expat_xmltok.lib %_DEST%\lib%_EXT%\expat_xmltok.lib
>>>  ..\%__SRC%\slb\expat_xmlparse.lib %_DEST%\lib%_EXT%\expat_xmlparse.lib
>>>  ..\%__SRC%\slb\ascii_expat_xmlparse.lib 
>>> %_DEST%\lib%_EXT%\ascii_expat_xmlparse.lib
>>> diff --git a/main/external_deps.lst b/main/external_deps.lst
>>> index 5a7ea06..2b0ebe6 100644
>>> --- a/main/external_deps.lst
>>> +++ b/main/external_deps.lst
>>> @@ -217,9 +217,9 @@ if (SYSTEM_VIGRA != YES)
>>>      URL2 = $(OOO_EXTRAS)$(MD5)-$(name)
>>>  
>>>  if (SYSTEM_EXPAT != YES)
>>> -    MD5 = f053af63ef5f39bd9b78d01fbc203334
>>> -    name = expat-2.2.3.tar.bz2
>>> -    URL1 = 
>>> https://sourceforge.net/projects/expat/files/expat/2.2.3/expat-2.2.3.tar.bz2
>>> +    MD5 = 875a2c2ff3e8eb9e5a5cd62db2033ab5
>>> +    name = expat-2.2.9.tar.bz2
>>> +    URL1 = 
>>> https://sourceforge.net/projects/expat/files/expat/2.2.9/expat-2.2.9.tar.bz2
>>>      URL2 = $(OOO_EXTRAS)$(MD5)-$(name)
>>>  
>>>  if (SYSTEM_CURL != YES)
>>> diff --git a/main/solenv/gbuild/gbuild.mk b/main/solenv/gbuild/gbuild.mk
>>> index 4770ae7..f874d37 100644
>>> --- a/main/solenv/gbuild/gbuild.mk
>>> +++ b/main/solenv/gbuild/gbuild.mk
>>> @@ -216,6 +216,11 @@ ifeq ($(strip $(ENABLE_GRAPHITE)),TRUE)
>>>  gb_GLOBALDEFS += -DENABLE_GRAPHITE
>>>  endif
>>>  
>>> +# Required for correct Windows function call ABI for expat static library
>>> +ifeq ($(SYSTEM_EXPAT),NO)
>>> +gb_GLOBALDEFS += -DXML_STATIC
>>> +endif
>>> +
>>>  gb_GLOBALDEFS := $(sort $(gb_GLOBALDEFS))
>>>  
>>>  include $(GBUILDDIR)/Deliver.mk
>>> diff --git a/main/solenv/inc/settings.mk b/main/solenv/inc/settings.mk
>>> index 3ccf863..43c945c 100644
>>> --- a/main/solenv/inc/settings.mk
>>> +++ b/main/solenv/inc/settings.mk
>>> @@ -892,6 +892,11 @@ CDEFS+=$(CDEFS_PRESET)
>>>  CDEFS+=-DTIMELOG
>>>  .ENDIF
>>>  
>>> +# Required for correct Windows function call ABI for expat static library
>>> +.IF "$(SYSTEM_EXPAT)"=="NO"
>>> +CDEFS+=-DXML_STATIC
>>> +.ENDIF
>>> +
>>>  CDEFSCXX=
>>>  CDEFSOBJ=
>>>  CDEFSSLO=-DSHAREDLIB -D_DLL_
>>>
> 


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

Reply via email to