221799 2021-02-11 18:33:07 -0800 'sync-commits.py' should be able to limit reporting scope to a specific branch on a Git repository. 2021-02-12 15:05:48 -0800 1 1 1 Unclassified WebKit New Bugs WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 dewei_zhu dewei_zhu dewei_zhu rniwa oldest_to_newest 1728411 0 dewei_zhu 2021-02-11 18:33:07 -0800 'sync-commits.py' should be able to limit reporting scope to a specific branch on a Git repository. 1728412 1 420076 dewei_zhu 2021-02-11 18:36:59 -0800 Created attachment 420076 Patch 1728413 2 420077 dewei_zhu 2021-02-11 18:38:15 -0800 Created attachment 420077 Patch 1728491 3 420077 rniwa 2021-02-12 01:33:56 -0800 Comment on attachment 420077 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=420077&action=review I guess we don't have any tests for this script... We probably should. > Websites/perf.webkit.org/tools/sync-commits.py:195 > - def __init__(self, name, git_checkout, git_url): > + def __init__(self, name, git_checkout, git_url, git_branch): should probably set the default parameter of None to signify the fact it's optional. 1728815 4 dewei_zhu 2021-02-12 15:05:48 -0800 rdar://74241548 Landed in r272814 420076 2021-02-11 18:36:59 -0800 2021-02-11 18:38:14 -0800 Patch bug-221799-20210211183659.patch text/plain 3244 dewei_zhu U3VidmVyc2lvbiBSZXZpc2lvbjogMjcyNzYwCmRpZmYgLS1naXQgYS9XZWJzaXRlcy9wZXJmLndl YmtpdC5vcmcvQ2hhbmdlTG9nIGIvV2Vic2l0ZXMvcGVyZi53ZWJraXQub3JnL0NoYW5nZUxvZwpp bmRleCA5MDc3ZmZkMDRjOTdhZmY0MDk4OWEyMTJmZGZmMDAyYzBiMmE1NDg0Li5mMjBiZDkxZWEz YWI4NTMzYWFiM2E5ZDg3NDY4NjliM2EwMWU2NzUwIDEwMDY0NAotLS0gYS9XZWJzaXRlcy9wZXJm LndlYmtpdC5vcmcvQ2hhbmdlTG9nCisrKyBiL1dlYnNpdGVzL3BlcmYud2Via2l0Lm9yZy9DaGFu Z2VMb2cKQEAgLTEsMyArMSwxOSBAQAorMjAyMS0wMi0xMSAgRGV3ZWkgWmh1ICA8ZGV3ZWlfemh1 QGFwcGxlLmNvbT4KKworICAgICAgICAnc3luYy1jb21taXRzLnB5JyBzaG91bGQgYmUgYWJsZSB0 byBsaW1pdCByZXBvcnRpbmcgc2NvcGUgdG8gYSBzcGVjaWZpYyBicmFuY2ggb24gYSBHaXQgcmVw b3NpdG9yeS4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lk PTIyMTc5OQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg IEZldGNoaW5nIGdpdCBjb21taXRzIHdpdGggJ2dpdCBsb2cgLS1hbGwgLi4uJyBtYXkgZW5kIHVw IGZldGNoaW5nIGNvbW1pdHMgZnJvbSBicmFuY2hlcyBjcmVhdGVkIGZvcgorICAgICAgICBwdWxs IHJlcXVlc3RzLiBBZGQgJ2JyYW5jaCcgdmFyaWFibGUgdG8gYWxsb3cgdXMgdG8gbGltaXQgdG8g YSBjZXJ0YWluIGJyYW5jaC4KKworICAgICAgICAqIHRvb2xzL3N5bmMtY29tbWl0cy5weTogQWRk ZWQgJ2JyYW5jaCcgdmFyaWFibGUgdG8gR2l0UmVwb3NpdG9yeSBzbyB0aGF0IHdlIGNhbiBmZXRj aCBmcm9tIGEgc3BlY2lmaWMgYnJhbmNoLgorICAgICAgICBJZiBicmFuY2ggaXMgbm90IHNwZWNp ZmllZCwgaXQgd2lsbCBzdGlsbCB1c2UgJy0tYWxsJyB0byBsaXN0IGNvbW1pdHMuCisgICAgICAg IChsb2FkX3JlcG9zaXRvcnkpOgorICAgICAgICAoR2l0UmVwb3NpdG9yeS5fX2luaXRfXyk6Cisg ICAgICAgIChHaXRSZXBvc2l0b3J5Ll9mZXRjaF9hbGxfaGFzaGVzKToKKwogMjAyMS0wMS0yMCAg RGV3ZWkgWmh1ICA8ZGV3ZWlfemh1QGFwcGxlLmNvbT4KIAogICAgICAgICBCdWlsZGJvdFRyaWdn ZXJhYmxlIHNob3VsZCBub3QgdXBkYXRlIGJ1aWxkIHJlcXVlc3RzIGZyb20gYW5vdGhlciB0cmln Z2VyYWJsZS4KZGlmZiAtLWdpdCBhL1dlYnNpdGVzL3BlcmYud2Via2l0Lm9yZy90b29scy9zeW5j LWNvbW1pdHMucHkgYi9XZWJzaXRlcy9wZXJmLndlYmtpdC5vcmcvdG9vbHMvc3luYy1jb21taXRz LnB5CmluZGV4IGE4M2M4ZTBkODU3YmEyNjA2NjZhZmQxZDk0YTJiNjU1YWQ0YTliMzIuLjhiMjA3 Zjg5ZDlmZDk2MmZiYWVlODUxNzhmM2Q0MjM0MzJkZmFlZmIgMTAwNzU1Ci0tLSBhL1dlYnNpdGVz L3BlcmYud2Via2l0Lm9yZy90b29scy9zeW5jLWNvbW1pdHMucHkKKysrIGIvV2Vic2l0ZXMvcGVy Zi53ZWJraXQub3JnL3Rvb2xzL3N5bmMtY29tbWl0cy5weQpAQCAtNDMsNyArNDMsNyBAQCBkZWYg bWFpbihhcmd2KToKIAogZGVmIGxvYWRfcmVwb3NpdG9yeShyZXBvc2l0b3J5KToKICAgICBpZiAn Z2l0Q2hlY2tvdXQnIGluIHJlcG9zaXRvcnk6Ci0gICAgICAgIHJldHVybiBHaXRSZXBvc2l0b3J5 KG5hbWU9cmVwb3NpdG9yeVsnbmFtZSddLCBnaXRfdXJsPXJlcG9zaXRvcnlbJ3VybCddLCBnaXRf Y2hlY2tvdXQ9cmVwb3NpdG9yeVsnZ2l0Q2hlY2tvdXQnXSkKKyAgICAgICAgcmV0dXJuIEdpdFJl cG9zaXRvcnkobmFtZT1yZXBvc2l0b3J5WyduYW1lJ10sIGdpdF91cmw9cmVwb3NpdG9yeVsndXJs J10sIGdpdF9jaGVja291dD1yZXBvc2l0b3J5WydnaXRDaGVja291dCddLCByZXBvc2l0b3J5Lmdl dCgnYnJhbmNoJykpCiAgICAgcmV0dXJuIFNWTlJlcG9zaXRvcnkobmFtZT1yZXBvc2l0b3J5Wydu YW1lJ10sIHN2bl91cmw9cmVwb3NpdG9yeVsndXJsJ10sIHNob3VsZF90cnVzdF9jZXJ0aWZpY2F0 ZT1yZXBvc2l0b3J5LmdldCgndHJ1c3RDZXJ0aWZpY2F0ZScsIEZhbHNlKSwKICAgICAgICAgdXNl X3NlcnZlcl9hdXRoPXJlcG9zaXRvcnkuZ2V0KCd1c2VTZXJ2ZXJBdXRoJywgRmFsc2UpLCBhY2Nv dW50X25hbWVfc2NyaXB0X3BhdGg9cmVwb3NpdG9yeS5nZXQoJ2FjY291bnROYW1lRmluZGVyU2Ny aXB0JykpCiAKQEAgLTE5MiwxMSArMTkyLDEyIEBAIGNsYXNzIFNWTlJlcG9zaXRvcnkoUmVwb3Np dG9yeSk6CiAKIGNsYXNzIEdpdFJlcG9zaXRvcnkoUmVwb3NpdG9yeSk6CiAKLSAgICBkZWYgX19p bml0X18oc2VsZiwgbmFtZSwgZ2l0X2NoZWNrb3V0LCBnaXRfdXJsKToKKyAgICBkZWYgX19pbml0 X18oc2VsZiwgbmFtZSwgZ2l0X2NoZWNrb3V0LCBnaXRfdXJsLCBnaXRfYnJhbmNoKToKICAgICAg ICAgYXNzZXJ0KG9zLnBhdGguaXNkaXIoZ2l0X2NoZWNrb3V0KSkKICAgICAgICAgc3VwZXIoR2l0 UmVwb3NpdG9yeSwgc2VsZikuX19pbml0X18obmFtZSkKICAgICAgICAgc2VsZi5fZ2l0X2NoZWNr b3V0ID0gZ2l0X2NoZWNrb3V0CiAgICAgICAgIHNlbGYuX2dpdF91cmwgPSBnaXRfdXJsCisgICAg ICAgIHNlbGYuX2dpdF9icmFuY2ggPSBnaXRfYnJhbmNoCiAgICAgICAgIHNlbGYuX3Rva2VuaXpl ZF9oYXNoZXMgPSBbXQogCiAgICAgZGVmIGZldGNoX25leHRfY29tbWl0KHNlbGYsIHNlcnZlcl9j b25maWcsIGxhc3RfZmV0Y2hlZCk6CkBAIC0yNDUsNyArMjQ2LDggQEAgY2xhc3MgR2l0UmVwb3Np dG9yeShSZXBvc2l0b3J5KToKIAogICAgIGRlZiBfZmV0Y2hfYWxsX2hhc2hlcyhzZWxmKToKICAg ICAgICAgc2VsZi5fcnVuX2dpdF9jb21tYW5kKFsncHVsbCcsIHNlbGYuX2dpdF91cmxdKQotICAg ICAgICBsaW5lcyA9IHNlbGYuX3J1bl9naXRfY29tbWFuZChbJ2xvZycsICctLWFsbCcsICctLWRh dGUtb3JkZXInLCAnLS1yZXZlcnNlJywgJy0tcHJldHR5PSVIICVjdCAlY2UgJVAnXSkuc3BsaXQo J1xuJykKKyAgICAgICAgc2NvcGUgPSBzZWxmLmdpdF9icmFuY2ggb3IgJy0tYWxsJworICAgICAg ICBsaW5lcyA9IHNlbGYuX3J1bl9naXRfY29tbWFuZChbJ2xvZycsIHNjb3BlLCAnLS1kYXRlLW9y ZGVyJywgJy0tcmV2ZXJzZScsICctLXByZXR0eT0lSCAlY3QgJWNlICVQJ10pLnNwbGl0KCdcbicp CiAgICAgICAgIHNlbGYuX3Rva2VuaXplZF9oYXNoZXMgPSBbbGluZS5zcGxpdCgpIGZvciBsaW5l IGluIGxpbmVzXQogCiAgICAgZGVmIF9ydW5fZ2l0X2NvbW1hbmQoc2VsZiwgYXJncyk6Cg== 420077 2021-02-11 18:38:15 -0800 2021-02-12 01:33:56 -0800 Patch bug-221799-20210211183815.patch text/plain 3245 dewei_zhu U3VidmVyc2lvbiBSZXZpc2lvbjogMjcyNzYwCmRpZmYgLS1naXQgYS9XZWJzaXRlcy9wZXJmLndl YmtpdC5vcmcvQ2hhbmdlTG9nIGIvV2Vic2l0ZXMvcGVyZi53ZWJraXQub3JnL0NoYW5nZUxvZwpp bmRleCA5MDc3ZmZkMDRjOTdhZmY0MDk4OWEyMTJmZGZmMDAyYzBiMmE1NDg0Li5mMjBiZDkxZWEz YWI4NTMzYWFiM2E5ZDg3NDY4NjliM2EwMWU2NzUwIDEwMDY0NAotLS0gYS9XZWJzaXRlcy9wZXJm LndlYmtpdC5vcmcvQ2hhbmdlTG9nCisrKyBiL1dlYnNpdGVzL3BlcmYud2Via2l0Lm9yZy9DaGFu Z2VMb2cKQEAgLTEsMyArMSwxOSBAQAorMjAyMS0wMi0xMSAgRGV3ZWkgWmh1ICA8ZGV3ZWlfemh1 QGFwcGxlLmNvbT4KKworICAgICAgICAnc3luYy1jb21taXRzLnB5JyBzaG91bGQgYmUgYWJsZSB0 byBsaW1pdCByZXBvcnRpbmcgc2NvcGUgdG8gYSBzcGVjaWZpYyBicmFuY2ggb24gYSBHaXQgcmVw b3NpdG9yeS4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lk PTIyMTc5OQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg IEZldGNoaW5nIGdpdCBjb21taXRzIHdpdGggJ2dpdCBsb2cgLS1hbGwgLi4uJyBtYXkgZW5kIHVw IGZldGNoaW5nIGNvbW1pdHMgZnJvbSBicmFuY2hlcyBjcmVhdGVkIGZvcgorICAgICAgICBwdWxs IHJlcXVlc3RzLiBBZGQgJ2JyYW5jaCcgdmFyaWFibGUgdG8gYWxsb3cgdXMgdG8gbGltaXQgdG8g YSBjZXJ0YWluIGJyYW5jaC4KKworICAgICAgICAqIHRvb2xzL3N5bmMtY29tbWl0cy5weTogQWRk ZWQgJ2JyYW5jaCcgdmFyaWFibGUgdG8gR2l0UmVwb3NpdG9yeSBzbyB0aGF0IHdlIGNhbiBmZXRj aCBmcm9tIGEgc3BlY2lmaWMgYnJhbmNoLgorICAgICAgICBJZiBicmFuY2ggaXMgbm90IHNwZWNp ZmllZCwgaXQgd2lsbCBzdGlsbCB1c2UgJy0tYWxsJyB0byBsaXN0IGNvbW1pdHMuCisgICAgICAg IChsb2FkX3JlcG9zaXRvcnkpOgorICAgICAgICAoR2l0UmVwb3NpdG9yeS5fX2luaXRfXyk6Cisg ICAgICAgIChHaXRSZXBvc2l0b3J5Ll9mZXRjaF9hbGxfaGFzaGVzKToKKwogMjAyMS0wMS0yMCAg RGV3ZWkgWmh1ICA8ZGV3ZWlfemh1QGFwcGxlLmNvbT4KIAogICAgICAgICBCdWlsZGJvdFRyaWdn ZXJhYmxlIHNob3VsZCBub3QgdXBkYXRlIGJ1aWxkIHJlcXVlc3RzIGZyb20gYW5vdGhlciB0cmln Z2VyYWJsZS4KZGlmZiAtLWdpdCBhL1dlYnNpdGVzL3BlcmYud2Via2l0Lm9yZy90b29scy9zeW5j LWNvbW1pdHMucHkgYi9XZWJzaXRlcy9wZXJmLndlYmtpdC5vcmcvdG9vbHMvc3luYy1jb21taXRz LnB5CmluZGV4IGE4M2M4ZTBkODU3YmEyNjA2NjZhZmQxZDk0YTJiNjU1YWQ0YTliMzIuLjFlYWVj ZGRiMTgxZWQ5YWQ0OTA3MjE3NzJkNzkwZmE1ZTI0Nzg3NDEgMTAwNzU1Ci0tLSBhL1dlYnNpdGVz L3BlcmYud2Via2l0Lm9yZy90b29scy9zeW5jLWNvbW1pdHMucHkKKysrIGIvV2Vic2l0ZXMvcGVy Zi53ZWJraXQub3JnL3Rvb2xzL3N5bmMtY29tbWl0cy5weQpAQCAtNDMsNyArNDMsNyBAQCBkZWYg bWFpbihhcmd2KToKIAogZGVmIGxvYWRfcmVwb3NpdG9yeShyZXBvc2l0b3J5KToKICAgICBpZiAn Z2l0Q2hlY2tvdXQnIGluIHJlcG9zaXRvcnk6Ci0gICAgICAgIHJldHVybiBHaXRSZXBvc2l0b3J5 KG5hbWU9cmVwb3NpdG9yeVsnbmFtZSddLCBnaXRfdXJsPXJlcG9zaXRvcnlbJ3VybCddLCBnaXRf Y2hlY2tvdXQ9cmVwb3NpdG9yeVsnZ2l0Q2hlY2tvdXQnXSkKKyAgICAgICAgcmV0dXJuIEdpdFJl cG9zaXRvcnkobmFtZT1yZXBvc2l0b3J5WyduYW1lJ10sIGdpdF91cmw9cmVwb3NpdG9yeVsndXJs J10sIGdpdF9jaGVja291dD1yZXBvc2l0b3J5WydnaXRDaGVja291dCddLCByZXBvc2l0b3J5Lmdl dCgnYnJhbmNoJykpCiAgICAgcmV0dXJuIFNWTlJlcG9zaXRvcnkobmFtZT1yZXBvc2l0b3J5Wydu YW1lJ10sIHN2bl91cmw9cmVwb3NpdG9yeVsndXJsJ10sIHNob3VsZF90cnVzdF9jZXJ0aWZpY2F0 ZT1yZXBvc2l0b3J5LmdldCgndHJ1c3RDZXJ0aWZpY2F0ZScsIEZhbHNlKSwKICAgICAgICAgdXNl X3NlcnZlcl9hdXRoPXJlcG9zaXRvcnkuZ2V0KCd1c2VTZXJ2ZXJBdXRoJywgRmFsc2UpLCBhY2Nv dW50X25hbWVfc2NyaXB0X3BhdGg9cmVwb3NpdG9yeS5nZXQoJ2FjY291bnROYW1lRmluZGVyU2Ny aXB0JykpCiAKQEAgLTE5MiwxMSArMTkyLDEyIEBAIGNsYXNzIFNWTlJlcG9zaXRvcnkoUmVwb3Np dG9yeSk6CiAKIGNsYXNzIEdpdFJlcG9zaXRvcnkoUmVwb3NpdG9yeSk6CiAKLSAgICBkZWYgX19p bml0X18oc2VsZiwgbmFtZSwgZ2l0X2NoZWNrb3V0LCBnaXRfdXJsKToKKyAgICBkZWYgX19pbml0 X18oc2VsZiwgbmFtZSwgZ2l0X2NoZWNrb3V0LCBnaXRfdXJsLCBnaXRfYnJhbmNoKToKICAgICAg ICAgYXNzZXJ0KG9zLnBhdGguaXNkaXIoZ2l0X2NoZWNrb3V0KSkKICAgICAgICAgc3VwZXIoR2l0 UmVwb3NpdG9yeSwgc2VsZikuX19pbml0X18obmFtZSkKICAgICAgICAgc2VsZi5fZ2l0X2NoZWNr b3V0ID0gZ2l0X2NoZWNrb3V0CiAgICAgICAgIHNlbGYuX2dpdF91cmwgPSBnaXRfdXJsCisgICAg ICAgIHNlbGYuX2dpdF9icmFuY2ggPSBnaXRfYnJhbmNoCiAgICAgICAgIHNlbGYuX3Rva2VuaXpl ZF9oYXNoZXMgPSBbXQogCiAgICAgZGVmIGZldGNoX25leHRfY29tbWl0KHNlbGYsIHNlcnZlcl9j b25maWcsIGxhc3RfZmV0Y2hlZCk6CkBAIC0yNDUsNyArMjQ2LDggQEAgY2xhc3MgR2l0UmVwb3Np dG9yeShSZXBvc2l0b3J5KToKIAogICAgIGRlZiBfZmV0Y2hfYWxsX2hhc2hlcyhzZWxmKToKICAg ICAgICAgc2VsZi5fcnVuX2dpdF9jb21tYW5kKFsncHVsbCcsIHNlbGYuX2dpdF91cmxdKQotICAg ICAgICBsaW5lcyA9IHNlbGYuX3J1bl9naXRfY29tbWFuZChbJ2xvZycsICctLWFsbCcsICctLWRh dGUtb3JkZXInLCAnLS1yZXZlcnNlJywgJy0tcHJldHR5PSVIICVjdCAlY2UgJVAnXSkuc3BsaXQo J1xuJykKKyAgICAgICAgc2NvcGUgPSBzZWxmLl9naXRfYnJhbmNoIG9yICctLWFsbCcKKyAgICAg ICAgbGluZXMgPSBzZWxmLl9ydW5fZ2l0X2NvbW1hbmQoWydsb2cnLCBzY29wZSwgJy0tZGF0ZS1v cmRlcicsICctLXJldmVyc2UnLCAnLS1wcmV0dHk9JUggJWN0ICVjZSAlUCddKS5zcGxpdCgnXG4n KQogICAgICAgICBzZWxmLl90b2tlbml6ZWRfaGFzaGVzID0gW2xpbmUuc3BsaXQoKSBmb3IgbGlu ZSBpbiBsaW5lc10KIAogICAgIGRlZiBfcnVuX2dpdF9jb21tYW5kKHNlbGYsIGFyZ3MpOgo=