Page MenuHomeSolus

Revert back to v3.7.3
AbandonedPublic

Authored by ZVNexus on Sun, Oct 27, 9:14 PM.

Details

Reviewers
None
Group Reviewers
Triage Team
Summary

Causes issues while building AOSP with ccache enabled.

ld.lld: error: undefined symbol: strcpy_a9

Bonito shouldn't even be using ld.lld, so this is extremely
strange.

Test Plan

Recompile AOSP with no issue.

Diff Detail

Repository
R499 ccache
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

ZVNexus created this revision.Sun, Oct 27, 9:14 PM
ZVNexus requested review of this revision.Sun, Oct 27, 9:14 PM
ZVNexus added a comment.EditedSun, Oct 27, 9:15 PM

This is still an upgrade to the version we were using prior, so semi revert I guess?

This comment was removed by ZVNexus.

Are you sure that this isn't a configuration issue for AOSP?

ZVNexus added a comment.EditedTue, Oct 29, 7:32 PM

I can't be sure. What I do know is 3.7.5 causes AOSP builds to fail on Android 10 (doesn't seem to affect Android 9) and I don't know what in AOSP was added to cause this to occur. They stopped using ccache internally for their builds so it's not like they support it anymore, but many custom distributions re-enabled it as it's still useful to us. Clean builds without ccache take hours, not having ccache as an option is not that great. What we could do is update the AOSP build instructions to include it's own version of ccache that we put in PATH so the main package wouldn't need to be changed as this seems to only affect AOSP not any other packages I build with ccache. Although, I can't see any packages Solus builds having an issue going back to 3.7.3 and it would benefit AOSP. It's your call.

JoshStrobl abandoned this revision.Tue, Oct 29, 7:51 PM
JoshStrobl added a subscriber: JoshStrobl.

Sounds more like a local configuration issue or one in Android. I don't see a need for us to downgrade this and like you stated, "They stopped using ccache internally for their builds" so doesn't even sound like something you should be using in the first place then.