Hi Eliot! sorry for the late answer. It seems it was overtaken by a<br>
slightly bigger issue now, but thanks for the explanation anyway.<br>
<br>
Cheers,<br>
Pocho<br>
<br>
On Sat, Feb 3, 2018 at 12:52 PM, OpenSmalltalk-Bot <notifications@github.com<br>
> wrote:<br>
<br>
> Hi Javier,<br>
><br>
> > On Feb 2, 2018, at 11:50 AM, Javier Pimas <notifications@github.com><br>
> wrote:<br>
> ><br>
> > As this is a submodule, .git is not a directory but a file. Changing the<br>
> line<br>
> ><br>
> > if [ -d `dirname $0`/../.git ]; then<br>
> > to<br>
> ><br>
> > if [ -e `dirname $0`/../.git ]; then<br>
> > makes the script work, but as I have no idea what this script does, I'm<br>
> not sure if the change is correct<br>
> ><br>
><br>
> The platforms/Cross/vm/sqSCCSVersion.h file is supposed to contain the<br>
> information that identifies a particular commit. It stands for Source Code<br>
> Control System Version. With many SCCS systems, including Subversion, which<br>
> was used to manage the source before git, one can use a metalanguage in the<br>
> source file that is recognized and replaced by commit information on each<br>
> commit. Git doesn't support this (?!?!?!). So we have to do it manually<br>
> with this script. The script gets run automatically by the commit script<br>
> scripts/gitci, but if you do a fresh check out you'll need to run the<br>
> script to get the VM's --version command to print meaningful commit<br>
> information.<br>
><br>
> Ideally we would have rewritten the makefiles to do this automatically but<br>
> not yet :-)<br>
><br>
> Anyway, your fix looks correct. Please feel free to commit. Also you might<br>
> copy my opening paragraph above ("The platforms ... meaningful commit<br>
> information.") as a comment at the start of the script.<br>
><br>
> Sorry for the mess. But most importantly, welcome!!! It's great to see you<br>
> and Guido involved. I'm very excited. Thank you!<br>
><br>
><br>
> Eliot _,,,^..^,,,_ (phone)<br>
><br>
> —<br>
> You are receiving this because you authored the thread.<br>
> Reply to this email directly, view it on GitHub<br>
> <https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/209#issuecomment-362825127>,<br>
> or mute the thread<br>
> <https://github.com/notifications/unsubscribe-auth/AATPaVcijxg2VuLMQvIJyCQAPPxhJMioks5tRIDUgaJpZM4R3brI><br>
> .<br>
><br>
<br>
<br>
<br>
-- <br>
Javier Pimás<br>
Ciudad de Buenos Aires<br>


<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you commented.<br />Reply to this email directly, <a href="https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/209#issuecomment-365622436">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AhLyW2uFSl11jI-72lyB4YJfVRsUI2Pcks5tUuzRgaJpZM4R3brI">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AhLyW4MPueVtI3z16d6eeKTOZQRoDfQVks5tUuzRgaJpZM4R3brI.gif" width="1" /></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/issues/209#issuecomment-365622436"></link>
  <meta itemprop="name" content="View Issue"></meta>
</div>
<meta itemprop="description" content="View this Issue 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":"@melkyades in #209: Hi Eliot! sorry for the late answer. It seems it was overtaken by a\nslightly bigger issue now, but thanks for the explanation anyway.\n\nCheers,\nPocho\n\nOn Sat, Feb 3, 2018 at 12:52 PM, OpenSmalltalk-Bot \u003cnotifications@github.com\n\u003e wrote:\n\n\u003e Hi Javier,\n\u003e\n\u003e \u003e On Feb 2, 2018, at 11:50 AM, Javier Pimas \u003cnotifications@github.com\u003e\n\u003e wrote:\n\u003e \u003e\n\u003e \u003e As this is a submodule, .git is not a directory but a file. Changing the\n\u003e line\n\u003e \u003e\n\u003e \u003e if [ -d `dirname $0`/../.git ]; then\n\u003e \u003e to\n\u003e \u003e\n\u003e \u003e if [ -e `dirname $0`/../.git ]; then\n\u003e \u003e makes the script work, but as I have no idea what this script does, I'm\n\u003e not sure if the change is correct\n\u003e \u003e\n\u003e\n\u003e The platforms/Cross/vm/sqSCCSVersion.h file is supposed to contain the\n\u003e information that identifies a particular commit. It stands for Source Code\n\u003e Control System Version. With many SCCS systems, including Subversion, which\n\u003e was used to manage the source before git, one can use a metalanguage in the\n\u003e source file that is recognized and replaced by commit information on each\n\u003e commit. Git doesn't support this (?!?!?!). So we have to do it manually\n\u003e with this script. The script gets run automatically by the commit script\n\u003e scripts/gitci, but if you do a fresh check out you'll need to run the\n\u003e script to get the VM's --version command to print meaningful commit\n\u003e information.\n\u003e\n\u003e Ideally we would have rewritten the makefiles to do this automatically but\n\u003e not yet :-)\n\u003e\n\u003e Anyway, your fix looks correct. Please feel free to commit. Also you might\n\u003e copy my opening paragraph above (\"The platforms ... meaningful commit\n\u003e information.\") as a comment at the start of the script.\n\u003e\n\u003e Sorry for the mess. But most importantly, welcome!!! It's great to see you\n\u003e and Guido involved. I'm very excited. Thank you!\n\u003e\n\u003e\n\u003e Eliot _,,,^..^,,,_ (phone)\n\u003e\n\u003e —\n\u003e You are receiving this because you authored the thread.\n\u003e Reply to this email directly, view it on GitHub\n\u003e \u003chttps://github.com/OpenSmalltalk/opensmalltalk-vm/issues/209#issuecomment-362825127\u003e,\n\u003e or mute the thread\n\u003e \u003chttps://github.com/notifications/unsubscribe-auth/AATPaVcijxg2VuLMQvIJyCQAPPxhJMioks5tRIDUgaJpZM4R3brI\u003e\n\u003e .\n\u003e\n\n\n\n-- \nJavier Pimás\nCiudad de Buenos Aires\n"}],"action":{"name":"View Issue","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/issues/209#issuecomment-365622436"}}}</script>