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

Bluepill w/ Xcode 11 support, more logging, minor fixes and partial revert of bugfix in PR#338 #361

Merged
merged 3 commits into from
Sep 26, 2019

Conversation

ravimandala
Copy link
Contributor

@ravimandala ravimandala commented Sep 17, 2019

I am looking for a Bluepill version to be used w/ Xcode 11.
Can you please merge this PR and generate a Bluepill release with the fix to #338 reverted and more logging added (cherry-picked a commit from your branch https://github.com/linkedin/bluepill/tree/more-logging)? I tested this on machines with Xcode 11 GM Seed#1.
Thanks. \cc @ob

Copy link
Member

@ob ob left a comment

Choose a reason for hiding this comment

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

LGTM, I just left some comments to keep the tests.

bp/src/Bluepill.m Show resolved Hide resolved
bp/tests/BluepillTests.m Show resolved Hide resolved
@ravimandala ravimandala changed the title Bazel with more logging and fix to #338 reverted Bluepill w/ Xcode 11 support, more logging, minor fixes and partial revert of bugfix in PR#338 Sep 26, 2019
@ravimandala
Copy link
Contributor Author

@ob I updated this PR with the below changes...

  • Picked changes from more-logging branch;
  • Made minor changes to fix some warnings and fixed the bluepill.sh for missing xcpretty use case;
  • Partial revert of "Fix wrong exit status (Fix wrong exit status #338)"

Can you please take another look?

Copy link
Member

@ob ob left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for picking this up and for the code cleanups.

@ob ob merged commit 7ddd815 into MobileNativeFoundation:master Sep 26, 2019
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.

None yet

2 participants