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.

Sign up for GitHub

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

Jump to bottom

Won't running pod setup with sudo create some root-owned stuff in my user-dir? #3002

Closed
grav opened this issue Jan 4, 2015 · 3 comments · Fixed by #3072
Closed

Won't running pod setup with sudo create some root-owned stuff in my user-dir? #3002

grav opened this issue Jan 4, 2015 · 3 comments · Fixed by #3072
Labels
d1:easy An easy ticket that is a good start for first-time contributors s2:confirmed Issues that have been confirmed by a CocoaPods contributor t2:defect These are known bugs. The issue should also contain steps to reproduce. PRs welcome!

Comments

@grav
Copy link

grav commented Jan 4, 2015 

$ pod setup

"Agreeing to the Xcode/iOS license requires admin privileges, please re-run as root via sudo."

This seems like a bad suggestion to me. Won't it create a ~/.cocoapods that is owned by root?

In my case, I just launched Xcode and agreed to the ULA, and then re-ran pod setup.
@grav grav changed the title Won't running pod setup with sudo create some root-owned stuff in my user-die Won't running pod setup with sudo create some root-owned stuff in my user-dir? Jan 4, 2015
@kylef
Copy link
Contributor

kylef commented Jan 4, 2015

Any pod command will actually prevent you from running them as root and provide an error.

This isn't a message from the pod command directly, it looks like it's a message printed by another command we are running, such as git.

@kylef kylef added the t3:discussion These are issues that can be non-issues, and encompass best practices, or plans for the future. label Jan 4, 2015
@neonichu
Copy link
Member

neonichu commented Jan 7, 2015

Yah, that's probably git. Maybe we could add a check similar to homebrew to make this less confusing: https://github.com/Homebrew/homebrew/blob/15806d6687ef5a9d233449acb49c8741d5d75a50/Library/Homebrew/cmd/doctor.rb#L1145-L1151?

@kylef kylef added d1:easy An easy ticket that is a good start for first-time contributors s2:confirmed Issues that have been confirmed by a CocoaPods contributor t2:defect These are known bugs. The issue should also contain steps to reproduce. PRs welcome! and removed t3:discussion These are issues that can be non-issues, and encompass best practices, or plans for the future. labels Jan 7, 2015
@kylef
Copy link
Contributor

kylef commented Jan 7, 2015

@neonichu Totally agree, changing the scope of this issue to implement that functionality.

@ghost ghost mentioned this issue Jan 25, 2015
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
d1:easy An easy ticket that is a good start for first-time contributors s2:confirmed Issues that have been confirmed by a CocoaPods contributor t2:defect These are known bugs. The issue should also contain steps to reproduce. PRs welcome!
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

implemented Xcode license check for pod setup
3 participants
@kylef @neonichu @grav