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

Missing headers search path for local pods #179

Closed
fabiopelosin opened this issue Mar 19, 2012 · 2 comments
Closed

Missing headers search path for local pods #179

fabiopelosin opened this issue Mar 19, 2012 · 2 comments
Assignees
Labels
t2:defect These are known bugs. The issue should also contain steps to reproduce. PRs welcome!

Comments

@fabiopelosin
Copy link
Member

Apparently the headers search paths of a pod obtained from the :local strategy are not included in the Pods.xconfig file.

Further investigation revealed that the problem is solved with the following change. Can I commit or Does it breaks anything?. It looks safe because the pod is not downloaded and therefore not cleaned, but I'm not sure if something else may be going on.

module Pod
  class LocalPod

    def root
      if !specification.local?
        @sandbox.root + specification.name
      else
        specification.local_path
      end
    end

  end
end
@ghost ghost assigned fabiopelosin Mar 19, 2012
@alloy
Copy link
Member

alloy commented Mar 19, 2012

So I think this is because, like stated in #178, the :local option is not meant for pods outside the sandbox.

fabiopelosin added a commit that referenced this issue Mar 23, 2012
* origin:
  Removed unrelated experimental changes to local_pod.rb
  [#153] Relaxed run if needed
  [#153] Hide pod setup
  [#179] Support for :local pods outside the sanbox

* master:
@fabiopelosin
Copy link
Member Author

I'm closing the issues as I'm proposing a different solution. The commit to master was introduced by accident and has been reverted.

jzapater pushed a commit to jzapater/CocoaPods that referenced this issue Sep 17, 2013
MKNetworkKit 0.83: Preserve header directory structure
fabiopelosin added a commit that referenced this issue Oct 25, 2014
* origin:
  Removed unrelated experimental changes to local_pod.rb
  [#153] Relaxed run if needed
  [#153] Hide pod setup
  [#179] Support for :local pods outside the sanbox

* master:
kylef added a commit that referenced this issue Oct 25, 2014
[Specification] Allow the name_and_version_from_string regex to match spaces
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t2:defect These are known bugs. The issue should also contain steps to reproduce. PRs welcome!
Projects
None yet
Development

No branches or pull requests

2 participants