Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove --ios_cpu flag #13872

Closed
wants to merge 1 commit into from
Closed

Conversation

keith
Copy link
Member

@keith keith commented Aug 19, 2021

This flag appears to be unused, it was set once but never read. Instead
users should prefer --ios_multi_cpus or --cpu=ios_XXX

@keith keith requested a review from lberki as a code owner August 19, 2021 00:23
@google-cla google-cla bot added the cla: yes label Aug 19, 2021
@keith keith force-pushed the ks/remove-ios_cpu-flag branch 2 times, most recently from 237559e to e4e72dd Compare August 19, 2021 03:31
@lberki lberki requested review from comius and removed request for lberki August 19, 2021 06:35
@lberki
Copy link
Contributor

lberki commented Aug 19, 2021

/cc @allevato

@allevato
Copy link
Member

This one would need some internal repo cleanup before we could accept it; we still have a few teams using --ios_cpu in some scripts and CI config files.

@aiuto
Copy link
Contributor

aiuto commented Dec 18, 2021

@allevato Should we close this PR and assume you will do it after the internal cleanup?

@allevato
Copy link
Member

Yes, this is cleanup that we'll most likely do after we've finished the transition over to platforms. (I don't see a need to rush it at this point; it would be better to tidy up all of that legacy stuff holistically.)

@keith keith closed this Jan 19, 2022
@keith keith deleted the ks/remove-ios_cpu-flag branch January 19, 2022 04:51
@keith keith restored the ks/remove-ios_cpu-flag branch July 15, 2022 17:50
@keith keith reopened this Jul 15, 2022
This flag appears to be unused, it was set once but never read. Instead
users should prefer `--ios_multi_cpus` or `--cpu=ios_XXX`
@keith
Copy link
Member Author

keith commented Jul 15, 2022

I reopened this one since we actually hit some related issues here. If you're getting a transition from bazel itself it includes ios_cpu, but the starlark transitions that have been written so far don't actually include it, which in some cases causes issues. We can make the external transition provide this value to match with our expectations, but given that this flag has been deprecated for quite a while now, it seems like a good chance to remove it and reduce confusion around which flag should be used.

Can someone from google remove the internal uses of this flag?

Related: bazelbuild/rules_apple#1579

@aiuto
Copy link
Contributor

aiuto commented Jul 19, 2022

@allevato, cc: @gregestren, @katre

Tony: I see fewer ios_cpu instances each month. Do you have a status update on internal movement.

@keith
Copy link
Member Author

keith commented Aug 15, 2022

seems like this was mostly handled in ecf9ac9

@keith keith closed this Aug 15, 2022
@keith keith deleted the ks/remove-ios_cpu-flag branch August 15, 2022 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants