Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Redundant static libs not being removed #1954

Closed
Daniel1of1 opened this Issue Mar 29, 2014 · 6 comments

Comments

Projects
None yet
5 participants

Directly from #1823 (comment) which seems to be a separate issue from the original #1823

Example where this can happen Wrapping pod declarations in target block will create after you previously have not eg:

    pod ...

to

target "Your_Target" do
    pod ...
end

The first will create libPods.a

The second will create libPods-Your_Target.a but original libPods.a will hang around and cause linker failure with duplicates.

Issue has been confirmed by @neonichu

Owner

fabiopelosin commented Mar 30, 2014

This will be fixed by the DSL refactor which will remove the implicit target.

@kylef kylef referenced this issue in Mantle/Mantle May 13, 2014

Closed

Weird assert #326

Contributor

kylef commented Sep 14, 2014

I believe this is fixed, the migrator was made to migrate to build configuration based libraries?

libPods.a > libPods.debug.a.

Owner

segiddins commented Sep 15, 2014

@kylef I don't think the issue described here is fixed. It's the cruft that's left behind when a target is removed.

Owner

fabiopelosin commented Sep 15, 2014

CocoaPods de-integration has never been implemented. So when you remove the dependencies of a target CocoaPods doesn't remove its integration from the client target. This feature would fix this ticket.

It is important to note the implicit target definition is adding confusion here.

I believe this is fixed, the migrator was made to migrate to build configuration based libraries?

@kylef this is not intuitive, The umbrella target (libPods.a) is added as a dependency of the user target to ensure that the single pod targets (libPodsAFNetworking.a) are built before the user target the integration is done via the xcconfigs. To clarify there is no an libPods.debug.a (because you can't add target dependencies per build configuration) but there is an Pods.debug.xcconfig file.

Contributor

kylef commented Sep 15, 2014

Superseded by #2471

@kylef kylef closed this Sep 15, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment