Hi Cyril,<br>
<br>
    wouldn't it be better to have the script in scripts with some visible name?<br>
<br>
_,,,^..^,,,_ (phone)<br>
<br>
> On Apr 7, 2018, at 7:59 AM, CyrilFerlicot <notifications@github.com> wrote:<br>
> <br>
> Now the installation is extracted to a reusable batch script and install all dependencies, including the ones installed by default on appveyor. This allows the users of the vm to use the script to build their own cygwin to compile the vm without having to look for all needed dependencies.<br>
> <br>
> I am not sure it will work at the first try. I do a PR to let appveyor test it for me.<br>
> <br>
> You can view, comment on, or merge this pull request online at:<br>
> <br>
>   https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/242<br>
> <br>
> Commit Summary<br>
> <br>
> Refactor cygwin installation for appveyor. Now the installation is extracted to a reusable batch script and install all dependencies, including the ones installed by default on appveyor. This allow the users of the vm to use the script to build their own cygwin to compile the vm without having to look for all neededs dependencies.<br>
> File Changes<br>
> <br>
> M .appveyor.yml (21)<br>
> A .appveyor_installCygwin.bat (70)<br>
> M build.win32x86/HowToBuild (3)<br>
> M build.win64x64/HowToBuild (2)<br>
> Patch Links:<br>
> <br>
> https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/242.patch<br>
> https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/242.diff<br>
> —<br>
> You are receiving this because you are subscribed to this thread.<br>
> Reply to this email directly, view it on GitHub, or mute the thread.<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/242#issuecomment-379476201">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AhLyW-6yrIMgHtmCsXaBwuW8aJMo-Iumks5tmNbCgaJpZM4TLHTw">mute the thread</a>.<img src="https://github.com/notifications/beacon/AhLyW93wrJY3Se2UrToFOz7htet1RhnMks5tmNbCgaJpZM4TLHTw.gif" height="1" width="1" alt="" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/242#issuecomment-379476201"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<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://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@eliotmiranda in #242: Hi Cyril,\n\n    wouldn't it be better to have the script in scripts with some visible name?\n\n_,,,^..^,,,_ (phone)\n\n\u003e On Apr 7, 2018, at 7:59 AM, CyrilFerlicot \u003cnotifications@github.com\u003e wrote:\n\u003e \n\u003e Now the installation is extracted to a reusable batch script and install all dependencies, including the ones installed by default on appveyor. This allows the users of the vm to use the script to build their own cygwin to compile the vm without having to look for all needed dependencies.\n\u003e \n\u003e I am not sure it will work at the first try. I do a PR to let appveyor test it for me.\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/242\n\u003e \n\u003e Commit Summary\n\u003e \n\u003e Refactor cygwin installation for appveyor. Now the installation is extracted to a reusable batch script and install all dependencies, including the ones installed by default on appveyor. This allow the users of the vm to use the script to build their own cygwin to compile the vm without having to look for all neededs dependencies.\n\u003e File Changes\n\u003e \n\u003e M .appveyor.yml (21)\n\u003e A .appveyor_installCygwin.bat (70)\n\u003e M build.win32x86/HowToBuild (3)\n\u003e M build.win64x64/HowToBuild (2)\n\u003e Patch Links:\n\u003e \n\u003e https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/242.patch\n\u003e https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/242.diff\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, or mute the thread.\n\u003e \n"}],"action":{"name":"View Pull Request","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/242#issuecomment-379476201"}}}</script>