[OpenSmalltalk/opensmalltalk-v
Locked
[OpenSmalltalk/opensmalltalk-v
 
|
Ronie submitted a pull request that fixes the build with cygwin for win32 ( #311 ) and I'm curious if any of my changes here are still applicable. @ronsaldo could you comment on these... You can view, comment on, or merge this pull request online at:https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/312 Commit Summary
File Changes
Patch Links:
— |
Re: [OpenSmalltalk/opensmalltalk-v
|
|
Hi Ben, On a quick glance, some of these changes are still required, particularly the FPU one, probably the alloca one. Other changes, such as the #define BUILD_VM_CORE is actually wrong. The minheadless builds the VM itself in a separate library (currently a static library) that can actually be embedded in any application. The public interface for this library is in include/OpenSmalltalkVM.h The problem in windows here was that BUILD_OSVM_STATIC in a previous version was BUILD_SQUEAK_STATIC . After a commentary from @estebanlm I changed the squeak_ prefix for the osvm_ in this public interface. The public interface file is intended as an stable standalone interface to the vm. So, dependencies on sq.h or similar should not be allowed in this file (In my opinion). The implementation for this interface is in platforms/minheadless/common/sqVirtualMachineInterface.c . This header should be something that an user can copy to /usr/include . osvm_main can be used as an example on how to embed the VM in an application:
A part of the better 32 bits detection and configuration should also be integrated. When I started the minheadless VM two years ago, there was no stable win64 version of the VM. — |
Re: [OpenSmalltalk/opensmalltalk-v
|
|
In reply to this post by David T Lewis
Okay. I see my BUILD_VM_CORE was to work around an unknown symbol "_imp_osvm_main", — |
Re: [OpenSmalltalk/opensmalltalk-v
|
|
In reply to this post by David T Lewis
@bencoman commented on this pull request. In CMakeLists.txt: > @@ -2,6 +2,15 @@
cmake_minimum_required(VERSION 3.1)
project(OpenSmalltalkVM)
+# The following is temporary while understanding/debugging CMake build.
+message("START VARIABLES (A)")
+get_cmake_property(_variableNames VARIABLES)
+list (SORT _variableNames)
+foreach (_variableName ${_variableNames})
+ message(STATUS "${_variableName}=${${_variableName}}")
+endforeach()
+message("END VARIABLES (A)")
+
Ignore lines 5-13, they were just adding verbosity for debugging In platforms/minheadless/windows/sqPlatformSpecific-Win32.c: > @@ -77,7 +77,7 @@ static void printCrashDebugInformation(LPEXCEPTION_POINTERS exp); #define PROCESS_PER_MONITOR_DPI_AWARE 2 #endif -typedef HRESULT WINAPI (*SetProcessDpiAwarenessFunctionPointer) (int awareness); +typedef HRESULT (WINAPI *SetProcessDpiAwarenessFunctionPointer) (int awareness); I see this is already covered in #311 In CMakeLists.txt: > @@ -530,6 +540,15 @@ endif()
add_library(${VM_LIBRARY_NAME} ${VM_CORE_LIBRARY_TYPE} ${VM_SOURCES} ${VM_INTERNAL_PLUGIN_SOURCES})
+# The following is temporary while understanding/debugging CMake build.
+message("START VARIABLES (B)")
+get_cmake_property(_variableNames VARIABLES)
+list (SORT _variableNames)
+foreach (_variableName ${_variableNames})
+ message(STATUS "${_variableName}=${${_variableName}}")
+endforeach()
+message("END VARIABLES (B)")
+
Ignore lines 543-551, they were just adding verbosity for debugging In platforms/minheadless/windows/sqWin32Main.c: > @@ -28,6 +28,7 @@ */ #define WIN32_LEAN_AND_MEAN +#define BUILD_VM_CORE Not required per Ronie's comment. It was to work around an unknown symbol "_imp_osvm_main", which BUILD_OSVM_STATIC fixed already. In platforms/minheadless/windows/sqPlatformSpecific-Win32.h: > @@ -114,7 +114,7 @@ size_t sqImageFileWrite(const void *ptr, size_t sz, size_t count, sqImageFile h) error "Not Win32!" #endif /* WIN32 */ -int ioSetCursorARGB(sqInt bitsIndex, sqInt w, sqInt h, sqInt x, sqInt y); +sqInt ioSetCursorARGB(sqInt bitsIndex, sqInt w, sqInt h, sqInt x, sqInt y); Required. Latest Cog branch minheadless/x64 had the same compile error. Including this change only let the build complete. In platforms/minheadless/windows/sqPlatformSpecific-Win32.c: > @@ -115,7 +115,12 @@ void
ioInitPlatformSpecific(void)
{
/* Setup the FPU */
- _controlfp(FPU_DEFAULT, _MCW_EM | _MCW_RC | _MCW_PC | _MCW_IC);
+ /**** 2018.08.07.BenComan TODO, help required.
+ **** x64 does not support _MCW_PC or _MCW_IC per https://msdn.microsoft.com/en-us/library/e9b52ceh.aspx
+ ****/
+ //Original Line// _controlfp(FPU_DEFAULT, _MCW_EM | _MCW_RC | _MCW_PC | _MCW_IC);
+ _controlfp(FPU_DEFAULT, _MCW_EM | _MCW_RC);
+
Required. I've included this per Ronie's comment. Fixes a runtime error. In platforms/minheadless/common/sqMain.c: > @@ -25,6 +25,7 @@ * * Author: [hidden email] */ +#define BUILD_VM_CORE Not required per Ronie's comment. It was to work around an unknown symbol "_imp_osvm_main", which BUILD_OSVM_STATIC fixed already. In platforms/Cross/plugins/IA32ABI/xabicc.c: > @@ -5,7 +5,7 @@ * The plugin is misnamed. It should be the AlienPlugin, but its history * dictates otherwise. */ -#if i386|i486|i586|i686 +#if i386|i486|i586|i686|_M_IX86 Not required. Already fixed by PR #311 using... In platforms/Cross/plugins/IA32ABI/ia32abicc.c: > @@ -8,7 +8,7 @@ /* null if compiled on other than x86, to get around gnu make bugs or * misunderstandings on our part. */ -#if i386|i486|i586|i686 +#if i386|i486|i586|i686|_M_IX86 Not required. Already fixed by PR #311 using... In platforms/Cross/plugins/IA32ABI/x64win64abicc.c: > @@ -38,6 +38,9 @@ extern #endif struct VirtualMachine* interpreterProxy; +#ifdef _MSC_VER +# define alloca _alloca +#endif Required. Solves MSVC build error... Visual Studio 2017" has no "alloca" only "_alloca" — |
Re: [OpenSmalltalk/opensmalltalk-v
|
|
In reply to this post by David T Lewis
@nicolas-cellier-aka-nice commented on this pull request. In platforms/minheadless/windows/sqPlatformSpecific-Win32.c: > @@ -115,7 +115,12 @@ void
ioInitPlatformSpecific(void)
{
/* Setup the FPU */
- _controlfp(FPU_DEFAULT, _MCW_EM | _MCW_RC | _MCW_PC | _MCW_IC);
+ /**** 2018.08.07.BenComan TODO, help required.
+ **** x64 does not support _MCW_PC or _MCW_IC per https://msdn.microsoft.com/en-us/library/e9b52ceh.aspx
+ ****/
+ //Original Line// _controlfp(FPU_DEFAULT, _MCW_EM | _MCW_RC | _MCW_PC | _MCW_IC);
+ _controlfp(FPU_DEFAULT, _MCW_EM | _MCW_RC);
+
I've fixed it in the main OpenSmalltalk VM and will backport. — |
Re: [OpenSmalltalk/opensmalltalk-v
|
|
In reply to this post by David T Lewis
Hi Ben, I'm currently working on building the Tertf (3D ICC) VM using MSVC Community edition. When I'm done I'll probably migrate the MSVC-specific Makefiles for build.win32* and platforms/win32/plugins into opensmalltalk-vm. — |
Re: [OpenSmalltalk/opensmalltalk-v
|
|
In reply to this post by David T Lewis
Is this still necessary with the current MSVC stuff? — |
Re: [OpenSmalltalk/opensmalltalk-v
|
|
In reply to this post by David T Lewis
Whether it's necessary or not, it's useful to have a minimal headless VM. That said, given that the MSVC 2017 build works, the headful makefiles could be used to inform the CMake build (settings, dealing with setjmp/longjmp issues, etc). — |
Re: [OpenSmalltalk/opensmalltalk-v
|
|
In reply to this post by David T Lewis
Makes sense! — |
Re: [OpenSmalltalk/opensmalltalk-v
|
|
In reply to this post by David T Lewis
If herein is simpler then herein :-) — |
Re: [OpenSmalltalk/opensmalltalk-v
|
|
In reply to this post by David T Lewis
since 4 out of 6 files have conflicts and some changes are marked "not required" or "already taken care of" by @bencoman, I'd say, starting over is easier? — |
Re: [OpenSmalltalk/opensmalltalk-v
|
|
In reply to this post by David T Lewis
Cool. Closing... — |
Re: [OpenSmalltalk/opensmalltalk-v
|
|
In reply to this post by David T Lewis
Closed #312. — |
| Free forum by Nabble | Edit this page |