Skip to content
This repository has been archived by the owner on Nov 19, 2018. It is now read-only.

Added error about granting permission #41

Merged
merged 3 commits into from
Jan 16, 2015

Conversation

joshdholtz
Copy link
Contributor

Changes

  • Waits for "instruments" process to listen for a failure
    • Let's user know how to solve if the error involves granting permission
  • Added some files to .gitignore
  • Added shared Example scheme to example Xcode project

How to test

  • Created a new user on my mac (testuser1)
  • Added testuser1 to sudoers file (because I needed to install gems and sometimes delete things when I messed up)
  • Cloned my fork
  • Ran example project and saw failure but help message 🎉 🎊

screen shot 2015-01-15 at 7 14 43 pm

@joshdholtz joshdholtz changed the title Needs to grant permission Added warning to user about granting permission Jan 16, 2015
@joshdholtz joshdholtz changed the title Added warning to user about granting permission Added error about granting permission Jan 16, 2015
@KrauseFx
Copy link
Collaborator

Thanks @joshdholtz, that's a great pull request.

Does instruments put out Instruments wants permission to analyze other processes? I haven't seen it before.

KrauseFx added a commit that referenced this pull request Jan 16, 2015
@KrauseFx KrauseFx merged commit 6f352bb into fastlane-old:master Jan 16, 2015
@KrauseFx
Copy link
Collaborator

Oh, it looks like you duplicated a scheme, I'll remove it again.

@joshdholtz
Copy link
Contributor Author

@KrauseFx Yes sir, it does spit out that "wants permission to analyze other processes" message. I can verify on some co-workers machines today.

Also, the scheme thing was weird. I opened the project right after I cloned and it had a scheme but then for some reason it stopped showing it which is why I added it.

And thanks for the merge 😁

@KrauseFx
Copy link
Collaborator

Thanks again @joshdholtz, it looks great. I published a new version already! https://github.com/KrauseFx/snapshot/releases

@fastlane-old fastlane-old locked and limited conversation to collaborators Mar 11, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants