{"action":"created","issue":{"url":"https://api.github.com/repos/ldmud/ldmud/issues/17","labels_url":"https://api.github.com/repos/ldmud/ldmud/issues/17/labels{/name}","comments_url":"https://api.github.com/repos/ldmud/ldmud/issues/17/comments","events_url":"https://api.github.com/repos/ldmud/ldmud/issues/17/events","html_url":"https://github.com/ldmud/ldmud/pull/17","id":95900627,"number":17,"title":"Cleanup of pre-compiled (provided) autoconf files (attempt 2)","user":{"login":"ktiedt","id":97242,"avatar_url":"https://avatars.githubusercontent.com/u/97242?v=3","gravatar_id":"","url":"https://api.github.com/users/ktiedt","html_url":"https://github.com/ktiedt","followers_url":"https://api.github.com/users/ktiedt/followers","following_url":"https://api.github.com/users/ktiedt/following{/other_user}","gists_url":"https://api.github.com/users/ktiedt/gists{/gist_id}","starred_url":"https://api.github.com/users/ktiedt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ktiedt/subscriptions","organizations_url":"https://api.github.com/users/ktiedt/orgs","repos_url":"https://api.github.com/users/ktiedt/repos","events_url":"https://api.github.com/users/ktiedt/events{/privacy}","received_events_url":"https://api.github.com/users/ktiedt/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-19T09:16:20Z","updated_at":"2016-01-22T20:00:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ldmud/ldmud/pulls/17","html_url":"https://github.com/ldmud/ldmud/pull/17","diff_url":"https://github.com/ldmud/ldmud/pull/17.diff","patch_url":"https://github.com/ldmud/ldmud/pull/17.patch"},"body":"@zesstra Getting back to working on my side project a bit and realized I could do better than my last attempt to remove these from .gitignore so here is my next solution (if you decide this is unwarranted, I will not press this issue again)\r\n\r\nBasically, I updated autogen.sh to copy the provided files INTO src from (newly created) ../autoconf if autoconf is not found and added src/update-autoconf.sh which will copy src/configure and src/machine.h.in into autoconf/* for updating purposes\r\n\r\nI only have Ubuntu to test on, but from my understanding the autoconf check should be pretty solid across most systems.\r\n\r\nThanks again and I hope you find this idea more compelling than just deleting the files and letting everyone fend for themselves ;)"},"comment":{"url":"https://api.github.com/repos/ldmud/ldmud/issues/comments/174029456","html_url":"https://github.com/ldmud/ldmud/pull/17#issuecomment-174029456","issue_url":"https://api.github.com/repos/ldmud/ldmud/issues/17","id":174029456,"user":{"login":"zesstra","id":131563,"avatar_url":"https://avatars.githubusercontent.com/u/131563?v=3","gravatar_id":"","url":"https://api.github.com/users/zesstra","html_url":"https://github.com/zesstra","followers_url":"https://api.github.com/users/zesstra/followers","following_url":"https://api.github.com/users/zesstra/following{/other_user}","gists_url":"https://api.github.com/users/zesstra/gists{/gist_id}","starred_url":"https://api.github.com/users/zesstra/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zesstra/subscriptions","organizations_url":"https://api.github.com/users/zesstra/orgs","repos_url":"https://api.github.com/users/zesstra/repos","events_url":"https://api.github.com/users/zesstra/events{/privacy}","received_events_url":"https://api.github.com/users/zesstra/received_events","type":"User","site_admin":false},"created_at":"2016-01-22T20:00:07Z","updated_at":"2016-01-22T20:00:07Z","body":"Hello Karl,\r\n\r\nI am sorry for being totally absent for this long time. I am trying to get back into being more active.\r\nI had a look at it and discussed shortly with Gnomi. We would like to merge the suggestion.\r\nHowever there is one detail: hash is not suitable for finding commands - it knows only of them once you used it. (So, if you start with a clean hash table, 'hash autoconf' will have no result even if autoconf ist installed.)\r\nTherefore we have to come up with a better plan for detecting autoconf portably. Gnomi suggests to use 'command -v' which should be portable on POSIX systems: http://pubs.opengroup.org/onlinepubs/9699919799/utilities/command.html\r\nAre you willing to update the pullrequest with this?\r\n\r\nZesstra\r\n"}}