Skip to content
This repository has been archived by the owner on Jun 20, 2021. It is now read-only.

Fix for building workspaces #13

Closed
wants to merge 4 commits into from
Closed

Conversation

calebhaye
Copy link
Contributor

This fix addresses issues surrounding building a project in a workspace.

  1. build.rb does not abort/fail if "Build Configurations" are not found on a workspace project. "Build Configurations" are not required for workspace builds.
  2. build.rb sets the default build configuration to Debug if one is not passed in via the --configuration or -c directives

@mattt
Copy link
Contributor

mattt commented Oct 9, 2012

Awesome work, @calebhaye! I squashed down your commits into 8423c39, which is now merged into master.

The only question I have is whether it makes sense to default to Debug, rather than Release configuration. Is there any particular reason why one makes more sense than the other?

@mattt mattt closed this Oct 9, 2012
@calebhaye
Copy link
Contributor Author

Hi Matt,

Thanks for the kind words, glad I could help :-)

With regard to build configurations, I'm not sure that Debug makes more sense than Release in general. I do think it's safer, because one could always use the Team Provisioning Profile to perform builds without having to setup app specific release profiles... which is a pain -- only since cupertino doesn't yet support it ;-)

Also, I had to unlock the keychain using system security unlock -p SYSTEM_PW ~/Library/Keychains/login.keychain

Have you ever considered integrating some of the keychain stuff into the build command? Just wondering.

@lukeredpath
Copy link

Unfortunately the intent of this pull request seems to have been reverted in 115c9b8

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants