{"action":"created","issue":{"url":"https://api.github.com/repos/fontforge/fontforge/issues/2120","labels_url":"https://api.github.com/repos/fontforge/fontforge/issues/2120/labels{/name}","comments_url":"https://api.github.com/repos/fontforge/fontforge/issues/2120/comments","events_url":"https://api.github.com/repos/fontforge/fontforge/issues/2120/events","html_url":"https://github.com/fontforge/fontforge/issues/2120","id":60177299,"number":2120,"title":"Bring back breakpad on osx","user":{"login":"monkeyiq","id":387108,"avatar_url":"https://avatars.githubusercontent.com/u/387108?v=3","gravatar_id":"","url":"https://api.github.com/users/monkeyiq","html_url":"https://github.com/monkeyiq","followers_url":"https://api.github.com/users/monkeyiq/followers","following_url":"https://api.github.com/users/monkeyiq/following{/other_user}","gists_url":"https://api.github.com/users/monkeyiq/gists{/gist_id}","starred_url":"https://api.github.com/users/monkeyiq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/monkeyiq/subscriptions","organizations_url":"https://api.github.com/users/monkeyiq/orgs","repos_url":"https://api.github.com/users/monkeyiq/repos","events_url":"https://api.github.com/users/monkeyiq/events{/privacy}","received_events_url":"https://api.github.com/users/monkeyiq/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/fontforge/fontforge/labels/clarifying","name":"clarifying","color":"d4c5f9"},{"url":"https://api.github.com/repos/fontforge/fontforge/labels/High%20Priority","name":"High Priority","color":"e11d21"}],"state":"open","locked":false,"assignee":{"login":"monkeyiq","id":387108,"avatar_url":"https://avatars.githubusercontent.com/u/387108?v=3","gravatar_id":"","url":"https://api.github.com/users/monkeyiq","html_url":"https://github.com/monkeyiq","followers_url":"https://api.github.com/users/monkeyiq/followers","following_url":"https://api.github.com/users/monkeyiq/following{/other_user}","gists_url":"https://api.github.com/users/monkeyiq/gists{/gist_id}","starred_url":"https://api.github.com/users/monkeyiq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/monkeyiq/subscriptions","organizations_url":"https://api.github.com/users/monkeyiq/orgs","repos_url":"https://api.github.com/users/monkeyiq/repos","events_url":"https://api.github.com/users/monkeyiq/events{/privacy}","received_events_url":"https://api.github.com/users/monkeyiq/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/fontforge/fontforge/milestones/13","html_url":"https://github.com/fontforge/fontforge/milestones/2015-03%20Release","labels_url":"https://api.github.com/repos/fontforge/fontforge/milestones/13/labels","id":998180,"number":13,"title":"2015-03 Release","description":"","creator":{"login":"davelab6","id":261579,"avatar_url":"https://avatars.githubusercontent.com/u/261579?v=3","gravatar_id":"","url":"https://api.github.com/users/davelab6","html_url":"https://github.com/davelab6","followers_url":"https://api.github.com/users/davelab6/followers","following_url":"https://api.github.com/users/davelab6/following{/other_user}","gists_url":"https://api.github.com/users/davelab6/gists{/gist_id}","starred_url":"https://api.github.com/users/davelab6/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/davelab6/subscriptions","organizations_url":"https://api.github.com/users/davelab6/orgs","repos_url":"https://api.github.com/users/davelab6/repos","events_url":"https://api.github.com/users/davelab6/events{/privacy}","received_events_url":"https://api.github.com/users/davelab6/received_events","type":"User","site_admin":false},"open_issues":27,"closed_issues":7,"state":"open","created_at":"2015-02-27T02:01:40Z","updated_at":"2015-03-12T07:02:08Z","due_on":"2015-03-27T04:00:00Z","closed_at":null},"comments":24,"created_at":"2015-03-07T00:05:49Z","updated_at":"2015-03-13T12:10:55Z","closed_at":null,"body":"The build 201412/14_0426 marked the merge of breakpad for the client side on osx. Unfortunately the breakpad functionality stopped working on 201412/20_0427 and later builds. I notice a commit at around that time https://github.com/fontforge/fontforge/commit/07834e53d33dd5356b468569edb4873cd700e1ec which disables listen_to_apple_events. Disabling apple_events also disables a whole block which implements the main loop on osx, falling back to the general main loop: [startui.c#L1359](https://github.com/fontforge/fontforge/blob/master/fontforgeexe/startui.c#L1359).\r\n\r\nI'm happy to not rely on RunApplicationEventLoop() on osx, but instead, after installing the apple_event handlers and the cocoa application, just use the same event handling main loop as Linux and Win use. Unfortunately, if you disable the cocoa application setup then you also disable the breakpad handling which wants to hook into the objective-c Application object [MacBreakpadStarterGuide](http://code.google.com/p/google-breakpad/wiki/MacBreakpadStarterGuide#Write_Code!).\r\n\r\n@frank-trampe I realize that it's months on now, but do you recall which issues were being solved by https://github.com/fontforge/fontforge/commit/07834e53d33dd5356b468569edb4873cd700e1ec so that I can try to keep an eye out for them when I reenable part of the apple_events code.\r\n"},"comment":{"url":"https://api.github.com/repos/fontforge/fontforge/issues/comments/78942745","html_url":"https://github.com/fontforge/fontforge/issues/2120#issuecomment-78942745","issue_url":"https://api.github.com/repos/fontforge/fontforge/issues/2120","id":78942745,"user":{"login":"monkeyiq","id":387108,"avatar_url":"https://avatars.githubusercontent.com/u/387108?v=3","gravatar_id":"","url":"https://api.github.com/users/monkeyiq","html_url":"https://github.com/monkeyiq","followers_url":"https://api.github.com/users/monkeyiq/followers","following_url":"https://api.github.com/users/monkeyiq/following{/other_user}","gists_url":"https://api.github.com/users/monkeyiq/gists{/gist_id}","starred_url":"https://api.github.com/users/monkeyiq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/monkeyiq/subscriptions","organizations_url":"https://api.github.com/users/monkeyiq/orgs","repos_url":"https://api.github.com/users/monkeyiq/repos","events_url":"https://api.github.com/users/monkeyiq/events{/privacy}","received_events_url":"https://api.github.com/users/monkeyiq/received_events","type":"User","site_admin":false},"created_at":"2015-03-13T12:10:55Z","updated_at":"2015-03-13T12:10:55Z","body":"An example with a fairly ugly intermediate web page is uploading to 201503/13_2155\r\n\r\nSee osx/FontForge.app/Contents/Resources/breakpad/webinterface for the current location of the web interface files. The index.html uses handlebar like substitution and the script will allow you to load png, css, and js files with a very limited naming scheme from the webinterface directory. The file name limitations are in place to avoid serving files that are not in that directory.\r\n\r\nfiles like foo.png and alice.css are fine, and such simple file names should be ok if they are just used to build this intermediate web page.\r\n\r\nThe github link should take you there, possibly needing you to login as normal, and a new issue will be setup with a body showing the backtrace.\r\n\r\nSo, in other words this is now getting close to being mergable. The index.html obviously could use liberal sprinklings of love to look better, but others are likely to know how to do that better than me. If we want to merge a template in then I can help lift or modify the file name restrictions for that part.\r\n\r\nI might also push gist link support for folks who don't want to log in to anything."}}