238351 2022-03-24 15:51:10 -0700 After r291814, we need to create the INSTALL_PATH for the iOS sandbox files 2022-03-24 16:38:40 -0700 1 1 1 Unclassified WebKit Platform WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 msaboff msaboff ap emw saam webkit-bug-importer oldest_to_newest 1854885 0 msaboff 2022-03-24 15:51:10 -0700 Before r291814, some of the iOS sandbox files were copied to $(INSTALL_PATH). That built in Xcode copying stpe created the INSTALL_PATH directory. When the output of the preprocessed .sb.in files where copied to $(INSTALL_PATH), it already existed. r291814 made it so that all of the iOS sandbox files are preprocessed and then copied to $(INSTALL_PATH) using the "cp" command. The problem is that $(INSTALL_PATH) hasn't been created yet. This appears to only manifest in the Apple build system and not local builds. 1854886 1 msaboff 2022-03-24 15:51:28 -0700 <rdar://90794287> 1854894 2 455698 msaboff 2022-03-24 16:03:06 -0700 Created attachment 455698 Patch 1854901 3 455698 saam 2022-03-24 16:07:11 -0700 Comment on attachment 455698 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=455698&action=review r=me > Source/WebKit/ChangeLog:11 > + That change set made it soe that all WebKit iOS sandbox profile files are preprocessed soe => so 1854923 4 ap 2022-03-24 16:24:09 -0700 I was going to ask if we need to add the directory to output paths because of this, then noticed that there aren't any inputs or outputs here. 1854932 5 msaboff 2022-03-24 16:38:40 -0700 Committed r291828 (248850@trunk): <https://commits.webkit.org/248850@trunk> 455698 2022-03-24 16:03:06 -0700 2022-03-24 16:07:11 -0700 Patch 238351.patch text/plain 2712 msaboff SW5kZXg6IFNvdXJjZS9XZWJLaXQvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJL aXQvQ2hhbmdlTG9nCShyZXZpc2lvbiAyOTE4MTgpCisrKyBTb3VyY2UvV2ViS2l0L0NoYW5nZUxv Zwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE4IEBACisyMDIyLTAzLTI0ICBNaWNoYWVsIFNh Ym9mZiAgPG1zYWJvZmZAYXBwbGUuY29tPgorCisgICAgICAgIEFmdGVyIHIyOTE4MTQsIHdlIG5l ZWQgdG8gY3JlYXRlIHRoZSBJTlNUQUxMX1BBVEggZm9yIHRoZSBpT1Mgc2FuZGJveCBmaWxlcwor ICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjM4MzUxCisK KyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgV2UgbmVlZCB0 byBjcmVhdGUgdGhlICQoSU5TVEFMTF9QQVRIKSBmb3IgdGhlIHByZXByb2Nlc3NlZCBpT1Mgc2Fu ZGJveCBmaWxlcy4KKyAgICAgICAgQmVmb3JlIHIyOTE4MTQsIHRoZSBidWlsdC1pbiBYY29kZSBj b3B5IHN0ZXAgdXNlZCBkaXR0byB3aGVuIGNvcHlpbmcgdGhlIC5zYgorICAgICAgICBmaWxlcyB0 aHVzIGNyZWF0aW5nICQoSU5TVEFMTF9QQVRIKSBhbmQgaXRzIGludGVybWVkaWF0ZSBkaXJlY3Rv cmllcy4KKyAgICAgICAgVGhhdCBjaGFuZ2Ugc2V0IG1hZGUgaXQgc29lIHRoYXQgYWxsIFdlYktp dCBpT1Mgc2FuZGJveCBwcm9maWxlIGZpbGVzIGFyZSBwcmVwcm9jZXNzZWQKKyAgICAgICAgdGh1 cyBlbGltaW5hdGluZyB0aGUgWGNvZGUgY29weSBzdGVwLgorCisgICAgICAgICogV2ViS2l0Lnhj b2RlcHJvai9wcm9qZWN0LnBieHByb2o6CisKIDIwMjItMDMtMjQgIEJyZW50IEZ1bGdoYW0gIDxi ZnVsZ2hhbUBhcHBsZS5jb20+CiAKICAgICAgICAgRGlzYWJsZSBSVENSdHBTY3JpcHRUcmFuc2Zv cm0gaW4gQ2FwdGl2ZVBvcnRhbCBtb2RlCkluZGV4OiBTb3VyY2UvV2ViS2l0L1dlYktpdC54Y29k ZXByb2ovcHJvamVjdC5wYnhwcm9qCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJLaXQvV2ViS2l0 Lnhjb2RlcHJvai9wcm9qZWN0LnBieHByb2oJKHJldmlzaW9uIDI5MTgxNCkKKysrIFNvdXJjZS9X ZWJLaXQvV2ViS2l0Lnhjb2RlcHJvai9wcm9qZWN0LnBieHByb2oJKHdvcmtpbmcgY29weSkKQEAg LTE2ODA4LDcgKzE2ODA4LDcgQEAKIAkJCSk7CiAJCQlydW5Pbmx5Rm9yRGVwbG95bWVudFBvc3Rw cm9jZXNzaW5nID0gMDsKIAkJCXNoZWxsUGF0aCA9IC9iaW4vc2g7Ci0JCQlzaGVsbFNjcmlwdCA9 ICJlY2hvIFwiUHJlcHJvY2Vzc2luZyBzYW5kYm94XCJcblNjcmlwdHMvZ2VuZXJhdGUtZGVyaXZl ZC1zb3VyY2VzLnNoIHNhbmRib3gtcHJvZmlsZXMtaW9zXG5jcCAke0JVSUxUX1BST0RVQ1RTX0RJ Un0vRGVyaXZlZFNvdXJjZXMvV2ViS2l0L2NvbS5hcHBsZS5XZWJLaXQuYWRhdHRyaWJ1dGlvbmQu c2IgJHtEU1RST09UfS8ke0lOU1RBTExfUEFUSH1cbmNwICR7QlVJTFRfUFJPRFVDVFNfRElSfS9E ZXJpdmVkU291cmNlcy9XZWJLaXQvY29tLmFwcGxlLldlYktpdC5HUFUuc2IgJHtEU1RST09UfS8k e0lOU1RBTExfUEFUSH1cbmNwICR7QlVJTFRfUFJPRFVDVFNfRElSfS9EZXJpdmVkU291cmNlcy9X ZWJLaXQvY29tLmFwcGxlLldlYktpdC5OZXR3b3JraW5nLnNiICR7RFNUUk9PVH0vJHtJTlNUQUxM X1BBVEh9XG5jcCAke0JVSUxUX1BST0RVQ1RTX0RJUn0vRGVyaXZlZFNvdXJjZXMvV2ViS2l0L2Nv bS5hcHBsZS5XZWJLaXQuV2ViQXV0aC5zYiAke0RTVFJPT1R9LyR7SU5TVEFMTF9QQVRIfVxuY3Ag JHtCVUlMVF9QUk9EVUNUU19ESVJ9L0Rlcml2ZWRTb3VyY2VzL1dlYktpdC9jb20uYXBwbGUuV2Vi S2l0LldlYkNvbnRlbnQuc2IgJHtEU1RST09UfS8ke0lOU1RBTExfUEFUSH1cbiI7CisJCQlzaGVs bFNjcmlwdCA9ICJlY2hvIFwiUHJlcHJvY2Vzc2luZyBzYW5kYm94XCJcblNjcmlwdHMvZ2VuZXJh dGUtZGVyaXZlZC1zb3VyY2VzLnNoIHNhbmRib3gtcHJvZmlsZXMtaW9zXG5ta2RpciAtcCAke0RT VFJPT1R9LyR7SU5TVEFMTF9QQVRIfVxuY3AgJHtCVUlMVF9QUk9EVUNUU19ESVJ9L0Rlcml2ZWRT b3VyY2VzL1dlYktpdC9jb20uYXBwbGUuV2ViS2l0LmFkYXR0cmlidXRpb25kLnNiICR7RFNUUk9P VH0vJHtJTlNUQUxMX1BBVEh9XG5jcCAke0JVSUxUX1BST0RVQ1RTX0RJUn0vRGVyaXZlZFNvdXJj ZXMvV2ViS2l0L2NvbS5hcHBsZS5XZWJLaXQuR1BVLnNiICR7RFNUUk9PVH0vJHtJTlNUQUxMX1BB VEh9XG5jcCAke0JVSUxUX1BST0RVQ1RTX0RJUn0vRGVyaXZlZFNvdXJjZXMvV2ViS2l0L2NvbS5h cHBsZS5XZWJLaXQuTmV0d29ya2luZy5zYiAke0RTVFJPT1R9LyR7SU5TVEFMTF9QQVRIfVxuY3Ag JHtCVUlMVF9QUk9EVUNUU19ESVJ9L0Rlcml2ZWRTb3VyY2VzL1dlYktpdC9jb20uYXBwbGUuV2Vi S2l0LldlYkF1dGguc2IgJHtEU1RST09UfS8ke0lOU1RBTExfUEFUSH1cbmNwICR7QlVJTFRfUFJP RFVDVFNfRElSfS9EZXJpdmVkU291cmNlcy9XZWJLaXQvY29tLmFwcGxlLldlYktpdC5XZWJDb250 ZW50LnNiICR7RFNUUk9PVH0vJHtJTlNUQUxMX1BBVEh9XG4iOwogCQl9OwogLyogRW5kIFBCWFNo ZWxsU2NyaXB0QnVpbGRQaGFzZSBzZWN0aW9uICovCiAK