{"action":"created","issue":{"url":"https://api.github.com/repos/geany/geany/issues/606","labels_url":"https://api.github.com/repos/geany/geany/issues/606/labels{/name}","comments_url":"https://api.github.com/repos/geany/geany/issues/606/comments","events_url":"https://api.github.com/repos/geany/geany/issues/606/events","html_url":"https://github.com/geany/geany/pull/606","id":100121605,"number":606,"title":"Fix \"Open in New Window\"","user":{"login":"zhekov","id":1112714,"avatar_url":"https://avatars.githubusercontent.com/u/1112714?v=3","gravatar_id":"","url":"https://api.github.com/users/zhekov","html_url":"https://github.com/zhekov","followers_url":"https://api.github.com/users/zhekov/followers","following_url":"https://api.github.com/users/zhekov/following{/other_user}","gists_url":"https://api.github.com/users/zhekov/gists{/gist_id}","starred_url":"https://api.github.com/users/zhekov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zhekov/subscriptions","organizations_url":"https://api.github.com/users/zhekov/orgs","repos_url":"https://api.github.com/users/zhekov/repos","events_url":"https://api.github.com/users/zhekov/events{/privacy}","received_events_url":"https://api.github.com/users/zhekov/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":13,"created_at":"2015-08-10T17:23:18Z","updated_at":"2015-08-18T01:01:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/geany/geany/pulls/606","html_url":"https://github.com/geany/geany/pull/606","diff_url":"https://github.com/geany/geany/pull/606.diff","patch_url":"https://github.com/geany/geany/pull/606.patch"},"body":"My own attempt to fix #590, inspired by PR #597.\r\n\r\nWin~1: the Windows executable name is preffered. Failing that, the *nix procedure is used.\r\n\r\nOSX bundle: Bundle executable name is preffered. Failing that, open Geany is used, with --args for OSX >= 10.6.\r\n\r\nUnix: if argv[0] is not a basename, realpath(argv[0]) is preffered. Failing that, g_get_prgname (~= basename argv[0]) is attempted, and then \"geany\".\r\n\r\nWhoever has OSX, please test with \"app->executable = NULL\" instead of \"= gtkosx_application_get_executable_path()\" in libmain, and fix the MAC code in utils.c if needed.\r\n\r\nDue to some last-moment changes, there si a chance that the compilation under Win~1 may fail. I'll re-check it soon.\r\n\r\n@elextr \r\n\r\nFixed realpath(basename).\r\n\r\n@codebrainz \r\n\r\nIf I say once again that something may be done \"easy\" and \"platform independent\", reming me to hit myself with something heavy. Thanks in advance."},"comment":{"url":"https://api.github.com/repos/geany/geany/issues/comments/132010656","html_url":"https://github.com/geany/geany/pull/606#issuecomment-132010656","issue_url":"https://api.github.com/repos/geany/geany/issues/606","id":132010656,"user":{"login":"elextr","id":811085,"avatar_url":"https://avatars.githubusercontent.com/u/811085?v=3","gravatar_id":"","url":"https://api.github.com/users/elextr","html_url":"https://github.com/elextr","followers_url":"https://api.github.com/users/elextr/followers","following_url":"https://api.github.com/users/elextr/following{/other_user}","gists_url":"https://api.github.com/users/elextr/gists{/gist_id}","starred_url":"https://api.github.com/users/elextr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/elextr/subscriptions","organizations_url":"https://api.github.com/users/elextr/orgs","repos_url":"https://api.github.com/users/elextr/repos","events_url":"https://api.github.com/users/elextr/events{/privacy}","received_events_url":"https://api.github.com/users/elextr/received_events","type":"User","site_admin":false},"created_at":"2015-08-18T01:01:07Z","updated_at":"2015-08-18T01:01:07Z","body":"@codebrainz has made a good point that this is only about advanced users/developers and most Geany users have only one version and config.\r\n\r\nSo I would say that point greatly supports the case for allowing it to be configurable, the default does what Geany does now, `geany -i %f`, so most people won't even notice or care about its configurability, and we troublemakers can have it do whatever we want, because we clearly have differing use-cases and are unlikely to agree on one hard coded case :)"}}