<p>I think a bigger problem is that you've changed the ruby code so that it unconditionally replaces the compiled versions of the methods with the ruby versions which will kind of defeat the whole point ;-)</p>
<p>The previous structure was designed so that it would still work even if somebody hadn't built the extension and would just be slower - if we're moving to a gem we can probably assume it will be compiled and delete the ruby versions.</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/openstreetmap/openstreetmap-website/pull/1899#issuecomment-396961140">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLVkdQbFzcwBZdtoG9Jmq9wedmNzkks5t8SPJgaJpZM4UlXyB">mute the thread</a>.<img src="https://github.com/notifications/beacon/ABWnLeW1nNxdYkJ4y2WYJqfa89yC9bM8ks5t8SPJgaJpZM4UlXyB.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">{"@context":"http://schema.org","@type":"EmailMessage","potentialAction":{"@type":"ViewAction","target":"https://github.com/openstreetmap/openstreetmap-website/pull/1899#issuecomment-396961140","url":"https://github.com/openstreetmap/openstreetmap-website/pull/1899#issuecomment-396961140","name":"View Pull Request"},"description":"View this Pull Request on GitHub","publisher":{"@type":"Organization","name":"GitHub","url":"https://github.com"}}</script>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/openstreetmap/openstreetmap-website","title":"openstreetmap/openstreetmap-website","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/openstreetmap/openstreetmap-website"}},"updates":{"snippets":[{"icon":"PERSON","message":"@tomhughes in #1899: I think a bigger problem is that you've changed the ruby code so that it unconditionally replaces the compiled versions of the methods with the ruby versions which will kind of defeat the whole point ;-)\r\n\r\nThe previous structure was designed so that it would still work even if somebody hadn't built the extension and would just be slower - if we're moving to a gem we can probably assume it will be compiled and delete the ruby versions."}],"action":{"name":"View Pull Request","url":"https://github.com/openstreetmap/openstreetmap-website/pull/1899#issuecomment-396961140"}}}</script>
<script type="application/ld+json">{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [openstreetmap/openstreetmap-website] Remove quad_tile library and extract to gem (#1899)",
"sections": [
{
"text": "",
"activityTitle": "**Tom Hughes**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@tomhughes",
"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\": \"openstreetmap/openstreetmap-website\",\n\"issueId\": 1899,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close pull request",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"PullRequestClose\",\n\"repositoryFullName\": \"openstreetmap/openstreetmap-website\",\n\"pullRequestId\": 1899\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/openstreetmap/openstreetmap-website/pull/1899#issuecomment-396961140"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 345341057\n}"
}
],
"themeColor": "26292E"
}</script>