harfbuzz-sys update

[paulrouget]

This change is 

[paulrouget]

@mbrubeck r?

[mbrubeck]

@bors-servo r+

[bors-servo]

📌 Commit 5f17f09 has been approved by mbrubeck

[paulrouget]

@bors-servo retry

• #18346

[bors-servo]

⌛ Testing commit 5f17f09 with merge 71e1628...

[bors-servo]

💔 Test failed - mac-rel-wpt2

[mbrubeck]

@bors-servo retry

• infra?

[bors-servo]

⌛ Testing commit 5f17f09 with merge 093bd1b...

[bors-servo]

💔 Test failed - linux-rel-css

[mbrubeck]

These might be actual behavior changes caused by servo/rust-harfbuzz#78:

  ▶ PASS [expected FAIL] /css-text-3_dev/html/word-break-break-all-008.htm

  ▶ FAIL [expected PASS] /css-text-3_dev/html/word-break-break-all-007.htm
  └   → /css-text-3_dev/html/word-break-break-all-007.htm 95da652e9d0cdadddf6e9bafaf1731634148d0ad
/css-text-3_dev/html/reference/word-break-break-all-ref-007.htm d192e2ee5ae282f00d2da7baf2995dea2d5ef85d
Testing 95da652e9d0cdadddf6e9bafaf1731634148d0ad == d192e2ee5ae282f00d2da7baf2995dea2d5ef85d

  ▶ FAIL [expected PASS] /css-text-3_dev/html/word-break-normal-bo-000.htm
  └   → /css-text-3_dev/html/word-break-normal-bo-000.htm b1818170ff108bd50214e99c2b4b67a2f4704e9f
/css-text-3_dev/html/reference/word-break-normal-bo-ref-000.htm 1b833a5263158622e7bf32f854f5e5a5f009420d
Testing b1818170ff108bd50214e99c2b4b67a2f4704e9f == 1b833a5263158622e7bf32f854f5e5a5f009420d

  ▶ FAIL [expected PASS] /css-text-3_dev/html/word-break-normal-hi-000.htm
  └   → /css-text-3_dev/html/word-break-normal-hi-000.htm ebe9ce78534f2ffe9f03a450a0963f0ae8b9a933
/css-text-3_dev/html/reference/word-break-normal-hi-ref-000.htm 0f747c946f460cb80065f9e4d5baad2f3ed259da
Testing ebe9ce78534f2ffe9f03a450a0963f0ae8b9a933 == 0f747c946f460cb80065f9e4d5baad2f3ed259da

[mbrubeck]

test-css failed on Mac with a different error:

  ▶ FAIL [expected PASS] /css-text-3_dev/html/word-break-normal-bo-000.htm
  └   → /css-text-3_dev/html/word-break-normal-bo-000.htm 5924dfb037ea00360d59ed709909d759eafb1948
/css-text-3_dev/html/reference/word-break-normal-bo-ref-000.htm 2c48ccbe00b5d470da2ad0a5fde778d46ee879e6
Testing 5924dfb037ea00360d59ed709909d759eafb1948 == 2c48ccbe00b5d470da2ad0a5fde778d46ee879e6

I think that all these tests were "accidentally passing" before through a combination of lack of fonts and out of date harfbuzz. Feel free to update the test metadata and file a follow-up bug about the failures.

[paulrouget]

I cannot reproduce locally. I'm blindly updating the manifest see if it works.

[paulrouget]

@mbrubeck any idea what went wrong?

[paulrouget]

@bors-servo retry

I can't reproduce the build failures locally.

[paulrouget]

@mbrubeck I can't reproduce the failure locally. Do you know if it's failing to compile because of the servo/rust-harfbuzz#78 or servo/rust-harfbuzz#80 ?

[mbrubeck]

@bors-servo try

[bors-servo]

⌛ Trying commit dffb2d6 with merge 50cd17c975655b0fe42e079b887fc6ad6794da6d...

[bors-servo]

💥 Test timed out

[jdm]

@bors-servo: retry

[bors-servo]

⌛ Trying commit dffb2d6 with merge 091871b...

[bors-servo]

💔 Test failed - mac-rel-css1

[jdm]

{"status": "PASS", "group": "default", "message": null, "stack": null, "subtest": null, "test": "/css-text-3_dev/html/word-break-break-all-007.htm", "line": 13802, "action": "test_result", "expected": "FAIL"}
{"status": "FAIL", "group": "default", "message": "/css-text-3_dev/html/word-break-break-all-008.htm 85d61c419cd6708197e352364dce716dd2ead0a1\n/css-text-3_dev/html/reference/word-break-break-all-ref-008.htm 9ca1d270e6fdcca72951b5e0d2044ee7cb950321\nTesting 85d61c419cd6708197e352364dce716dd2ead0a1 == 9ca1d270e6fdcca72951b5e0d2044ee7cb950321", "stack": null, "subtest": null, "test": "/css-text-3_dev/html/word-break-break-all-008.htm", "line": 13832, "action": "test_result", "expected": "PASS"}
{"status": "PASS", "group": "default", "message": null, "stack": null, "subtest": null, "test": "/css-text-3_dev/html/word-break-normal-hi-000.htm", "line": 13911, "action": "test_result", "expected": "FAIL"}

[mbrubeck]

That matches the Mac results from last time; it looks like several of these test failures (all-007, all-008, and normal-hi-000) need to be marked platform-specific.

[paulrouget]

Ok. Now it's working.

Can someone r+ this?

[mbrubeck]

@bors-servo r+

[bors-servo]

📌 Commit d0e9ad6 has been approved by mbrubeck

[bors-servo]

⌛ Testing commit d0e9ad6 with merge 1600647...

[bors-servo]

⛄ The build was interrupted to prioritize another pull request.

[mbrubeck]

@bors-servo treeclosed-

[bors-servo]

⌛ Testing commit d0e9ad6 with merge 564fb5c...

[bors-servo]

☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel-css, linux-rel-wpt, mac-dev-unit, mac-rel-css1, mac-rel-css2, mac-rel-wpt1, mac-rel-wpt2, mac-rel-wpt3, mac-rel-wpt4, windows-msvc-dev
Approved by: mbrubeck
Pushing 564fb5c to master...