Did you have a pre-implementation call with anybody about this?
Just one minor comment - can you clarify the comment on enabled_architectures to say more than just that it contains the enabled architectures? As we had a property earlier with the same name but different contents I think that would help prevent some potential confusion.
Looks good, thanks for working on this.
Did you have a pre-implementation call with anybody about this?
Just one minor comment - can you clarify the comment on enabled_ architectures to say more than just that it contains the enabled architectures? As we had a property earlier with the same name but different contents I think that would help prevent some potential confusion.
Do you need me to land this branch?