<blockquote>
<p>Well shouldn't it work that way? I mean, there are other drawbacks of identifying VMs by commit id (e.g. the need for git hooks).</p>
</blockquote>
<p>I'm not sure.  If we introduced a version number or build number there may still be a need to substitute it in to other files, so there may still be a use for the commit hooks.  And event with the other identifier, I wouldn't want to get rid of the CommitHash - it makes it easy as a developer to find the code.</p>
<blockquote>
<p>More importantly, why rebuild all Linux/macOS VMs when only Windows code was modified?</p>
</blockquote>
<p>This is a good point, of course.</p>
<blockquote>
<p>I have never expected bleeding edge "releases" on Bintray to be complete, no one really should because they are mainly for development and builds happen to fail from time to time. Stable release builds will still be triggered simultaneously on AppVeyor and Travis. So we should be fine, right?</p>
</blockquote>
<p>Implicit in what you're saying is that someone then manually starts a full build for every release candidate.</p>
<p>So, I'm fine with the change you're proposing, it just requires buy-in from the people that are actually responsible for the releases in the various flavours (and thus are responsible for ensuring a full build for the released VM).</p>
<p>It also doesn't completely remove the usefulness of being able to skip building when desired. :-)</p>

<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/349#issuecomment-451719955">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AhLyW4XmCc_UTS5FQV8M-1FEk1HOwWdyks5vAZxdgaJpZM4ZyCx8">mute the thread</a>.<img src="https://github.com/notifications/beacon/AhLyW6Iv5NUirMLMxvsC0QSJnZ9vHYE1ks5vAZxdgaJpZM4ZyCx8.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://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@akgrant43 in #349: \u003e Well shouldn't it work that way? I mean, there are other drawbacks of identifying VMs by commit id (e.g. the need for git hooks).\r\n\r\nI'm not sure.  If we introduced a version number or build number there may still be a need to substitute it in to other files, so there may still be a use for the commit hooks.  And event with the other identifier, I wouldn't want to get rid of the CommitHash - it makes it easy as a developer to find the code.\r\n\r\n\r\n\u003e More importantly, why rebuild all Linux/macOS VMs when only Windows code was modified?\r\n\r\nThis is a good point, of course.\r\n\r\n\r\n\u003e I have never expected bleeding edge \"releases\" on Bintray to be complete, no one really should because they are mainly for development and builds happen to fail from time to time. Stable release builds will still be triggered simultaneously on AppVeyor and Travis. So we should be fine, right?\r\n\r\nImplicit in what you're saying is that someone then manually starts a full build for every release candidate.\r\n\r\nSo, I'm fine with the change you're proposing, it just requires buy-in from the people that are actually responsible for the releases in the various flavours (and thus are responsible for ensuring a full build for the released VM).\r\n\r\nIt also doesn't completely remove the usefulness of being able to skip building when desired. :-)\r\n"}],"action":{"name":"View Pull Request","url":"https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/349#issuecomment-451719955"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/349#issuecomment-451719955",
"url": "https://github.com/OpenSmalltalk/opensmalltalk-vm/pull/349#issuecomment-451719955",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>