Different PODS_ROOT calculation with and without project integration #918

Closed
jerolimov opened this Issue Mar 30, 2013 · 2 comments

Comments

Projects
None yet
2 participants

Hi,

i'm still analyse this problem, but maybe some of you have an idea why this happend.

Background stuff: I think this starts with CocoaPods 0.17.0. We add the Pods directory and Podfile.lock file to our .gitignore and download the dependency on each development system (and our CI) after cloning it.

To disable the pod cmd to touch the project file each time we integrate the libPods.a, copy resource script, etc one time and commit it with the workspace.

The problem is now that the pod install and pod install --no-integrate generate different PODS_ROOT paths in the Pods project and in the Pods/Pods.xcconfig. You can see here:

$ rm -rf Pods Podfile.lock ; pod install
Analyzing dependencies
Downloading dependencies
Installing ... (all dependencies, up updates, thats ok for me)
Generating Pods project
Integrating client project
$ grep '^PODS_ROOT' Pods/Pods.xcconfig
PODS_ROOT = ${SRCROOT}/Pods
$ rm -rf Pods Podfile.lock ; pod install --no-integrate
Analyzing dependencies
Downloading dependencies
Installing ... (all dependencies, up updates, thats ok for me)
Generating Pods project
$ grep '^PODS_ROOT' Pods/Pods.xcconfig
PODS_ROOT = ${SRCROOT}/repository-name-again/Pods

Have you any idea if there is some variables are missed to set then there is no integration was called?

I found only that the result of this method was generated into the xcconfig file: https://github.com/CocoaPods/CocoaPods/blob/master/lib/cocoapods/library.rb#L156-161

For me it works to remove the --no-integrate argument from all our scripts. Seems that you check the target project carefully when integrating the dependencies.

When the project was instructed once it wasn't touched a second time. That's cool.

I keep the issue open because i think the different results are still inadvertently. 😄

Owner

fabiopelosin commented Apr 3, 2013

Thanks for the detailed report.

@jzapater jzapater pushed a commit to jzapater/CocoaPods that referenced this issue Sep 17, 2013

@shoumikhin shoumikhin Merge pull request #918 from shoumikhin/master
NSXtensions.podspec 0.3
683a618
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment