Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Expose only required headers #1249

Open
fabiopelosin opened this Issue · 4 comments

4 participants

@fabiopelosin
Owner

Currently CP exposes all the headers of all the targets in the xcconfings. This causes issues and should be fixed.

This is not possible at the moment because the :exclusive option of the Podfile DSL is used to integrate test targets and thus relies on the exposition of the headers of the parent target.

Depends on #840

@CocoaPodsBot
Collaborator

Issue has been confirmed by @segiddins

@CocoaPodsBot
Collaborator

Issue has been confirmed by @jefflovejapan

@CocoaPodsBot
Collaborator

Issue has been confirmed by @jefflovejapan

@michaelmelanson

Quoting my comment on #904:

I feel your pain, and I'm really annoyed by how the project maintainers have handled this issue. I submitted a pull request almost a year ago that was initially treated positively but eventually ignored because there was work on the go for #840 and #1249 that would fix this problem.

Now, almost a year later, this bug is still open and we cannot use the official fork of CocoaPods because of it. I'm left in the position that our fork at toushay/CocoaPods has fallen so far behind that I must bring ourselves up to date to keep using CocoaPods. But that means spending time forward porting our changes to take into account a year's worth of changes to CocoaPods.

I want an answer here about the status of this issue. If I update pull request #1590, will it be merged in? Or am I just wasting my time here?

cc @kylef @fabiopelosin

@michaelmelanson michaelmelanson referenced this issue from a commit in toushay/CocoaPods
@michaelmelanson michaelmelanson [CocoaPods/CocoaPods#1249] Separate header paths for different target…
…s in .xcconfig files.
8f32072
@michaelmelanson michaelmelanson referenced this issue from a commit in toushay/CocoaPods
@michaelmelanson michaelmelanson [CocoaPods/CocoaPods#1249] Separate header paths for different target…
…s in .xcconfig files.
0910ccb
@kylef kylef referenced this issue from a commit
@kylef kylef [CHANGELOG] Updates for #1249 7bfc1a7
@kylef kylef added the s5:blocked label
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.