Ubuntu One Music App for iOS

Merge lp://qastaging/~rockstar/ubuntuone-ios-music/dependencies into lp://qastaging/~ubuntuone-ios-client-team/ubuntuone-ios-music/moriarty

  1. dependencies
  2. Merge into moriarty
Proposed by Paul Hummer
Status: Merged
Merged at revision: 242
Proposed branch: lp://qastaging/~rockstar/ubuntuone-ios-music/dependencies
Merge into: lp://qastaging/~ubuntuone-ios-client-team/ubuntuone-ios-music/moriarty
Prerequisite: lp://qastaging/~rockstar/ubuntuone-ios-music/scorched-earth
Diff against target: 246 lines (+125/-17)
1 file modified
U1Music.xcodeproj/project.pbxproj (+125/-17)
To merge this branch: bzr merge lp://qastaging/~rockstar/ubuntuone-ios-music/dependencies
Reviewer Review Type Date Requested Status
Mike McCracken (community) Approve
dobey (community) Approve
Review via email: mp+135053@code.qastaging.launchpad.net

Description of the change

This branch adds the RestKit dependency.

To post a comment you must log in.
Revision history for this message
dobey (dobey) :
review: Approve
Revision history for this message
Mike McCracken (mikemc) wrote :

on line 208 and 231, this looks weird:

+ "\"$(BUILT_PRODUCTS_DIR)/../../Headers\"/**",

What does that look like in XCode's UI? Looking at the backslash and double-star, I wonder if that's actually doing anything or just getting ignored?

review: Needs Information
Revision history for this message
Paul Hummer (rockstar) wrote :

The double star is some sort of flag that means "recursive". In the UI, it's just a checkbox.

Revision history for this message
Mike McCracken (mikemc) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches