Closed Bug 805202 Opened 7 years ago Closed 7 years ago
depend on gio 2
.20 for --enable-gio builds and directly call g _app _info _get _commandline()
Per bug 803815 comment 5, not depending on glib 2.20 back in 2009 when the dlopen() code was written, but those days it should be safe enough, and simplifies the code.
Comment on attachment 674839 [details] [diff] [review] Depend on gio 2.20, directly call g_app_info_get_commandline Untested, but you get the idea.. i'll do some real testing tmrw.
Comment on attachment 674839 [details] [diff] [review] Depend on gio 2.20, directly call g_app_info_get_commandline The docs say that g_app_info_get_commandline can return NULL, so I think we still need the null check on the const char * return value. Otherwise, looks good thanks.
Damn right, totally overlooked that.
https://tbpl.mozilla.org/?tree=Try&rev=428f53b1197f just to be on the safe side, since i have no idea what gio version is installed on the linux builders...
All green on try. https://hg.mozilla.org/integration/mozilla-inbound/rev/a4e9d21b8ca8
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
You need to log in before you can comment on or make changes to this bug.