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

Fix obfuscate failed by extracting SwiftFileList file #68

Merged
merged 1 commit into from
Oct 21, 2019

Conversation

ldp940622
Copy link
Contributor

This PR wants to fix #64 and make tool compatible with Xcode11.
However, I noticed that the Circle.CI seems not worked for this repo.
Maybe we need a better way to test these changes on CI.

Copy link
Owner

@rockbruno rockbruno left a comment

Choose a reason for hiding this comment

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

This is great! Thanks a lot for fixing this!
We don't have CI on this repo, so I usually test things by hand. I have a few example apps that I run tests on.

@@ -28,7 +28,7 @@ final class XcodeProjectBuilder {
let arguments: [String] = [projectParameter, projectToBuild, "-scheme", schemeToBuild]
let cleanTask = Process()
cleanTask.launchPath = path
cleanTask.arguments = ["clean", "build"] + arguments
cleanTask.arguments = ["clean", "build"] + arguments + ["-sdk", "iphonesimulator"]
Copy link
Owner

Choose a reason for hiding this comment

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

Is this necessary on Xcode 11? What happens if you have a watchOS target on your app?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Without -sdk parameter, the build will be failed with error

Code Signing Error: "SwiftShieldExample" requires a provisioning profile. Select a provisioning profile in the Signing & Capabilities editor.
Code Signing Error: Code signing is required for product type 'Application' in SDK 'iOS 13.0'

Of course most of projects should contain these items. But if we want to obfuscate the demo project and without code sign and profiles, this job will not be successful.

Copy link
Owner

Choose a reason for hiding this comment

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

Ahhh that's intended -- for the example project, you need to sign it manually for it to work. I should update the README to reflect this.
If it works for regular projects, we can keep this change out.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, I have reverted these changes.

@rockbruno
Copy link
Owner

Thanks a lot for working on this! I'll update the READMEs separately.

@rockbruno rockbruno merged commit 077204b into rockbruno:master Oct 21, 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.

Xcode 11 beta5 obfuscate failed
2 participants