Generate optional info plist and entitlements

[yonaskolb]

Resolves #406

This allows an Info.plist or .entitlements file to be generated per target via the project spec.
Some basic attributes are generated by default, but app specific ones are not for now.

This features is powerful when used in combination with include and targetTemplates.

• read and generate
• basic defaults
• tests
• fine grained per platform/product defaults
• documentation

[lukewakeford]

I've just tested this functionality by adding the following to my app.yml:

info:
  path: App.plist
entitlements:
  attributes:
    com.apple.security.application-groups: group.com.app

I was expecting the entitlements file to be generated without defining a path - it wasn't so I added a path like so:

info:
  path: App.plist
entitlements:
  path: App.entitlements
  attributes:
    com.apple.security.application-groups: group.com.app

This works well and I appreciate why giving a file name is helpful.
Should the xcodegen output warn when the file was not generated due to no path provided? - or is this just something for the documentation?

Otherwise this is great, thanks again 👍

[brentleyjones]

I’ll review this tonight. Looks neat!

_{Sent with GitHawk}

[brentleyjones]

Could this be stored in a static property?

[yonaskolb]

It could indeed. I'll refactor this stuff out into a InfoPlistGenerator as well

[lukewakeford]

@yonaskolb I've just deleted my previous comments as I've got things working with a few additional yml files.

Is this likely to merge soon? - i'd like to start using this on my build servers.

[yonaskolb]

I'll work on this this weekend and get everything up to scratch. I'd like to add it to the next release.

I was expecting the entitlements file to be generated without defining a path

We couldn't make any good guesses about where to generate this path, so it's a required property for now

Should the xcodegen output warn when the file was not generated due to no path provided? - or is this just something for the documentation?

Yep, this should throw an error. I'll add this

[yonaskolb]

@lukewakeford completed. Note that attributes is now called properties