Summary: | [GTK] Add libnotify in gtk install-dependencies | ||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joonghun Park <jh718.park> | ||||||||||||||
Component: | WebKitGTK | Assignee: | Joonghun Park <jh718.park> | ||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||
Severity: | Normal | CC: | commit-queue, gyuyoung.kim, mcatanzaro, ossy, zan | ||||||||||||||
Priority: | P2 | ||||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||||
Hardware: | Unspecified | ||||||||||||||||
OS: | Unspecified | ||||||||||||||||
Attachments: |
|
Description
Joonghun Park
2015-04-29 00:28:22 PDT
Created attachment 251929 [details]
Patch
Comment on attachment 251929 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=251929&action=review > Tools/gtk/install-dependencies:81 > + libnotify-dev \ This should use spaces instead of tabs. Please fix before committing. (In reply to comment #2) > Comment on attachment 251929 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=251929&action=review > > > Tools/gtk/install-dependencies:81 > > + libnotify-dev \ > > This should use spaces instead of tabs. Please fix before committing. Ok, I will revise it :) Created attachment 251933 [details]
Patch
Comment on attachment 251933 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=251933&action=review > Tools/ChangeLog:6 > + Reviewed by NOBODY (OOPS!). Missing reviewer name here, because Zan set r+ed. (In reply to comment #5) > Comment on attachment 251933 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=251933&action=review > > > Tools/ChangeLog:6 > > + Reviewed by NOBODY (OOPS!). > > Missing reviewer name here, because Zan set r+ed. Ah, I had to write it into ChangeLog, but I missed it. Thank you, I will re-upload this one. Created attachment 251934 [details]
Patch
Created attachment 251935 [details]
Patch
Created attachment 251936 [details]
Patch
Comment on attachment 251936 [details] Patch Clearing flags on attachment: 251936 Committed r183539: <http://trac.webkit.org/changeset/183539> All reviewed patches have been landed. Closing bug. Reopening: please make sure to fix all three supported distros (Arch, Debian, Fedora) when modifying this file. The Arch package is simply libnotify and the Fedora package is libnotify-devel. (In reply to comment #12) > Reopening: please make sure to fix all three supported distros (Arch, > Debian, Fedora) when modifying this file. The Arch package is simply > libnotify and the Fedora package is libnotify-devel. Has GTK port's install-dependency script supported Fedora, Arch as well ? (In reply to comment #13) > Has GTK port's install-dependency script supported Fedora, Arch as well ? Yup. (More distros welcome.) (In reply to comment #12) > Reopening: please make sure to fix all three supported distros (Arch, > Debian, Fedora) when modifying this file. The Arch package is simply > libnotify and the Fedora package is libnotify-devel. I didn't know which distros GTK port supports currently. Thank you for your teaching :) For now, I will add what you commented here for Fedora and Arch. Created attachment 252143 [details]
Patch
Comment on attachment 252143 [details]
Patch
LGTM
Comment on attachment 252143 [details] Patch Clearing flags on attachment: 252143 Committed r183743: <http://trac.webkit.org/changeset/183743> All reviewed patches have been landed. Closing bug. |