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

xcode: update to 13.4 #2409

Merged
merged 3 commits into from
Jul 19, 2022
Merged

xcode: update to 13.4 #2409

merged 3 commits into from
Jul 19, 2022

Conversation

jpsim
Copy link
Contributor

@jpsim jpsim commented Jul 6, 2022

Updates the version of Xcode Envoy Mobile is compatible with to the latest stable version (sorta, 13.4.1 also exists but doesn't have relevant changes).

Risk Level: Low.
Testing: Local & CI.
Docs Changes: Done.
Release Notes: Done.
Fixes #2100

Signed-off-by: JP Simard jp@jpsim.com

jpsim added 2 commits July 6, 2022 18:30
Signed-off-by: JP Simard <jp@jpsim.com>
Signed-off-by: JP Simard <jp@jpsim.com>
@lfpino
Copy link
Contributor

lfpino commented Jul 8, 2022

@Yannic I checked the error and it seems to be coming from https://github.com/google/xctestrunner/blob/master/simulator_control/simulator_util.py#L149 so I went ahead and ran on the MacOS 12 worker:

$ xcrun simctl
xcrun: error: unable to find utility "simctl", not a developer tool or in PATH

Are you sure you installed Command_Line_Tools_for_Xcode_13.4.dmg ? According to https://stackoverflow.com/questions/29108172/how-do-i-fix-the-xcrun-unable-to-find-simctl-error that's how you fix that kind of error.

The command succeeds in the local GH runner.

@Yannic
Copy link
Contributor

Yannic commented Jul 8, 2022

Thanks @lfpino! I need to read up on the command-line tools a bit. Will provide an update next week

@jpsim
Copy link
Contributor Author

jpsim commented Jul 8, 2022

Probably just needs this command on the executors: xcode-select --install

@jpsim jpsim marked this pull request as ready for review July 18, 2022 21:59
* main:
  tools: fix source mapping (#2429)
  Update Envoy (#2427)
  tools: add arm64 debug run configuration (#2424)
  Revert "docs: use sphinx githubpages extension (#2418)" (#2425)
  Remove Tulsi configuration & docs (#2421)
  Fix isCleartextTrafficPermitted (#2420)
  Squelch two JNI build warnings (#2426)
  dns: allow using the `getaddrinfo`-based system DNS resolver (#2419)
  docs: use sphinx githubpages extension (#2418)
  Update Envoy (#2417)
  Remove rules_cc uses
  Bump Lyft Support Rotation (#2414)
  iOS: fix retain cycles in `EnvoyNetworkMonitor` (#2415)
  Update Envoy (#2413)
  engine: remove outdated comment about `registerFactories()` (#2412)
  api: make RequestTrailersBuilder and ResponseHeadersBuilder constructors public (#2410)

Signed-off-by: JP Simard <jp@jpsim.com>
@jpsim jpsim requested a review from keith July 18, 2022 22:10
@jpsim
Copy link
Contributor Author

jpsim commented Jul 18, 2022

The EngFlow executors have been updated to have macOS 12 & Xcode 13.4 so this is now ready for review.

@jpsim jpsim merged commit 737204f into main Jul 19, 2022
@jpsim jpsim deleted the jp-xcode-13.4 branch July 19, 2022 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] Use Xcode 13.4
4 participants