Pod install should fail with no pods defined, always #1830

Closed
mtitolo opened this Issue Feb 20, 2014 · 6 comments

Comments

Projects
None yet
6 participants
@mtitolo
Member

mtitolo commented Feb 20, 2014

Currently, if you create a Podfile and add a target declaration, but still have no pods, it will not raise an error.

platform :ios, '7.0' 

target 'MyApp' do 

end

Expected:
If there are no pods, display the same error message that you see when you have an empty Podfile:

[!] [!] The Podfile does not contain any dependencies.
@kylef

This comment has been minimized.

Show comment Hide comment
@kylef

kylef Feb 20, 2014

Contributor

Oh, so I wasn't imagining things /cc #1735

Contributor

kylef commented Feb 20, 2014

Oh, so I wasn't imagining things /cc #1735

@CocoaPodsBot

This comment has been minimized.

Show comment Hide comment
@CocoaPodsBot

CocoaPodsBot Mar 29, 2014

@lukabratos closed with reason "Can't reproduce this issue."

@lukabratos closed with reason "Can't reproduce this issue."

@fabiopelosin

This comment has been minimized.

Show comment Hide comment
@fabiopelosin

fabiopelosin Mar 30, 2014

Owner

@mtitolo This is a feature... some users want to integrate asap (even without deps) to setup CI and other parts of their project.

Owner

fabiopelosin commented Mar 30, 2014

@mtitolo This is a feature... some users want to integrate asap (even without deps) to setup CI and other parts of their project.

@neonichu

This comment has been minimized.

Show comment Hide comment
@neonichu

neonichu Apr 20, 2014

Owner

I think we should create a new issue for all those NoMethodError issues closed as a duplicate of this, as that is still a thing. @irrationalfab ?

Owner

neonichu commented Apr 20, 2014

I think we should create a new issue for all those NoMethodError issues closed as a duplicate of this, as that is still a thing. @irrationalfab ?

@kylef kylef reopened this May 5, 2014

@kylef

This comment has been minimized.

Show comment Hide comment
@kylef

kylef May 5, 2014

Contributor

Re-opened because it's still applicable. It would be nice to automatically integrate, but I don't think that's priority. Just not showing the user an exception is sufficient to close this issue.

Contributor

kylef commented May 5, 2014

Re-opened because it's still applicable. It would be nice to automatically integrate, but I don't think that's priority. Just not showing the user an exception is sufficient to close this issue.

@fabiopelosin

This comment has been minimized.

Show comment Hide comment
@fabiopelosin

fabiopelosin Sep 15, 2014

Owner

Moving to #2472

Owner

fabiopelosin commented Sep 15, 2014

Moving to #2472

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment