{"action":"created","issue":{"url":"https://api.github.com/repos/MLton/mlton/issues/151","repository_url":"https://api.github.com/repos/MLton/mlton","labels_url":"https://api.github.com/repos/MLton/mlton/issues/151/labels{/name}","comments_url":"https://api.github.com/repos/MLton/mlton/issues/151/comments","events_url":"https://api.github.com/repos/MLton/mlton/issues/151/events","html_url":"https://github.com/MLton/mlton/pull/151","id":160592500,"number":151,"title":"Try to not violate W^X","user":{"login":"AlexanderAA","id":202360,"avatar_url":"https://avatars.githubusercontent.com/u/202360?v=3","gravatar_id":"","url":"https://api.github.com/users/AlexanderAA","html_url":"https://github.com/AlexanderAA","followers_url":"https://api.github.com/users/AlexanderAA/followers","following_url":"https://api.github.com/users/AlexanderAA/following{/other_user}","gists_url":"https://api.github.com/users/AlexanderAA/gists{/gist_id}","starred_url":"https://api.github.com/users/AlexanderAA/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AlexanderAA/subscriptions","organizations_url":"https://api.github.com/users/AlexanderAA/orgs","repos_url":"https://api.github.com/users/AlexanderAA/repos","events_url":"https://api.github.com/users/AlexanderAA/events{/privacy}","received_events_url":"https://api.github.com/users/AlexanderAA/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-06-16T07:10:09Z","updated_at":"2016-06-22T10:59:19Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/MLton/mlton/pulls/151","html_url":"https://github.com/MLton/mlton/pull/151","diff_url":"https://github.com/MLton/mlton/pull/151.diff","patch_url":"https://github.com/MLton/mlton/pull/151.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/MLton/mlton/issues/comments/227710656","html_url":"https://github.com/MLton/mlton/pull/151#issuecomment-227710656","issue_url":"https://api.github.com/repos/MLton/mlton/issues/151","id":227710656,"user":{"login":"MatthewFluet","id":931523,"avatar_url":"https://avatars.githubusercontent.com/u/931523?v=3","gravatar_id":"","url":"https://api.github.com/users/MatthewFluet","html_url":"https://github.com/MatthewFluet","followers_url":"https://api.github.com/users/MatthewFluet/followers","following_url":"https://api.github.com/users/MatthewFluet/following{/other_user}","gists_url":"https://api.github.com/users/MatthewFluet/gists{/gist_id}","starred_url":"https://api.github.com/users/MatthewFluet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MatthewFluet/subscriptions","organizations_url":"https://api.github.com/users/MatthewFluet/orgs","repos_url":"https://api.github.com/users/MatthewFluet/repos","events_url":"https://api.github.com/users/MatthewFluet/events{/privacy}","received_events_url":"https://api.github.com/users/MatthewFluet/received_events","type":"User","site_admin":false},"created_at":"2016-06-22T10:59:18Z","updated_at":"2016-06-22T10:59:18Z","body":"Sorry, I misread the patch and thought that it was adding `PROT_EXEC`, not removing it.\r\n\r\nFor the record, the `PROT_EXEC` was purposefully added:\r\n\r\n* http://www.mlton.org/pipermail/mlton-devel/2004-December/026512.html\r\n\r\nand removing the `PROT_EXEC` has come up before:\r\n\r\n* http://www.mlton.org/pipermail/mlton-devel/2008-February/030169.html\r\n\r\nAt the time, I made a suggestion for conditionally adding `PROT_EXEC` via a\r\n`#define` configuration parameter:\r\n\r\n* http://www.mlton.org/pipermail/mlton-devel/2008-February/030175.html\r\n\r\nbut that seems to never have materialized.\r\n\r\nOf course, the `PROT_EXEC` seems only necessary for platforms that we don't really seem to support any more."}}
{"actor":{"display_login":"MatthewFluet"}}