Hi Ronie,<br>
<br>
What is your procedure for building with MSVC ?<br>
I'd like to have a go at it.<br>
<br>
cheers -ben<br>
<br>
On Sat, 1 Dec 2018 at 07:20, Ronie Salgado <notifications@github.com> wrote:<br>
<br>
> Quick fixup of the minheadless VM for windows using the CMake building<br>
> scripts.<br>
> The VM builds and works with Cygwin for 32 bits.<br>
> It builds with MSVC, but fails with several errors on runtime.<br>
> ------------------------------<br>
> You can view, comment on, or merge this pull request online at:<br>
><br>
>   https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311<br>
> Commit Summary<br>
><br>
>    - Fix the build of the minheadless VM for Pharo in Win32.<br>
><br>
> File Changes<br>
><br>
>    - *M* CMakeLists.txt<br>
>    <https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-0><br>
>    (23)<br>
>    - *M* build.minheadless.cmake/x64/common/configure_variant.sh<br>
>    <https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-1><br>
>    (1)<br>
>    - *M* cmake/Plugins.cmake<br>
>    <https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-2><br>
>    (9)<br>
>    - *M* cmake/PluginsPharo.cmake<br>
>    <https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-3><br>
>    (19)<br>
>    - *M* platforms/Cross/plugins/IA32ABI/ia32abicc.c<br>
>    <https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-4><br>
>    (4)<br>
>    - *M* platforms/Cross/plugins/IA32ABI/x64win64abicc.c<br>
>    <https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-5><br>
>    (8)<br>
>    - *M* platforms/Cross/plugins/IA32ABI/xabicc.c<br>
>    <https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-6><br>
>    (6)<br>
>    - *M* platforms/Cross/plugins/Mpeg3Plugin/libmpeg/changesForSqueak.c<br>
>    <https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-7><br>
>    (3)<br>
>    - *M* platforms/Cross/plugins/Mpeg3Plugin/libmpeg/mpeg3private.h<br>
>    <https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-8><br>
>    (3)<br>
>    - *M* platforms/minheadless/windows/sqPlatformSpecific-Win32.c<br>
>    <https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-9><br>
>    (123)<br>
>    - *M* platforms/minheadless/windows/sqWin32Directory.c<br>
>    <https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-10><br>
>    (269)<br>
>    - *A* platforms/minheadless/windows/sqWin32Stubs.c<br>
>    <https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-11><br>
>    (19)<br>
><br>
> Patch Links:<br>
><br>
>    - https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311.patch<br>
>    - https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311.diff<br>
><br>
> —<br>
> You are receiving this because you are subscribed to this thread.<br>
> Reply to this email directly, view it on GitHub<br>
> <https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311>, or mute the<br>
> thread<br>
> <https://github.com/notifications/unsubscribe-auth/ABolJ8jAUbhxA8NQPK4dTjyPdyEuzmS1ks5u0b00gaJpZM4Y8jIn><br>
> .<br>
><br>


<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311#issuecomment-443418024">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AhLyWyKB8dxHDgOpMt4VuvG_Pg-y9Pgyks5u0mFggaJpZM4Y8jIn">mute the thread</a>.<img src="https://github.com/notifications/beacon/AhLyW6JxiL9y08DgNZ-yt8Q4D6nW8Ry_ks5u0mFggaJpZM4Y8jIn.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/OpenSmalltalk/opensmalltalk-vm","title":"OpenSmalltalk/opensmalltalk-vm","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@bencoman in #311: Hi Ronie,\n\nWhat is your procedure for building with MSVC ?\nI'd like to have a go at it.\n\ncheers -ben\n\nOn Sat, 1 Dec 2018 at 07:20, Ronie Salgado \u003cnotifications@github.com\u003e wrote:\n\n\u003e Quick fixup of the minheadless VM for windows using the CMake building\n\u003e scripts.\n\u003e The VM builds and works with Cygwin for 32 bits.\n\u003e It builds with MSVC, but fails with several errors on runtime.\n\u003e ------------------------------\n\u003e You can view, comment on, or merge this pull request online at:\n\u003e\n\u003e   https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311\n\u003e Commit Summary\n\u003e\n\u003e    - Fix the build of the minheadless VM for Pharo in Win32.\n\u003e\n\u003e File Changes\n\u003e\n\u003e    - *M* CMakeLists.txt\n\u003e    \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-0\u003e\n\u003e    (23)\n\u003e    - *M* build.minheadless.cmake/x64/common/configure_variant.sh\n\u003e    \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-1\u003e\n\u003e    (1)\n\u003e    - *M* cmake/Plugins.cmake\n\u003e    \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-2\u003e\n\u003e    (9)\n\u003e    - *M* cmake/PluginsPharo.cmake\n\u003e    \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-3\u003e\n\u003e    (19)\n\u003e    - *M* platforms/Cross/plugins/IA32ABI/ia32abicc.c\n\u003e    \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-4\u003e\n\u003e    (4)\n\u003e    - *M* platforms/Cross/plugins/IA32ABI/x64win64abicc.c\n\u003e    \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-5\u003e\n\u003e    (8)\n\u003e    - *M* platforms/Cross/plugins/IA32ABI/xabicc.c\n\u003e    \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-6\u003e\n\u003e    (6)\n\u003e    - *M* platforms/Cross/plugins/Mpeg3Plugin/libmpeg/changesForSqueak.c\n\u003e    \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-7\u003e\n\u003e    (3)\n\u003e    - *M* platforms/Cross/plugins/Mpeg3Plugin/libmpeg/mpeg3private.h\n\u003e    \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-8\u003e\n\u003e    (3)\n\u003e    - *M* platforms/minheadless/windows/sqPlatformSpecific-Win32.c\n\u003e    \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-9\u003e\n\u003e    (123)\n\u003e    - *M* platforms/minheadless/windows/sqWin32Directory.c\n\u003e    \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-10\u003e\n\u003e    (269)\n\u003e    - *A* platforms/minheadless/windows/sqWin32Stubs.c\n\u003e    \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311/files#diff-11\u003e\n\u003e    (19)\n\u003e\n\u003e Patch Links:\n\u003e\n\u003e    - https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311.patch\n\u003e    - https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311.diff\n\u003e\n\u003e —\n\u003e You are receiving this because you are subscribed to this thread.\n\u003e Reply to this email directly, view it on GitHub\n\u003e \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311\u003e, or mute the\n\u003e thread\n\u003e \u003chttps://github.com/notifications/unsubscribe-auth/ABolJ8jAUbhxA8NQPK4dTjyPdyEuzmS1ks5u0b00gaJpZM4Y8jIn\u003e\n\u003e .\n\u003e\n"}],"action":{"name":"View Pull Request","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311#issuecomment-443418024"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311#issuecomment-443418024",
"url": "https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311#issuecomment-443418024",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [OpenSmalltalk/opensmalltalk-vm] Fix the build of the minheadless VM for Pharo in Win32. (#311)",
"sections": [
{
"text": "",
"activityTitle": "**Ben Coman**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@bencoman",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"OpenSmalltalk/opensmalltalk-vm\",\n\"issueId\": 311,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/311#issuecomment-443418024"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 418525735\n}"
}
],
"themeColor": "26292E"
}
]</script>