[OpenJDK 2D-Dev] RFR: 8247872: Upgrade HarfBuzz to the latest 2.7.2 [v3]

Thu Nov 5 23:55:56 UTC 2020

On Thu, 5 Nov 2020 17:52:53 GMT, Phil Race <prr at openjdk.org> wrote:

>> @mrserb Good catch! Yes indeed, it looks like `range-loop-analysis` needs to be disabled for clang for Xcode 12 to not complain.
>
> I added that to the disabled warnings .. now I suppose the actions will re-run but I don't see any sign of it yet.
> Just that the results that were there are now gone.
> A few thoughts on this 
> - github sends so many messages that it is easy to miss the important ones
> - using the default for github we are at the whim of when ever they switch and if this change
>    had been pushed 6 weeks ago likely it would have passed until the day github upgraded.
> - it also means passing a github build does not guarantee it will pass on the "official" toolchains.

@prrace I agree, the github actions are a double-edged sword -- at best. At worst, it's a clear regression in terms of how we work. :-(

We might need to take a step back and rethink this.

Anyway, your changes looks good from a build PoV now.

-------------

PR: https://git.openjdk.java.net/jdk/pull/993