8000 Issue with thinning dSYM files · Issue #7196 · CocoaPods/CocoaPods · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Issue with thinning dSYM files #7196

8000
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.

Sign up for GitHub

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

Jump to bottom
Closed
mwoollard opened this issue Nov 3, 2017 · 5 comments · Fixed by #7197
Closed

Issue with thinning dSYM files #7196

mwoollard opened this issue Nov 3, 2017 · 5 comments · Fixed by #7197
Labels
d1:easy An easy ticket that is a good start for first-time contributors r:vendored Issues related to vendored libraries and frameworks s2:confirmed Issues that have been confirmed by a CocoaPods contributor
Milestone
1.4.0

Comments

@mwoollard
Copy link

Report

What did you do?

integrate a project with ArcGIS-Runtime-SDK-iOS pod

pod install

Followed by building of resulting workspace in Xcode for simulator

What did you expect to happen?

Project to build and run on simulator

What happened instead?

When building for simulator environment project failed whilst running [CP] Embed Pod Frameworks step. The above pod vendors a binary static framework that includes dSYM files but there are only FAT slices for device architectures not simulator. When building for simulator the script fails when stripping the last device architecture lipo fails.

fatal error: lipo: -remove's specified would result in an empty fat file

When building for device rather than simulator the build succeeds since the device slices are not removed. Note that commenting out:

strip_invalid_archs "$binary"

For the dSYM files allowed the app to build and run in the simulator.

Can this case be detected and the dSYM copy skipped rather than failing with the above error?

CocoaPods Environment

Stack

   CocoaPods : 1.4.0.beta.2
        Ruby : ruby 2.3.3p222 (2016-11-21 revision 56859) [universal.x86_64-darwin17]
    RubyGems : 2.6.14
        Host : Mac OS X 10.13.1 (17B48)
       Xcode : 9.1 (9B55)
         Git : git version 2.13.6 (Apple Git-96)
Ruby lib dir : /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib
Repositories : master - https://github.com/CocoaPods/Specs.git @ 2f034bfbd3c18eef1ea398afa5cf9824c7f858fd
               urbanthings - git@bitbucket.org:urbanthings/podspecs.git @ 8a820e4853f85582b0db6eb5dd876fce61ffe322

Installation Source

Executable Path: /usr/local/bin/pod

Plugins

cocoapods-deintegrate : 1.0.1
cocoapods-plugins     : 1.0.0
cocoapods-search      : 1.0.0
cocoapods-stats       : 1.0.0
cocoapods-trunk       : 1.3.0
cocoapods-try         : 1.1.0

Podfile

use_frameworks!

target 'CPBug' do
    pod 'ArcGIS-Runtime-SDK-iOS'
end

Project that demonstrates the issue

A project to reproduce issue available here:
https://github.com/mwoollard/cpbug.git
@mwoollard mwoollard changed the title Issue with stripping dSYM files Issue with thinning dSYM files Nov 3, 2017
@dnkoutso
Copy link
Contributor
dnkoutso commented Nov 3, 2017

You should also contact the pod author to ship a fat dSYM but I do not think people will do the right thing so yes we should fix it to handle this case.

@dnkoutso dnkoutso added d1:easy An easy ticket that is a good start for first-time contributors s2:confirmed Issues that have been confirmed by a CocoaPods contributor labels Nov 3, 2017
@dnkoutso dnkoutso added this to the 1.4.0 milestone Nov 3, 2017
@mwoollard
Copy link
Author

Thanks for quick response - I suspect they've excluded the simulator dSYM slices due to the overall size of their vendors contents which is over 1GB.

@dnkoutso dnkoutso mentioned this issue Nov 3, 2017
Closed
@dnkoutso dnkoutso added the r:vendored Issues related to vendored libraries and frameworks label Nov 3, 2017
@dnkoutso dnkoutso mentioned this issue Nov 3, 2017
Merged
@dnkoutso
Copy link
Contributor
dnkoutso commented Nov 3, 2017

@mwoollard can you try this patch from this PR #7197?

@dnkoutso
Copy link
Contributor
dnkoutso commented Nov 4, 2017

Merged.

@mwoollard
Copy link
Author

Confirmed PR #7197 resolves issue. Now builds with suitable warning:

warning: [CP] Vendored binary '/Users/markw/Library/Developer/Xcode/DerivedData/CPBug-dsocezizfuodkxbqwmxvnfwfflzi/Build/Products/Debug-iphonesimulator/ArcGIS.framework.dSYM/Contents/Resources/DWARF/ArcGIS' contains architectures (armv7 arm64) none of which match the current build architectures (x86_64).

Thanks for prompt resolution!

@piellarda piellarda mentioned this issue Nov 8, 2017
Closed
@radazzouz radazzouz mentioned this issue Sep 21, 2020
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
d1:easy An easy ticket that is a good start for first-time contributors r:vendored Issues related to vendored libraries and frameworks s2:confirmed Issues that have been confirmed by a CocoaPods contributor
Projects
None yet
Milestone
1.4.0
Development

Successfully merging a pull request may close this issue.

Be more lenient when stripping frameworks and dSYMs for non fat binaries dnkoutso/CocoaPods
2 participants
@mwoollard @dnkoutso
0