173801 2017-06-23 18:29:10 -0700 [GTK+] Homebrew is not allowed to run any script under sudo. 2019-08-12 17:53:20 -0700 1 1 1 Unclassified WebKit Tools / Tests WebKit Local Build Unspecified Unspecified RESOLVED FIXED https://github.com/WebKit/webkit/blob/master/Tools/gtk/install-dependencies#L3 InRadar P2 Normal --- 1 yhorg webkit-unassigned agomez ashvayka clopez commit-queue dhaval ews-watchlist lforschler ltilve rniwa webkit-bug-importer oldest_to_newest 1322532 0 yhorg 2017-06-23 18:29:10 -0700 /Tools/gtk/install-dependencies ... line 3 # This script needs to be run with root rights. if [ $UID -ne 0 ]; then sudo $0 exit 0 fi These code snippet will cause error while installing dependencies. 1501508 1 dhaval 2019-02-02 11:20:28 -0800 fix proposed at https://github.com/WebKit/webkit/pull/30 1543574 2 371812 ashvayka 2019-06-10 19:48:24 -0700 Created attachment 371812 Patch 1543575 3 ashvayka 2019-06-10 19:52:52 -0700 Tools/gtk/install-dependencies works fine for me w/o root privileges. Homebrew 2.1.4 macOS 10.14.4 1543784 4 dhaval 2019-06-11 12:00:41 -0700 Yeah that patch makes more sense. Brew should not run as root. 1560633 5 371812 commit-queue 2019-08-12 17:52:55 -0700 Comment on attachment 371812 Patch Clearing flags on attachment: 371812 Committed r248584: <https://trac.webkit.org/changeset/248584> 1560634 6 commit-queue 2019-08-12 17:52:57 -0700 All reviewed patches have been landed. Closing bug. 1560635 7 webkit-bug-importer 2019-08-12 17:53:20 -0700 <rdar://problem/54235619> 371812 2019-06-10 19:48:24 -0700 2019-08-12 17:52:55 -0700 Patch bug-173801-20190611054823.patch text/plain 1143 ashvayka SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl dmlzaW9uIDI0NjMwMikKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz ICsxLDE1IEBACisyMDE5LTA2LTEwICBBbGV4ZXkgU2h2YXlrYSAgPHNodmFpa2FsZXNoQGdtYWls LmNvbT4KKworICAgICAgICBBWDogSG9tZWJyZXcgaXMgbm90IGFsbG93ZWQgdG8gcnVuIGFueSBz Y3JpcHQgdW5kZXIgc3Vkby4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i dWcuY2dpP2lkPTE3MzgwMQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor CisgICAgICAgIExhdGVzdCB2ZXJzaW9ucyBvZiBIb21lYnJldyB0aHJvdyBhbiBlcnJvciB3aGVu IHJ1biBhcyByb290LgorICAgICAgICBEZXBlbmRlbmNpZXMgYXJlIHN1Y2Nlc3NmdWxseSBpbnN0 YWxsZWQgdy9vIGBzdWRvYCBvbiBtYWNPUywgc28gc2tpcCBpdC4KKworICAgICAgICAqIGd0ay9p bnN0YWxsLWRlcGVuZGVuY2llczoKKwogMjAxOS0wNi0xMCAgU2FtIFdlaW5pZyAgPHdlaW5pZ0Bh cHBsZS5jb20+CiAKICAgICAgICAgUmVtb3ZlIERhc2hib2FyZCBzdXBwb3J0CkluZGV4OiBUb29s cy9ndGsvaW5zdGFsbC1kZXBlbmRlbmNpZXMKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gVG9vbHMvZ3RrL2luc3Rh bGwtZGVwZW5kZW5jaWVzCShyZXZpc2lvbiAyNDYwNjcpCisrKyBUb29scy9ndGsvaW5zdGFsbC1k ZXBlbmRlbmNpZXMJKHdvcmtpbmcgY29weSkKQEAgLTEsNyArMSw3IEBACiAjIS91c3IvYmluL2Vu diBiYXNoCiAKLSMgVGhpcyBzY3JpcHQgbmVlZHMgdG8gYmUgcnVuIHdpdGggcm9vdCByaWdodHMu Ci1pZiBbICRVSUQgLW5lIDAgXTsgdGhlbgorIyBPbiBMaW51eCBzeXN0ZW1zLCB0aGlzIHNjcmlw dCBuZWVkcyB0byBiZSBydW4gd2l0aCByb290IHJpZ2h0cy4KK2lmIFsgYHVuYW1lYCAhPSAiRGFy d2luIiBdICYmIFsgJFVJRCAtbmUgMCBdOyB0aGVuCiAgICAgc3VkbyAkMAogICAgIGV4aXQgMAog ZmkK