edudnyk / SheeKit

Customize and resize sheets in SwiftUI with SheeKit. Utilise the power of `UISheetPresentationController` and other UIKit features.
MIT License
91 stars 7 forks source link

Library not loaded #1

Closed koedal closed 3 years ago

koedal commented 3 years ago

I added the repository as a dependency in Xcode. It builds fine but when I got to run it on device I get the following error:

dyld[880]: Library not loaded: @rpath/SheeKit.framework/SheeKit Referenced from: /private/var/containers/Bundle/Application/6DCB14C0-1835-447A-ADC9-55A89EFD9EA5/New.app/New Reason: tried: '/private/var/containers/Bundle/Application/6DCB14C0-1835-447A-ADC9-55A89EFD9EA5/New.app/Frameworks/SheeKit.framework/SheeKit' (no such file), '/private/var/containers/Bundle/Application/6DCB14C0-1835-447A-ADC9-55A89EFD9EA5/New.app/Frameworks/SheeKit.framework/SheeKit' (no such file), '/System/Library/Frameworks/SheeKit.framework/SheeKit' (no such file)

edudnyk commented 3 years ago

Do you use Swift Package Manager? Seems like you haven't configured the framework to be embedded into the app. You should have "Embed and Sign" for "SheeKit.framework" (see screenshot)

Screenshot 2021-10-06 at 03 50 04
koedal commented 3 years ago

That fixed the issue! Thanks. I did use the Swift Package Manager but wasn't aware I needed to add the Embed and Sign setting. I evaluated another project today too with Xcode -> File -> Add Packages and it worked without changing the Embed settings: https://github.com/adamfootdev/BottomSheet

Frameworks

Can that setting be set automatically when adding a package?

edudnyk commented 3 years ago

Unfortunately, no. The library syntax in the Package description only allows to specify that the library is dynamic. It would be logical for Xcode to interpret it as "Embed & Sign" by default in this case, but that does not happen. You can submit a bug report to apple about that.

edudnyk commented 3 years ago

I can make the library static so that you won't have to choose that setting. Will release it in the next version.

koedal commented 3 years ago

Dynamic is fine. You may just want to add that step to the readme. Thanks!

On Oct 5, 2021, at 11:15 PM, Eugene Dudnyk @.***> wrote:

 I can make the library static so that you won't have to choose that setting. Will release it in the next version.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.