Page MenuHomeFreeBSD

www/chromium: Fix aarch64 by reapplying r527876
ClosedPublic

Authored by cem on Mar 11 2020, 4:22 PM.

Details

Summary

It was lost in the 80.x update, unfortunately. Mea culpa.

Test Plan

I don't have any way to build-test this for aarch64, looking for help.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

cem created this revision.Mar 11 2020, 4:22 PM
jrm added a comment.Mar 11 2020, 4:26 PM

Since this will change the package, PORTREVISION should be bumped.

In D24026#528366, @jrm wrote:

Since this will change the package, PORTREVISION should be bumped.

It's a tier2 arch and affects only head, no need to bump it.

jrm added a comment.Mar 11 2020, 4:41 PM

You're right. I didn't notice that the first two files were specific to aarch64/arm64 and the changes in the third file are protected by #defined(ARMV8_OS_.... Sorry for the noise.

mikael accepted this revision.Mar 11 2020, 7:25 PM

LGTM, thanks for the quick patch.

This revision is now accepted and ready to land.Mar 11 2020, 7:25 PM
This revision was automatically updated to reflect the committed changes.