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

pin Xcode version to 15.1 & modify xcodeWrapper #19822

Merged
merged 1 commit into from May 8, 2024

Conversation

siddarthkay
Copy link
Contributor

@siddarthkay siddarthkay commented Apr 28, 2024

fixes #19770

Summary

Devs are often unaware of the supported Xcode version and often run into weird build issues with either higher or lower Xcode versions.
We currently only support Xcode 15.1
This change ensures we convey that change when devs execute make run-ios
This PR also attempts to ensure xcodebuild command is wrapped with XcodeWrapper.

Platforms

  • iOS
  • macOS

status: ready

@status-im-auto
Copy link
Member

status-im-auto commented Apr 28, 2024

Jenkins Builds

Click to see older builds (29)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ d7bf8de #1 2024-04-28 17:24:11 ~4 min tests 📄log
✔️ d7bf8de #1 2024-04-28 17:27:34 ~8 min android-e2e 🤖apk 📲
✔️ d7bf8de #1 2024-04-28 17:27:39 ~8 min android 🤖apk 📲
✔️ d7bf8de #1 2024-04-28 17:28:55 ~9 min ios 📱ipa 📲
✔️ 17b2edd #2 2024-05-03 11:39:27 ~4 min tests 📄log
✔️ 17b2edd #2 2024-05-03 11:43:04 ~8 min android 🤖apk 📲
✔️ 17b2edd #2 2024-05-03 11:43:08 ~8 min android-e2e 🤖apk 📲
✔️ 17b2edd #2 2024-05-03 11:44:16 ~9 min ios 📱ipa 📲
✔️ 47b722a #4 2024-05-06 06:24:41 ~8 min android 🤖apk 📲
✔️ 47b722a #4 2024-05-06 06:25:58 ~9 min tests 📄log
✔️ 47b722a #4 2024-05-06 06:27:49 ~11 min android-e2e 🤖apk 📲
✔️ 47b722a #4 2024-05-06 06:31:16 ~15 min ios 📱ipa 📲
✔️ 682e466 #6 2024-05-06 17:48:39 ~3 min tests 📄log
✔️ fa4ac8a #7 2024-05-06 17:56:13 ~4 min tests 📄log
✔️ fa4ac8a #7 2024-05-06 18:13:47 ~21 min android-e2e 🤖apk 📲
fa4ac8a #7 2024-05-06 18:13:51 ~21 min ios 📄log
✔️ fa4ac8a #7 2024-05-06 18:13:51 ~22 min android 🤖apk 📲
✔️ 561a0da #8 2024-05-07 03:19:43 ~4 min tests 📄log
✔️ 561a0da #8 2024-05-07 03:23:25 ~8 min android-e2e 🤖apk 📲
✔️ 561a0da #8 2024-05-07 03:23:31 ~8 min android 🤖apk 📲
✔️ 561a0da #8 2024-05-07 03:24:48 ~9 min ios 📱ipa 📲
753433c #10 2024-05-07 13:01:44 ~2 min tests 📄log
✔️ 753433c #10 2024-05-07 13:10:29 ~10 min ios 📱ipa 📲
✔️ 753433c #10 2024-05-07 13:11:00 ~11 min android-e2e 🤖apk 📲
✔️ 753433c #10 2024-05-07 13:12:30 ~12 min android 🤖apk 📲
✔️ 662efd0 #11 2024-05-07 18:18:22 ~4 min tests 📄log
✔️ 662efd0 #11 2024-05-07 18:22:06 ~8 min android-e2e 🤖apk 📲
✔️ 662efd0 #11 2024-05-07 18:22:11 ~8 min android 🤖apk 📲
✔️ 662efd0 #11 2024-05-07 18:22:12 ~8 min ios 📱ipa 📲
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 640894d #12 2024-05-08 03:37:11 ~3 min tests 📄log
✔️ 640894d #12 2024-05-08 03:39:52 ~6 min android-e2e 🤖apk 📲
✔️ 640894d #12 2024-05-08 03:41:25 ~8 min android 🤖apk 📲
✔️ 640894d #12 2024-05-08 03:42:45 ~9 min ios 📱ipa 📲
✔️ 18b19c4 #14 2024-05-08 07:08:02 ~6 min tests 📄log
✔️ 18b19c4 #14 2024-05-08 07:11:09 ~9 min ios 📱ipa 📲
✔️ 18b19c4 #14 2024-05-08 07:13:26 ~12 min android 🤖apk 📲
✔️ 18b19c4 #14 2024-05-08 07:13:31 ~12 min android-e2e 🤖apk 📲

@siddarthkay siddarthkay marked this pull request as draft April 29, 2024 13:52
@siddarthkay siddarthkay force-pushed the fix-xcode-wrapper branch 3 times, most recently from c7274a1 to 47b722a Compare May 6, 2024 06:15
Copy link
Member

@jakubgs jakubgs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Please make a PR to that effect for nixpkgs as well when you have time.

nix/overlay.nix Outdated Show resolved Hide resolved
nix/overlay.nix Outdated Show resolved Hide resolved
nix/overlay.nix Outdated Show resolved Hide resolved
@siddarthkay siddarthkay force-pushed the fix-xcode-wrapper branch 3 times, most recently from 682e466 to fa4ac8a Compare May 6, 2024 17:51
@siddarthkay siddarthkay marked this pull request as ready for review May 6, 2024 17:51
@siddarthkay
Copy link
Contributor Author

@jakubgs : I requested re-review, this PR has changed a lot from your last review.

@siddarthkay siddarthkay changed the title pin Xcode version to 15.1 & set xcodebuildWrapper pin Xcode version to 15.1 & modify xcodeWrapper May 7, 2024
@siddarthkay siddarthkay force-pushed the fix-xcode-wrapper branch 3 times, most recently from 753433c to 662efd0 Compare May 7, 2024 18:13
Copy link
Member

@jakubgs jakubgs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

nix/pkgs/xcodeenv/compose-xcodewrapper.nix Outdated Show resolved Hide resolved
@siddarthkay siddarthkay force-pushed the fix-xcode-wrapper branch 2 times, most recently from 640894d to 0f28da2 Compare May 8, 2024 06:59
fixes #19770

Devs are often unaware of the supported Xcode version and often run into weird build issues with either higher or lower Xcode versions.
This change ensures we convey that change when devs execute `make run-ios`
This commit also attempts to ensure `xcodebuild` command is wrapped with `XcodeWrapper`.

- iOS
- macOS

status: ready
@siddarthkay siddarthkay merged commit 43951b9 into develop May 8, 2024
6 checks passed
Pipeline for QA automation moved this from REVIEW to DONE May 8, 2024
@siddarthkay siddarthkay deleted the fix-xcode-wrapper branch May 8, 2024 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

Unable to run the app on ios
3 participants