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

Podfile DSL #26

Closed
alloy opened this issue Oct 16, 2011 · 3 comments
Closed

Podfile DSL #26

alloy opened this issue Oct 16, 2011 · 3 comments
Milestone

Comments

@alloy
Copy link
Member

alloy commented Oct 16, 2011

With the addition of the `platform' attribute being important in a Podfile, it became clear that we’ll have to rethink the way in which a Podfile is defined a bit. For now (in master) I have changed it to be the same as a normal specification:

Pod::File.new do |f|
  f.platform :ios

  f.dependency 'JSONKit'
end

However, this will become more of an aesthetic issue once we, for example, add the API to define dependency groups:

Pod::File.new do |f|
  f.platform :ios

  f.dependency 'JSONKit'

  f.group :debug do
    f.dependency "Kiwi", "1.0"
    f.dependency "LRMocky", "0.9.0"
  end
end

We could create a separate DSL for a Podfile which assigns attributes without the = sign, which would make it possible to retain the previous way of defining a Podfile:

platform :ios

dependency 'JSONKit'

group :debug do
  dependency "Kiwi", "1.0"
  dependency "LRMocky", "0.9.0"
end

I’m still on the fence about this, please let me know what you think.

@betamike
Copy link

I like the DSL better. The syntax is easier to remember (especially for those who are not Rubyists), and looks more like a 'config' file syntax.

@alloy
Copy link
Member Author

alloy commented Oct 17, 2011

That’s a very good point, Mike! (Re: for the not Rubyists.)

@alloy
Copy link
Member Author

alloy commented Oct 20, 2011

This is the current show stopper for 0.1.0, which I hope to be able to implement tonight.

@alloy alloy closed this as completed in 4d9fecd Oct 20, 2011
jzapater pushed a commit to jzapater/CocoaPods that referenced this issue Sep 17, 2013
fabiopelosin added a commit that referenced this issue Oct 25, 2014
[Specification] Add support for framework_bundles
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

No branches or pull requests

2 participants