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

Add a flag to disable Xcode version check failure #436

Conversation

jerrymarino
Copy link
Contributor

For Xcode 10 and 11, many PRs to update bluepill have required simply
updating the default SDK and Xcode version. Throughout most releases of Xcode
11, bluepill has worked without many major changes. At the speed that Apple
is moving on Xcode releases, it's a lot of effort to bump these: PRs,
branching, rebuilding, etc.

This flag makes it a bit easier for us to test new versions of Xcode
without having to create a build and PR for bluepill.

For now, it's defaulted, but perhaps we should make it default to on and print
a warning if it doesn't match. Additionally, we should consider dynamically
pulling in the runtime version

For Xcode 10 and 11, many Xcode PRs to update bluepill have required simply
updating the default SDK and Xcode version. Throughout most releases of Xcode
11.4, bluepill has worked without many major changes. At the speed that Apple
is moving on Xcode releases, it's a lot of effort to bump these: PRs,
branching, rebuilding, etc.

This flag makes it a bit easier for us to test new versions of Xcode
without having to create a build and PR for bluepill.

For now, it's defaulted, but perhaps we should make it default to on and print
a warning if it doesn't match. Additionally, we should consider dynamically
pulling in the runtime version
ravimandala
ravimandala previously approved these changes Apr 22, 2020
Copy link
Contributor

@ravimandala ravimandala left a comment

Choose a reason for hiding this comment

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

Thanks for adding this option. This might be very useful if the user knows and understands what they are doing. The unsafe prefix helps provided it is explained in README.md and in command's help. Please take a look at my comments. Thanks.

bp/src/BPConfiguration.m Outdated Show resolved Hide resolved
bp/src/BPConfiguration.m Outdated Show resolved Hide resolved
@ravimandala ravimandala merged commit 89010ca into MobileNativeFoundation:master Apr 29, 2020
RainNapper pushed a commit to RainNapper/bluepill that referenced this pull request May 8, 2020
…tion#436)

* Add a flag to disable Xcode version check failure

For Xcode 10 and 11, many Xcode PRs to update bluepill have required simply
updating the default SDK and Xcode version. Throughout most releases of Xcode
11.4, bluepill has worked without a lot of major changes. At the speed that Apple
is moving on Xcode releases, it's a lot of effort to bump these: PRs,
branching, rebuilding, etc.

This flag makes it a bit easier for us to test new versions of Xcode
without having to create a build and PR for bluepill.

For now, it's defaulted, but perhaps we should make it default to on and print
a warning if it doesn't match. Additionally, we should consider dynamically
pulling in the runtime version

Co-authored-by: Ravi Mandala <rmandala@linkedin.com>
RainNapper pushed a commit to RainNapper/bluepill that referenced this pull request May 8, 2020
…tion#436)

* Add a flag to disable Xcode version check failure

For Xcode 10 and 11, many Xcode PRs to update bluepill have required simply
updating the default SDK and Xcode version. Throughout most releases of Xcode
11.4, bluepill has worked without a lot of major changes. At the speed that Apple
is moving on Xcode releases, it's a lot of effort to bump these: PRs,
branching, rebuilding, etc.

This flag makes it a bit easier for us to test new versions of Xcode
without having to create a build and PR for bluepill.

For now, it's defaulted, but perhaps we should make it default to on and print
a warning if it doesn't match. Additionally, we should consider dynamically
pulling in the runtime version

Co-authored-by: Ravi Mandala <rmandala@linkedin.com>
RainNapper pushed a commit to jerrymarino/bluepill that referenced this pull request May 8, 2020
…tion#436)

* Add a flag to disable Xcode version check failure

For Xcode 10 and 11, many Xcode PRs to update bluepill have required simply
updating the default SDK and Xcode version. Throughout most releases of Xcode
11.4, bluepill has worked without a lot of major changes. At the speed that Apple
is moving on Xcode releases, it's a lot of effort to bump these: PRs,
branching, rebuilding, etc.

This flag makes it a bit easier for us to test new versions of Xcode
without having to create a build and PR for bluepill.

For now, it's defaulted, but perhaps we should make it default to on and print
a warning if it doesn't match. Additionally, we should consider dynamically
pulling in the runtime version

Co-authored-by: Ravi Mandala <rmandala@linkedin.com>
AhmedEid added a commit to tinyspeck/bluepill that referenced this pull request Nov 18, 2020
* master:
  Reference libXCTestBundleInject.dylib only if it exists (MobileNativeFoundation#460)
  Record video using simctl (MobileNativeFoundation#441)
  Xcode 12.0 support (MobileNativeFoundation#458)
  Retry app crash tests and consider then non-fatal if they pass (MobileNativeFoundation#456)
  Do not retry crashed tests (MobileNativeFoundation#443)
  Fixing minor issues and doing some minor refactoring/clean-up (MobileNativeFoundation#444)
  Xcode 11.5 support (MobileNativeFoundation#447)
  Xcode 11.4 support (MobileNativeFoundation#446)
  Add a flag to disable Xcode version check failure (MobileNativeFoundation#436)
  Support sidecar applications (MobileNativeFoundation#438)
  Added a few more tests to mock test failure scenarios and fixed a few final Exit Status issues  (MobileNativeFoundation#430)
  make retry checks more resilient to prevent infinite retries (MobileNativeFoundation#432)
  Xcode 11.3 support (MobileNativeFoundation#426)
  Supporting tests and app hosts in subfolders (MobileNativeFoundation#419)

# Conflicts:
#	bp/src/SimulatorHelper.m
#	bp/tests/SimulatorHelperTests.m
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.

2 participants