{"action":"created","issue":{"url":"https://api.github.com/repos/hetland/pygridgen/issues/21","labels_url":"https://api.github.com/repos/hetland/pygridgen/issues/21/labels{/name}","comments_url":"https://api.github.com/repos/hetland/pygridgen/issues/21/comments","events_url":"https://api.github.com/repos/hetland/pygridgen/issues/21/events","html_url":"https://github.com/hetland/pygridgen/issues/21","id":57065553,"number":21,"title":"External libs not building on Mac OS X","user":{"login":"phobson","id":1163939,"avatar_url":"https://avatars.githubusercontent.com/u/1163939?v=3","gravatar_id":"","url":"https://api.github.com/users/phobson","html_url":"https://github.com/phobson","followers_url":"https://api.github.com/users/phobson/followers","following_url":"https://api.github.com/users/phobson/following{/other_user}","gists_url":"https://api.github.com/users/phobson/gists{/gist_id}","starred_url":"https://api.github.com/users/phobson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/phobson/subscriptions","organizations_url":"https://api.github.com/users/phobson/orgs","repos_url":"https://api.github.com/users/phobson/repos","events_url":"https://api.github.com/users/phobson/events{/privacy}","received_events_url":"https://api.github.com/users/phobson/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":17,"created_at":"2015-02-09T17:53:47Z","updated_at":"2015-07-17T07:40:52Z","closed_at":null,"body":"Using GDB, I've tracked down the segfaults and made some headway to fixing this. I'm still exploring different combinations of revisions of the libraries to see what works.\r\n\r\n### Background\r\nI could never get the external libs bundled with pygridgen to build. Instead I always pulled those from octant. In a recent overhaul, pygridgen's external libs were updated/reverted as necessary to build on linux\r\n\r\n\r\n### The way forward\r\nThe way I see it we have the following options:\r\n\r\n1. make the existing bundled libs work by patching them\r\n2. revert fully to the libs to the versions in octant (assuming those actually build on Mac's now)\r\n3. figure out how to make the most recent version of the libs compile on a Mac\r\n\r\nThe third option is the most appealing, I think for obvious reasons. Plus, if I can get those to work, then  I can upload the build to conda/binstar.org and then no one else has to worry about it until Apply overhauls Mac OS X and breaks everything anyway.\r\n\r\n### Etc\r\nMy other feeling is that if we go that route, we should explore using `git submodule` to package them. Then we pygridgen gets clone, live versions of those libs also get cloned and we can apply whatever OS-specific patches we need to make them compile."},"comment":{"url":"https://api.github.com/repos/hetland/pygridgen/issues/comments/122205457","html_url":"https://github.com/hetland/pygridgen/issues/21#issuecomment-122205457","issue_url":"https://api.github.com/repos/hetland/pygridgen/issues/21","id":122205457,"user":{"login":"nikhil003","id":5336184,"avatar_url":"https://avatars.githubusercontent.com/u/5336184?v=3","gravatar_id":"","url":"https://api.github.com/users/nikhil003","html_url":"https://github.com/nikhil003","followers_url":"https://api.github.com/users/nikhil003/followers","following_url":"https://api.github.com/users/nikhil003/following{/other_user}","gists_url":"https://api.github.com/users/nikhil003/gists{/gist_id}","starred_url":"https://api.github.com/users/nikhil003/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nikhil003/subscriptions","organizations_url":"https://api.github.com/users/nikhil003/orgs","repos_url":"https://api.github.com/users/nikhil003/repos","events_url":"https://api.github.com/users/nikhil003/events{/privacy}","received_events_url":"https://api.github.com/users/nikhil003/received_events","type":"User","site_admin":false},"created_at":"2015-07-17T07:40:52Z","updated_at":"2015-07-17T07:40:52Z","body":"I investigated it a bit further and i suppose there is no need to have a class. It just need require a change from \r\n\r\n```python\r\nself._libgridgen.gridgen_generategrid2.restype = ctypes.c_void_p\r\n```\r\nto \r\n```python\r\nself._libgridgen.gridgen_generategrid2.restype = ctypes.POINTER(ctypes.c_void_p)\r\n```\r\nand hopefully it should fix it without any more additions"}}